[PATCH] ARM: dts: imx6sx: Add LDB support

Fabio Estevam festevam at denx.de
Wed May 17 16:45:09 PDT 2023


Hi Marek,

On 17/05/2023 20:15, Marek Vasut wrote:

>> diff --git a/arch/arm/boot/dts/imx6sx.dtsi 
>> b/arch/arm/boot/dts/imx6sx.dtsi
>> index 4233943a1cca..7a8a00d81666 100644
>> --- a/arch/arm/boot/dts/imx6sx.dtsi
>> +++ b/arch/arm/boot/dts/imx6sx.dtsi
>> @@ -843,8 +843,39 @@ iomuxc: pinctrl at 20e0000 {
>>     			gpr: iomuxc-gpr at 20e4000 {
>>   				compatible = "fsl,imx6sx-iomuxc-gpr",
>> -					     "fsl,imx6q-iomuxc-gpr", "syscon";
>> +					     "fsl,imx6q-iomuxc-gpr", "simple-bus", "syscon";
> 
> Can we really combine simple-bus and syscon or do we need something 
> like

Thanks for your review.

> 5a51e1f2b0834 ("arm64: dts: imx8mp: Drop simple-bus from
> fsl,imx8mp-media-blk-ctrl")

As is, if simple-bus is dropped, the LDB bus is not probed.

> 9cb6d1b39a8f5 ("soc: imx: imx8m-blk-ctrl: Scan subnodes and bind
> drivers to them")

On the i.MX6SX, I am not sure we can add devm_of_platform_populate() 
like in the
imx8m-blk-ctrl case.

imx8m-blk-ctrl has a probe function, but imx6q-iomuxc-gpr does not.

Thanks



More information about the linux-arm-kernel mailing list