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 <ma...@denx.de> --- Cc: "NXP i.MX U-Boot Team" <uboot-...@nxp.com> Cc: Fabio Estevam <feste...@gmail.com> Cc: Quentin Schulz <quentin.sch...@cherry.de> Cc: Sean Anderson <sean...@gmail.com> Cc: Simon Glass <s...@chromium.org> Cc: Stefano Babic <sba...@denx.de> Cc: Tom Rini <tr...@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-relocation-support If not, isn't bootph-pre-ram implied when bootph-all is selected? Cheers, Quentin
