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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@20e0000 {
    			gpr: iomuxc-gpr@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



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux