Messages in this thread | | | From | Zhangjin Wu <> | Subject | Re: [PATCH v1 1/8] tools/nolibc: add support for powerpc | Date | Tue, 25 Jul 2023 14:14:11 +0800 |
| |
Hi, Thomas
> > On 2023-07-19 05:10:48+0800, Zhangjin Wu wrote: > > Both syscall declarations and _start code definition are added for > > powerpc to nolibc. > > [...] > > > > Signed-off-by: Zhangjin Wu <falcon@tinylab.org> > > --- > > tools/include/nolibc/arch-powerpc.h | 156 ++++++++++++++++++++++++++++ > > This also should be added to nolibc/arch.h. >
Thanks, it should be.
> > 1 file changed, 156 insertions(+) > > create mode 100644 tools/include/nolibc/arch-powerpc.h > > > > diff --git a/tools/include/nolibc/arch-powerpc.h b/tools/include/nolibc/arch-powerpc.h > > new file mode 100644 > > index 000000000000..100ec0f412dc > > --- /dev/null > > +++ b/tools/include/nolibc/arch-powerpc.h > > @@ -0,0 +1,156 @@ > > +/* SPDX-License-Identifier: LGPL-2.1 OR MIT */ > > +/* > > + * PowerPC specific definitions for NOLIBC > > + * Copyright (C) 2023 Zhangjin Wu <falcon@tinylab.org> > > If it is taken from musl, shouldn't there also be a musl copyright? >
For this copyright issue, I have prepared two new versions without a line from musl. even in our old version, most of them are different except the 'sc; bns+ 1f; neg %1, %1; 1:' line and the register variables.
Seems 'sc; bns+ 1f; neg %1, %1; 1:' is also used in linux kernel: arch/powerpc/include/asm/vdso/gettimeofday.h, so, it should be ok enough to apply it.
The register varibles have been changed and aligned with othe arch-<ARCH>.h locally, they are completely different now, and even further with the new syscall.h mentioned in this reply [1], the file will be completely different.
Thomas, Have added your Reviewed-by lines too, thanks a lot!
Best regards, Zhangjin ---- [1]: https://lore.kernel.org/lkml/20230725054414.15055-1-falcon@tinylab.org/
| |