On 23/8/22 18:04, Peter Maydell wrote:
Arm system emulation targets always have TARGET_BIG_ENDIAN clear, so
there is no need to have handling in armv7m_load_kernel() for the
case when it is defined.  Remove the unnecessary code.

Side notes:
  * our M-profile implementation is always little-endian (that is, it
    makes the IMPDEF choice that the read-only AIRCR.ENDIANNESS is 0)
  * if we did want to handle big-endian ELF files here we should do it
    the way that hw/arm/boot.c:arm_load_elf() does, by looking at the
    ELF header to see what endianness the file itself is

Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
---
  hw/arm/armv7m.c | 9 +--------
  1 file changed, 1 insertion(+), 8 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

Reply via email to