mbox series

[v2,0/3] xilinx: Enable Versal allocation from low memory

Message ID cover.1594384853.git.michal.simek@xilinx.com
Headers show
Series xilinx: Enable Versal allocation from low memory | expand

Message

Michal Simek July 10, 2020, 12:40 p.m. UTC
Hi,

Generated DTS files don't need to have memory nodes sorted out that's why
it can happen that the first memory node is not pointing to lowest DDR
which is normally used for system boot. That's why new function was
introduced which finds out the lowest memory available and use it for
u-boot and later can be also used for OS.

The series is based on
https://lists.denx.de/pipermail/u-boot/2020-July/419656.html
which simplified conversion to livetree API which was asked by Simon.

And also I keep conversion to livetree in separate patch on the top of v1
which shouldn't be a problem.

The series is still missing tests which Simon asked for but I would like to
get feedback about these patches. If they are fine I would like to have
discussion how/where to write little test for it.

Thanks,
Michal

Changes in v2:
- Separate Xilinx change from generic patch
- new patch asked by Simon
- split from generic patch sent in v1

Michal Simek (3):
  lib: fdt: Introduce fdtdec_setup_mem_size_base_lowest()
  lib: fdt: Convert fdtdes_setup_mem..() to livetree API
  xilinx: versal: Use lowest memory for U-Boot

 board/xilinx/versal/board.c |  2 +-
 include/fdtdec.h            | 17 +++++++
 lib/fdtdec.c                | 90 ++++++++++++++++++++++++++++---------
 3 files changed, 87 insertions(+), 22 deletions(-)

Comments

Michal Simek July 24, 2020, 12:17 p.m. UTC | #1
pá 10. 7. 2020 v 14:40 odesílatel Michal Simek <michal.simek@xilinx.com> napsal:
>

> Hi,

>

> Generated DTS files don't need to have memory nodes sorted out that's why

> it can happen that the first memory node is not pointing to lowest DDR

> which is normally used for system boot. That's why new function was

> introduced which finds out the lowest memory available and use it for

> u-boot and later can be also used for OS.

>

> The series is based on

> https://lists.denx.de/pipermail/u-boot/2020-July/419656.html

> which simplified conversion to livetree API which was asked by Simon.

>

> And also I keep conversion to livetree in separate patch on the top of v1

> which shouldn't be a problem.

>

> The series is still missing tests which Simon asked for but I would like to

> get feedback about these patches. If they are fine I would like to have

> discussion how/where to write little test for it.

>

> Thanks,

> Michal

>

> Changes in v2:

> - Separate Xilinx change from generic patch

> - new patch asked by Simon

> - split from generic patch sent in v1

>

> Michal Simek (3):

>   lib: fdt: Introduce fdtdec_setup_mem_size_base_lowest()

>   lib: fdt: Convert fdtdes_setup_mem..() to livetree API

>   xilinx: versal: Use lowest memory for U-Boot

>

>  board/xilinx/versal/board.c |  2 +-

>  include/fdtdec.h            | 17 +++++++

>  lib/fdtdec.c                | 90 ++++++++++++++++++++++++++++---------

>  3 files changed, 87 insertions(+), 22 deletions(-)

>

> --

> 2.27.0

>


Applied all.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs