hw/arm: Remove big-endian checks on machine code

[PATCH 0/5] hw/arm: Remove big-endian checks on machine code

Philippe Mathieu-Daudé posted 5 patches 5 years, 8 months ago

Download series mbox

Test docker-quick@centos7 failed

Test FreeBSD failed

Test docker-mingw@fedora failed

Test checkpatch failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200228125351.29482-1-philmd@redhat.com

Maintainers: "Philippe Mathieu-Daudé" <philmd@redhat.com>, Jan Kiszka <jan.kiszka@web.de>, Andrzej Zaborowski <balrogg@gmail.com>, Peter Maydell <peter.maydell@linaro.org>

hw/arm/gumstix.c   | 16 ++--------------
hw/arm/mainstone.c |  8 +-------
hw/arm/musicpal.c  | 10 ----------
hw/arm/omap_sx1.c  | 11 ++---------
hw/arm/z2.c        |  8 +-------
5 files changed, 6 insertions(+), 47 deletions(-)

Expand all Fold all

[PATCH 0/5] hw/arm: Remove big-endian checks on machine code

Posted by Philippe Mathieu-Daudé 5 years, 8 months ago

We only build the little-endian softmmu configurations.
Checking for big endian is pointless, remove the unused code.

Philippe Mathieu-Daudé (5):
  hw/arm/gumstix: Simplify since the machines are little-endian only
  hw/arm/mainstone: Simplify since the machines are little-endian only
  hw/arm/omap_sx1: Simplify since the machines are little-endian only
  hw/arm/z2: Simplify since the machines are little-endian only
  hw/arm/musicpal: Simplify since the machines are little-endian only

 hw/arm/gumstix.c   | 16 ++--------------
 hw/arm/mainstone.c |  8 +-------
 hw/arm/musicpal.c  | 10 ----------
 hw/arm/omap_sx1.c  | 11 ++---------
 hw/arm/z2.c        |  8 +-------
 5 files changed, 6 insertions(+), 47 deletions(-)

-- 
2.21.1

Re: [PATCH 0/5] hw/arm: Remove big-endian checks on machine code

Posted by Peter Maydell 5 years, 8 months ago

On Fri, 28 Feb 2020 at 12:53, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> We only build the little-endian softmmu configurations.
> Checking for big endian is pointless, remove the unused code.
>
> Philippe Mathieu-Daudé (5):
>   hw/arm/gumstix: Simplify since the machines are little-endian only
>   hw/arm/mainstone: Simplify since the machines are little-endian only
>   hw/arm/omap_sx1: Simplify since the machines are little-endian only
>   hw/arm/z2: Simplify since the machines are little-endian only
>   hw/arm/musicpal: Simplify since the machines are little-endian only
>
>  hw/arm/gumstix.c   | 16 ++--------------
>  hw/arm/mainstone.c |  8 +-------
>  hw/arm/musicpal.c  | 10 ----------
>  hw/arm/omap_sx1.c  | 11 ++---------
>  hw/arm/z2.c        |  8 +-------
>  5 files changed, 6 insertions(+), 47 deletions(-)



Applied to target-arm.next, thanks.

-- PMM

Re: [PATCH 0/5] hw/arm: Remove big-endian checks on machine code

Posted by Richard Henderson 5 years, 8 months ago

On 2/28/20 4:53 AM, Philippe Mathieu-Daudé wrote:
> We only build the little-endian softmmu configurations.
> Checking for big endian is pointless, remove the unused code.
> 
> Philippe Mathieu-Daudé (5):
>   hw/arm/gumstix: Simplify since the machines are little-endian only
>   hw/arm/mainstone: Simplify since the machines are little-endian only
>   hw/arm/omap_sx1: Simplify since the machines are little-endian only
>   hw/arm/z2: Simplify since the machines are little-endian only
>   hw/arm/musicpal: Simplify since the machines are little-endian only

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~