[PATCH v2 0/6] hw/arm: Create the RAM in the board

Philippe Mathieu-Daudé posted 6 patches 6 years ago
Test FreeBSD passed
Test checkpatch passed
Test docker-clang@ubuntu passed
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191021190653.9511-1-philmd@redhat.com
Maintainers: Alistair Francis <alistair@alistair23.me>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Andrzej Zaborowski <balrogg@gmail.com>, Peter Maydell <peter.maydell@linaro.org>, Antony Pavlov <antonynpavlov@gmail.com>
hw/arm/collie.c       |  8 ++++++--
hw/arm/digic_boards.c |  9 ++-------
hw/arm/mps2-tz.c      |  3 ++-
hw/arm/mps2.c         |  3 ++-
hw/arm/nseries.c      | 10 +++++++---
hw/arm/omap1.c        | 12 +++++-------
hw/arm/omap2.c        | 13 +++++--------
hw/arm/omap_sx1.c     |  8 ++++++--
hw/arm/palm.c         |  8 ++++++--
hw/arm/strongarm.c    |  7 +------
hw/arm/strongarm.h    |  4 +---
hw/arm/xilinx_zynq.c  |  3 ++-
include/hw/arm/omap.h | 10 +++-------
13 files changed, 48 insertions(+), 50 deletions(-)
[PATCH v2 0/6] hw/arm: Create the RAM in the board
Posted by Philippe Mathieu-Daudé 6 years ago
Hi Peter,

This series contains the ARM patches extracted from the
"Let the machine be the owner of the system memory" series [1]
reviewed by Richard.

These are cleanups moving the creation of the system ram at
the board level. The other series will enforce
memory_region_allocate_system_memory() get called with a
MachineState argument, this preliminary cleanup is required.

Regard,

Phil.

[1] https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg05085.html

Philippe Mathieu-Daudé (6):
  hw/arm/xilinx_zynq: Use the IEC binary prefix definitions
  hw/arm/mps2: Use the IEC binary prefix definitions
  hw/arm/collie: Create the RAM in the board
  hw/arm/omap2: Create the RAM in the board
  hw/arm/omap1: Create the RAM in the board
  hw/arm/digic4: Inline digic4_board_setup_ram() function

 hw/arm/collie.c       |  8 ++++++--
 hw/arm/digic_boards.c |  9 ++-------
 hw/arm/mps2-tz.c      |  3 ++-
 hw/arm/mps2.c         |  3 ++-
 hw/arm/nseries.c      | 10 +++++++---
 hw/arm/omap1.c        | 12 +++++-------
 hw/arm/omap2.c        | 13 +++++--------
 hw/arm/omap_sx1.c     |  8 ++++++--
 hw/arm/palm.c         |  8 ++++++--
 hw/arm/strongarm.c    |  7 +------
 hw/arm/strongarm.h    |  4 +---
 hw/arm/xilinx_zynq.c  |  3 ++-
 include/hw/arm/omap.h | 10 +++-------
 13 files changed, 48 insertions(+), 50 deletions(-)

-- 
2.21.0


Re: [PATCH v2 0/6] hw/arm: Create the RAM in the board
Posted by Alistair Francis 6 years ago
On Mon, Oct 21, 2019 at 12:07 PM Philippe Mathieu-Daudé
<philmd@redhat.com> wrote:
>
> Hi Peter,
>
> This series contains the ARM patches extracted from the
> "Let the machine be the owner of the system memory" series [1]
> reviewed by Richard.
>
> These are cleanups moving the creation of the system ram at
> the board level. The other series will enforce
> memory_region_allocate_system_memory() get called with a
> MachineState argument, this preliminary cleanup is required.

For the whole series:

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

>
> Regard,
>
> Phil.
>
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg05085.html
>
> Philippe Mathieu-Daudé (6):
>   hw/arm/xilinx_zynq: Use the IEC binary prefix definitions
>   hw/arm/mps2: Use the IEC binary prefix definitions
>   hw/arm/collie: Create the RAM in the board
>   hw/arm/omap2: Create the RAM in the board
>   hw/arm/omap1: Create the RAM in the board
>   hw/arm/digic4: Inline digic4_board_setup_ram() function
>
>  hw/arm/collie.c       |  8 ++++++--
>  hw/arm/digic_boards.c |  9 ++-------
>  hw/arm/mps2-tz.c      |  3 ++-
>  hw/arm/mps2.c         |  3 ++-
>  hw/arm/nseries.c      | 10 +++++++---
>  hw/arm/omap1.c        | 12 +++++-------
>  hw/arm/omap2.c        | 13 +++++--------
>  hw/arm/omap_sx1.c     |  8 ++++++--
>  hw/arm/palm.c         |  8 ++++++--
>  hw/arm/strongarm.c    |  7 +------
>  hw/arm/strongarm.h    |  4 +---
>  hw/arm/xilinx_zynq.c  |  3 ++-
>  include/hw/arm/omap.h | 10 +++-------
>  13 files changed, 48 insertions(+), 50 deletions(-)
>
> --
> 2.21.0
>
>

Re: [PATCH v2 0/6] hw/arm: Create the RAM in the board
Posted by Peter Maydell 6 years ago
On Mon, 21 Oct 2019 at 20:07, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
>
> Hi Peter,
>
> This series contains the ARM patches extracted from the
> "Let the machine be the owner of the system memory" series [1]
> reviewed by Richard.
>
> These are cleanups moving the creation of the system ram at
> the board level. The other series will enforce
> memory_region_allocate_system_memory() get called with a
> MachineState argument, this preliminary cleanup is required.
>
> Regard,
>
> Phil.
>
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg05085.html
>
> Philippe Mathieu-Daudé (6):
>   hw/arm/xilinx_zynq: Use the IEC binary prefix definitions
>   hw/arm/mps2: Use the IEC binary prefix definitions
>   hw/arm/collie: Create the RAM in the board
>   hw/arm/omap2: Create the RAM in the board
>   hw/arm/omap1: Create the RAM in the board
>   hw/arm/digic4: Inline digic4_board_setup_ram() function



Applied to target-arm.next, thanks.

-- PMM