Re: [PATCH v2 1/1] hw/arm/boot: set initrd with #[address/size]-cells type in fdt

Peter Maydell writes: > On Tue, 29 Nov 2022 at 10:48, Schspa Shi wrote: >> >> We use 32bit value for linux,initrd-[start/end], when we have >> loader_start > 4GB, there will be a wrong initrd_start passed >> to the kernel, and the kernel will report the followin

[PATCH v3 1/1] hw/arm/boot: set initrd with #address-cells type in fdt

lls type. Signed-off-by: Schspa Shi -- Changelog: v1 -> v2: - Use #[address/size]-cells for data type. v2 -> v3: - Use #address-cells for all data type. - Fix indent. --- hw/arm/boot.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git

Re: [PATCH v2 1/1] hw/arm/boot: set initrd with #[address/size]-cells type in fdt

Philippe Mathieu-Daudé writes: > On 29/11/22 11:48, Schspa Shi wrote: >> We use 32bit value for linux,initrd-[start/end], when we have >> loader_start > 4GB, there will be a wrong initrd_start passed >> to the kernel, and the kernel will report the following w

[PATCH v2 1/1] hw/arm/boot: set initrd with #[address/size]-cells type in fdt

lls type. Signed-off-by: Schspa Shi -- Changelog: v1 -> v2: - Use #[address/size]-cells for data type. --- hw/arm/boot.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/hw/arm/boot.c b/hw/arm/boot.c index 57efb61ee419..98cd1fdad2c6 100644 --- a/hw/arm/

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

Peter Maydell writes: > On Tue, 8 Nov 2022 at 02:35, Schspa Shi wrote: >> >> We use 32bit value for linux,initrd-[start/end], when we have >> loader_start > 4GB, there will be a wrong initrd_start passed >> to the kernel, and the kernel will report the following

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

Peter Maydell writes: > On Wed, 16 Nov 2022 at 06:11, Schspa Shi wrote: >> >> >> Peter Maydell writes: >> >> > On Tue, 8 Nov 2022 at 15:50, Schspa Shi wrote: >> >> >> >> >> >> Peter Maydell writes:

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

Peter Maydell writes: > On Tue, 8 Nov 2022 at 15:50, Schspa Shi wrote: >> >> >> Peter Maydell writes: >> >> > On Tue, 8 Nov 2022 at 13:54, Peter Maydell >> > wrote: >> >> >> >> On Tue, 8 Nov 2022 at 12:52, Schspa Shi wro

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

Peter Maydell writes: > On Tue, 8 Nov 2022 at 13:54, Peter Maydell wrote: >> >> On Tue, 8 Nov 2022 at 12:52, Schspa Shi wrote: >> > Alex Bennée writes: >> > > There is a whole comment in boot.c talking about keeping initrd within >> > > lowme

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

Alex Bennée writes: > Schspa Shi writes: > >> We use 32bit value for linux,initrd-[start/end], when we have >> loader_start > 4GB, there will be a wrong initrd_start passed >> to the kernel, and the kernel will report the following warning. >> >>

[PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

off-by: Schspa Shi --- hw/arm/boot.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/arm/boot.c b/hw/arm/boot.c index 57efb61ee419..da719a4f8874 100644 --- a/hw/arm/boot.c +++ b/hw/arm/boot.c @@ -638,14 +638,14 @@ int arm_load_dtb(hwaddr addr, const struct arm_boot_i