Re: [Buildroot] [PATCH v2 1/1] board/xilinx: bump to linux 6.6.40

From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: "Frager, Neal" <neal.frager@amd.com>
Cc: "luca.ceresoli@bootlin.com" <luca.ceresoli@bootlin.com>,
	Brandon Maier <brandon.maier@collins.com>,
	"buildroot@buildroot.org" <buildroot@buildroot.org>,
	"romain.naour@smile.fr" <romain.naour@smile.fr>,
	"Simek, Michal" <michal.simek@amd.com>,
	"yann.morin.1998@free.fr" <yann.morin.1998@free.fr>
Subject: Re: [Buildroot] [PATCH v2 1/1] board/xilinx: bump to linux 6.6.40
Date: Fri, 9 Aug 2024 08:30:13 +0200	[thread overview]
Message-ID: <20240809083013.05d0d0fc@windsurf> (raw)
In-Reply-To: <CH2PR12MB50043F73B67812E0158B2C2CF0BA2@CH2PR12MB5004.namprd12.prod.outlook.com>

On Fri, 9 Aug 2024 03:52:30 +0000
"Frager, Neal" <neal.frager@amd.com> wrote:

> I can help you with this problem.  It is because there are two different
> versions of the zcu106 having different DDR memories.  This is because the
> DDR that came on the revA version of the board became obsolete.
> 
> I left the revA device tree in buildroot as I know this is the version of the
> zcu106 that Peter Korsgaard has.  Based on your comment, I believe you have
> the newer version of the zcu106.
> 
> To fix this, only the u-boot psu_init_gpl.c needs to be changed, so the
> correct spl psu_init routine will run.
> 
> Please implement the following in your zcu106 defconfig:
> 
> - BR2_TARGET_UBOOT_CUSTOM_MAKEOPTS="DEVICE_TREE=zynqmp-zcu106-revA
> + BR2_TARGET_UBOOT_CUSTOM_MAKEOPTS="DEVICE_TREE=zynqmp-zcu106-rev1.0
> 
> This will build in the correct psu_init_gpl.c from the two choices below:
> https://github.com/Xilinx/u-boot-xlnx/tree/master/board/xilinx/zynqmp/zynqmp-zcu106-rev1.0
> https://github.com/Xilinx/u-boot-xlnx/tree/master/board/xilinx/zynqmp/zynqmp-zcu106-revA
> 
> You can read about this issue for the zcu102 and zcu106 here:
> https://support.xilinx.com/s/article/71961?language=en_US
> 
> Do you have any opinions for how this can be better handled in buildroot to
> support both versions of the zcu102 and zcu106?

The only nice option from a Buildroot perspective would be to runtime
detect the board revision in U-Boot and apply the correct DDR
configuration accordingly. This way a single image would work on both
revisions. Having two full defconfigs to address those two board
revisions would be a bit annoying.

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot