hw/arm: Cleanups before pflash refactor

[PATCH 00/13] hw/arm: Cleanups before pflash refactor

Philippe Mathieu-Daudé posted 13 patches 1 year, 4 months ago

Download series mbox

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

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

hw/arm/collie.c      | 25 +++++++++++-----------
hw/arm/gumstix.c     | 45 +++++++++++++++++---------------------
hw/arm/mainstone.c   | 37 +++++++++++++++-----------------
hw/arm/musicpal.c    |  9 +++++---
hw/arm/omap_sx1.c    | 51 +++++++++++++++++++-------------------------
hw/arm/pxa2xx.c      |  8 ++++---
hw/arm/spitz.c       |  6 ++----
hw/arm/tosa.c        |  2 +-
hw/arm/versatilepb.c |  6 ++----
hw/arm/vexpress.c    | 10 +--------
hw/arm/z2.c          | 16 ++++++--------
include/hw/arm/pxa.h |  5 ++---
12 files changed, 98 insertions(+), 122 deletions(-)

Expand all Fold all

[PATCH 00/13] hw/arm: Cleanups before pflash refactor

Posted by Philippe Mathieu-Daudé 1 year, 4 months ago

Various trivial cleanups:
- Remove unnecessary sysmem argument from pxa2xx_init()
- Use IEC binary prefix definitions
- Remove unreachable code calling pflash_cfi01_register()
  (another use in PPC sam460ex will be removed separately).

Philippe Mathieu-Daudé (13):
  hw/arm/pxa2xx: Simplify pxa255_init()
  hw/arm/pxa2xx: Simplify pxa270_init()
  hw/arm/collie: Use the IEC binary prefix definitions
  hw/arm/collie: Simplify flash creation using for() loop
  hw/arm/gumstix: Improve documentation
  hw/arm/gumstix: Use the IEC binary prefix definitions
  hw/arm/mainstone: Use the IEC binary prefix definitions
  hw/arm/musicpal: Use the IEC binary prefix definitions
  hw/arm/omap_sx1: Remove unused 'total_ram' definitions
  hw/arm/omap_sx1: Use the IEC binary prefix definitions
  hw/arm/z2: Use the IEC binary prefix definitions
  hw/arm/vexpress: Remove dead code in vexpress_common_init()
  hw/arm: Remove unreachable code calling pflash_cfi01_register()

 hw/arm/collie.c      | 25 +++++++++++-----------
 hw/arm/gumstix.c     | 45 +++++++++++++++++---------------------
 hw/arm/mainstone.c   | 37 +++++++++++++++-----------------
 hw/arm/musicpal.c    |  9 +++++---
 hw/arm/omap_sx1.c    | 51 +++++++++++++++++++-------------------------
 hw/arm/pxa2xx.c      |  8 ++++---
 hw/arm/spitz.c       |  6 ++----
 hw/arm/tosa.c        |  2 +-
 hw/arm/versatilepb.c |  6 ++----
 hw/arm/vexpress.c    | 10 +--------
 hw/arm/z2.c          | 16 ++++++--------
 include/hw/arm/pxa.h |  5 ++---
 12 files changed, 98 insertions(+), 122 deletions(-)

-- 
2.38.1

Re: [PATCH 00/13] hw/arm: Cleanups before pflash refactor

Posted by Peter Maydell 1 year, 4 months ago

On Mon, 9 Jan 2023 at 11:53, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> Various trivial cleanups:
> - Remove unnecessary sysmem argument from pxa2xx_init()
> - Use IEC binary prefix definitions
> - Remove unreachable code calling pflash_cfi01_register()
>   (another use in PPC sam460ex will be removed separately).
>



Applied to target-arm.next, thanks.

-- PMM

Re: [PATCH 00/13] hw/arm: Cleanups before pflash refactor

Posted by Richard Henderson 1 year, 4 months ago

On 1/9/23 03:53, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (13):
>    hw/arm/pxa2xx: Simplify pxa255_init()
>    hw/arm/pxa2xx: Simplify pxa270_init()
>    hw/arm/collie: Use the IEC binary prefix definitions
>    hw/arm/collie: Simplify flash creation using for() loop
>    hw/arm/gumstix: Improve documentation
>    hw/arm/gumstix: Use the IEC binary prefix definitions
>    hw/arm/mainstone: Use the IEC binary prefix definitions
>    hw/arm/musicpal: Use the IEC binary prefix definitions
>    hw/arm/omap_sx1: Remove unused 'total_ram' definitions
>    hw/arm/omap_sx1: Use the IEC binary prefix definitions
>    hw/arm/z2: Use the IEC binary prefix definitions
>    hw/arm/vexpress: Remove dead code in vexpress_common_init()
>    hw/arm: Remove unreachable code calling pflash_cfi01_register()

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

r~