public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marcus Comstedt <marcus@mc.pp.se>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/2] riscv: tools: Add big endian target support to prelink-riscv
Date: Wed, 07 Aug 2019 22:36:20 +0200	[thread overview]
Message-ID: <yf98ss4r9vf.fsf@mc.pp.se> (raw)
In-Reply-To: <CAN5B=eL8RKZ1nefDRDfCbXsakq2A3wrg-ZEnAvFfrKL2uDhrXg@mail.gmail.com> (Rick Chen's message of "Wed, 7 Aug 2019 16:49:08 +0800")


Hi Rick,

Rick Chen <rickchen36@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.  :-)


  // Marcus

  reply	other threads:[~2019-08-07 20:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-02 17:45 [U-Boot] [PATCH v2 1/2] riscv: tools: Fix prelink-riscv to work on big endian hosts Marcus Comstedt
2019-08-02 17:45 ` [U-Boot] [PATCH v2 2/2] riscv: tools: Add big endian target support to prelink-riscv Marcus Comstedt
     [not found]   ` <752D002CFF5D0F4FA35C0100F1D73F3FA40FA9E8@ATCPCS16.andestech.com>
2019-08-07  8:49     ` Rick Chen
2019-08-07 20:36       ` Marcus Comstedt [this message]
     [not found]         ` <752D002CFF5D0F4FA35C0100F1D73F3FA40FAF35@ATCPCS16.andestech.com>
2019-08-08  2:34           ` Rick Chen
     [not found] ` <752D002CFF5D0F4FA35C0100F1D73F3FA40FA9DF@ATCPCS16.andestech.com>
2019-08-07  8:44   ` [U-Boot] [PATCH v2 1/2] riscv: tools: Fix prelink-riscv to work on big endian hosts Rick Chen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=yf98ss4r9vf.fsf@mc.pp.se \
    --to=marcus@mc.pp.se \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox