diff mbox

[3/3] ARM: dts: imx6dl, fix i2c4 device node

Message ID 1398694356-22511-4-git-send-email-mpa@pengutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Markus Pargmann April 28, 2014, 2:12 p.m. UTC
i2c4 is exactly the same hardware module as i2c 1 to 3. The compatible
property is wrong and the appropriate clock was missing.

Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
---
 arch/arm/boot/dts/imx6dl.dtsi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Shawn Guo April 30, 2014, 6:42 a.m. UTC | #1
On Mon, Apr 28, 2014 at 04:12:36PM +0200, Markus Pargmann wrote:
> i2c4 is exactly the same hardware module as i2c 1 to 3. The compatible
> property is wrong and the appropriate clock was missing.
> 
> Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> ---
>  arch/arm/boot/dts/imx6dl.dtsi | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi
> index 9c4942f..a48a542 100644
> --- a/arch/arm/boot/dts/imx6dl.dtsi
> +++ b/arch/arm/boot/dts/imx6dl.dtsi
> @@ -84,9 +84,10 @@
>  			i2c4: i2c@021f8000 {
>  				#address-cells = <1>;
>  				#size-cells = <0>;
> -				compatible = "fsl,imx1-i2c";
> +				compatible = "fsl,imx6q-i2c", "fsl,imx21-i2c";
>  				reg = <0x021f8000 0x4000>;
>  				interrupts = <0 35 IRQ_TYPE_LEVEL_HIGH>;
> +				clocks = <&clks 209>;

It needs to be a different number when you rebase the patch on my
branch.

Shawn

>  				status = "disabled";
>  			};
>  		};
> -- 
> 1.9.1
>
Markus Pargmann April 30, 2014, 7:02 a.m. UTC | #2
On Wed, Apr 30, 2014 at 02:42:01PM +0800, Shawn Guo wrote:
> On Mon, Apr 28, 2014 at 04:12:36PM +0200, Markus Pargmann wrote:
> > i2c4 is exactly the same hardware module as i2c 1 to 3. The compatible
> > property is wrong and the appropriate clock was missing.
> > 
> > Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> > ---
> >  arch/arm/boot/dts/imx6dl.dtsi | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi
> > index 9c4942f..a48a542 100644
> > --- a/arch/arm/boot/dts/imx6dl.dtsi
> > +++ b/arch/arm/boot/dts/imx6dl.dtsi
> > @@ -84,9 +84,10 @@
> >  			i2c4: i2c@021f8000 {
> >  				#address-cells = <1>;
> >  				#size-cells = <0>;
> > -				compatible = "fsl,imx1-i2c";
> > +				compatible = "fsl,imx6q-i2c", "fsl,imx21-i2c";
> >  				reg = <0x021f8000 0x4000>;
> >  				interrupts = <0 35 IRQ_TYPE_LEVEL_HIGH>;
> > +				clocks = <&clks 209>;
> 
> It needs to be a different number when you rebase the patch on my
> branch.

I will rebase this patch and resend with the correct clock number.

Regards,

Markus
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi
index 9c4942f..a48a542 100644
--- a/arch/arm/boot/dts/imx6dl.dtsi
+++ b/arch/arm/boot/dts/imx6dl.dtsi
@@ -84,9 +84,10 @@ 
 			i2c4: i2c@021f8000 {
 				#address-cells = <1>;
 				#size-cells = <0>;
-				compatible = "fsl,imx1-i2c";
+				compatible = "fsl,imx6q-i2c", "fsl,imx21-i2c";
 				reg = <0x021f8000 0x4000>;
 				interrupts = <0 35 IRQ_TYPE_LEVEL_HIGH>;
+				clocks = <&clks 209>;
 				status = "disabled";
 			};
 		};