Hi Marek,
On 11/28/24 4:25 AM, Marek Vasut wrote:
The DRAM configuration GPIO straps must be accessible in all boot phases because the GPIOs are sampled both in U-Boot SPL as well as early on in U-Boot proper. The already present bootph-pre-ram DT property made the GPIO controllers available in SPL, but not early on in U-Boot proper, which made gpio_request_list_by_name_nodev() in board dmo_get_memcfg() fail. Add missing bootph-all DT property to all GPIO controllers which are referenced in dmo,ram-coding-gpios DT property to make those GPIO controllers available early in U-Boot proper as well.
Fixes: 8c103c33fb14 ("dm: dts: Convert driver model tags to use new schema") Signed-off-by: Marek Vasut marex@denx.de
Cc: "NXP i.MX U-Boot Team" uboot-imx@nxp.com Cc: Fabio Estevam festevam@gmail.com Cc: Quentin Schulz quentin.schulz@cherry.de Cc: Sean Anderson seanga2@gmail.com Cc: Simon Glass sjg@chromium.org Cc: Stefano Babic sbabic@denx.de Cc: Tom Rini trini@konsulko.com Cc: u-boot@lists.denx.de
NOTE: For v2025.01
arch/arm/dts/imx8mm-data-modul-edm-sbc-u-boot.dtsi | 1 + arch/arm/dts/imx8mp-data-modul-edm-sbc-u-boot.dtsi | 2 ++ 2 files changed, 3 insertions(+)
diff --git a/arch/arm/dts/imx8mm-data-modul-edm-sbc-u-boot.dtsi b/arch/arm/dts/imx8mm-data-modul-edm-sbc-u-boot.dtsi index a235e088fa4..babe51ebdc1 100644 --- a/arch/arm/dts/imx8mm-data-modul-edm-sbc-u-boot.dtsi +++ b/arch/arm/dts/imx8mm-data-modul-edm-sbc-u-boot.dtsi @@ -77,6 +77,7 @@
&gpio2 { bootph-pre-ram;
- bootph-all;
aren't those supposed to be bootph-some-ram for matching U-Boot proper pre-reloc? bootph-all would enable it in TPL as well (if there's one on imx8mp).
https://docs.u-boot.org/en/latest/develop/driver-model/design.html#pre-reloc...
If not, isn't bootph-pre-ram implied when bootph-all is selected?
Cheers, Quentin