On Sat, Dec 26, 2020 at 01:09:02AM +0100, Lukasz Majewski wrote: > On the XEA board (imx28) one needs in the SPL support for GPIO, MMC and > SPI. Two last ones are necessary for booting the device. The GPIO support > allows deciding which medium will be used. For example the GPIO DTS node > (gpio@0 at imx28.dtsi) has pinctrl parent (pinctrl@80018000) for which we > don't need driver asigned for correct operation. > In the spl/dts/dt-platdata.c the gpio@0 has index 4 and its parent - > pinctrl@80018000 has index 5. > > In the bind_drivers_pass() function (at drivers/core/lists.c) call to > device_bind_by_name() for `fsl_imx23_pinctrl` returns -2, which is > expected. > > With current setup - when the SPL_OF_PLATDATA_PARENT=y > The gpio@0 node with index 4 is skipped as its parent with 5 is not yet > bound. It cannot be as we don't need and provide the driver for it. > As a result the gpio@0 is never bound and we end up with bricked board in > the SPL stage. > > When CONFIG_SPL_OF_PLATDATA_PARENT is NOT set, all entries from > spl/dts/dt-platdata.c are scanned in ascending index order, so gpio@0 is > properly initialized. For `fsl_imx_pinctrl` we simply check 10 times if > the driver for is available (which is not) and exit. > > As a result the GPIOs are initialized and can be used in early SPL stage. > This commit fixes XEA regression introduced with e41651fffda7da55f6. > > Signed-off-by: Lukasz Majewski <lu...@denx.de> >
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
