The iomux-gpr syscon device has a very loose register definition. Many
registers have a multi purpose, e.g. one register controlls the WDG
behavior and the CSI2 mux settings. This would lead into multiple
sub-devices which use the same 'reg' which is wrong.

Instead of specifying the 'reg' in the OF each sub-device driver needs
to handle the correct bits on their own. Therefore drop the 'reg'
porperty from the bridge node as well as the @18 register suffix.

Signed-off-by: Marco Felsch <[email protected]>
---
 arch/arm/boot/dts/nxp/imx/imx6sx.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi 
b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
index 
aefae5a3a6bebd43c67070376196db559d0d20cb..4b974b54195950e8bd2ef6d590f3530b2a43106e
 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6sx.dtsi
@@ -850,9 +850,8 @@ gpr: syscon@20e4000 {
                                #size-cells = <1>;
                                reg = <0x020e4000 0x4000>;
 
-                               lvds_bridge: bridge@18 {
+                               lvds_bridge: bridge {
                                        compatible = "fsl,imx6sx-ldb";
-                                       reg = <0x18 0x4>;
                                        clocks = <&clks IMX6SX_CLK_LDB_DI0>;
                                        clock-names = "ldb";
                                        status = "disabled";

-- 
2.47.3

Reply via email to