[U-Boot] [PATCH v2 2/2] riscv: tools: Add big endian target support to prelink-riscv

Rick Chen rickchen36 at gmail.com
Thu Aug 8 02:34:41 UTC 2019


Hi Marcus

> > From: Marcus Comstedt [mailto:marcus at mc.pp.se]
> > Sent: Thursday, August 08, 2019 4:36 AM
> > To: Rick Chen
> > Cc: U-Boot Mailing List; Rick Jian-Zhi Chen(陳建志); K.C. Kuen-Chern Lin(林坤
> > 成)
> > Subject: Re: [PATCH v2 2/2] riscv: tools: Add big endian target support to
> > prelink-riscv
> >
> >
> > Hi Rick,
> >
> > Rick Chen <rickchen36 at gmail.com> writes:
> >
> > > Thanks for your efforts about the support for big endian ELFs.
> > > But I don't have big endian toolchain to verify.
> >
> > I don't think anyone has a big endian toolchain for riscv at this juncture.  :-)
> > (No specification for big/bi endian has been nailed down yet, AFAIK, although
> > there has been discussion about how to implement it.)
> >
> > I expect the patch to work for symmetry reasons - the places that need
> > byteswapping when the host is little endian and the target is big endian are
> > exactly the same as the places that need byteswapping when the host is big
> > endian and the target is little endian (a case which I have tested), so it's just a
> > matter of keeping track of the endianness of the host (something which
> > U-Boot does internally already) versus the endianness of the target (which is
> > handled by this patch).
> >
> > Merging this right now is probably about as useful as merging 128-bit ELF
> > support, but I don't think it really adds any maintainence burden, so it might
> > be simpler than keeping it around in patchwork.  It's your call.  :-)
> >

Thanks for your explanation. :)

I have applied to u-boot-riscv/master, thanks!

Regards,
Rick

> >
> >   // Marcus
> >
>


More information about the U-Boot mailing list