RE: [PATCH V2 3/6] ARM: davinci: da850: add DT node for I2C0
On Sun, Feb 03, 2013 at 18:19:05, Nori, Sekhar wrote:
>
>
> On 1/29/2013 1:08 PM, Vishwanathrao Badarkhe, Manish wrote:
> > Add I2C0 device tree node information to da850-evm.
> > Also, add I2C0 pin muxing information in da850-evm.
> >
> > Signed-off-by: Vishwanathrao Badarkhe, Manish
> > ---
> > Changes since V1:
> > - Updated i2c0 node names in dts and dtsi file.
> > - Removed interrupt parent from i2c0 node.
> > - Handled i2c0 pin mux inside i2c0 node.
> >
> > :100644 100644 433027f... c9ed683... M arch/arm/boot/dts/da850-evm.dts
> > :100644 100644 5e0eb5c... 245ab9a... M arch/arm/boot/dts/da850.dtsi
> > arch/arm/boot/dts/da850-evm.dts |5 +
> > arch/arm/boot/dts/da850.dtsi| 15 +++
> > 2 files changed, 20 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/da850-evm.dts
> > b/arch/arm/boot/dts/da850-evm.dts index 433027f..c9ed683 100644
> > --- a/arch/arm/boot/dts/da850-evm.dts
> > +++ b/arch/arm/boot/dts/da850-evm.dts
> > @@ -27,6 +27,11 @@
> > serial2: serial@1d0d000 {
> > status = "okay";
> > };
> > + i2c0: i2c0@1c22000 {
>
> This should be:
>
> i2c0: i2c@1c22000
>
> inline with what has been done in rest of this file.
Ok, I will update this name everywhere.
>
> > + status = "okay";
> > + pinctrl-names = "default";
> > + pinctrl-0 = <_pins>;
> > + };
> > };
> > nand_cs3@6200 {
> > status = "okay";
> > diff --git a/arch/arm/boot/dts/da850.dtsi
> > b/arch/arm/boot/dts/da850.dtsi index 5e0eb5c..245ab9a 100644
> > --- a/arch/arm/boot/dts/da850.dtsi
> > +++ b/arch/arm/boot/dts/da850.dtsi
> > @@ -56,6 +56,12 @@
> > 0x30 0x0110 0x0ff0
> > >;
> > };
> > + i2c0_pins: pinmux_i2c0_pins {
> > + pinctrl-single,bits = <
> > + /* I2C0_SDA,I2C0_SCL */
> > + 0x10 0x2200 0xff00
> > + >;
> > + };
> > };
> > serial0: serial@1c42000 {
> > compatible = "ns16550a";
> > @@ -81,6 +87,15 @@
> > interrupts = <61>;
> > status = "disabled";
> > };
> > + i2c0: i2c0@1c22000 {
> > + compatible = "ti,davinci-i2c";
> > + reg = <0x22000 0x1000>;
> > + clock-frequency = <10>;
>
> The clock frequency is board specific. This should come from da850-evm.dts.
Ok, I will add this board specific information in da850-evm.dts.
>
> Also, I think you can merge the auxdata addition with this patch. Can you
> post the i2c0 support separate from this series, so it can be taken
> independently? Looks like other patches have some dependencies/acks needed.
Sure, I will merge auxdata addition with this patch and post this patch
independently from this series.
>
> Thanks,
> Sekhar
>
Regards,
Manish
N�r��yb�X��ǧv�^�){.n�+{zX����ܨ}���Ơz�:+v���zZ+��+zf���h���~i���z� �w���?�&�)ߢf��^jǫy�m��@A�a���
0��h���i
Re: [PATCH V2 3/6] ARM: davinci: da850: add DT node for I2C0
On 1/29/2013 1:08 PM, Vishwanathrao Badarkhe, Manish wrote:
> Add I2C0 device tree node information to da850-evm.
> Also, add I2C0 pin muxing information in da850-evm.
>
> Signed-off-by: Vishwanathrao Badarkhe, Manish
> ---
> Changes since V1:
> - Updated i2c0 node names in dts and dtsi file.
> - Removed interrupt parent from i2c0 node.
> - Handled i2c0 pin mux inside i2c0 node.
>
> :100644 100644 433027f... c9ed683... March/arm/boot/dts/da850-evm.dts
> :100644 100644 5e0eb5c... 245ab9a... March/arm/boot/dts/da850.dtsi
> arch/arm/boot/dts/da850-evm.dts |5 +
> arch/arm/boot/dts/da850.dtsi| 15 +++
> 2 files changed, 20 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index 433027f..c9ed683 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -27,6 +27,11 @@
> serial2: serial@1d0d000 {
> status = "okay";
> };
> + i2c0: i2c0@1c22000 {
This should be:
i2c0: i2c@1c22000
inline with what has been done in rest of this file.
> + status = "okay";
> + pinctrl-names = "default";
> + pinctrl-0 = <_pins>;
> + };
> };
> nand_cs3@6200 {
> status = "okay";
> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
> index 5e0eb5c..245ab9a 100644
> --- a/arch/arm/boot/dts/da850.dtsi
> +++ b/arch/arm/boot/dts/da850.dtsi
> @@ -56,6 +56,12 @@
> 0x30 0x0110 0x0ff0
> >;
> };
> + i2c0_pins: pinmux_i2c0_pins {
> + pinctrl-single,bits = <
> + /* I2C0_SDA,I2C0_SCL */
> + 0x10 0x2200 0xff00
> + >;
> + };
> };
> serial0: serial@1c42000 {
> compatible = "ns16550a";
> @@ -81,6 +87,15 @@
> interrupts = <61>;
> status = "disabled";
> };
> + i2c0: i2c0@1c22000 {
> + compatible = "ti,davinci-i2c";
> + reg = <0x22000 0x1000>;
> + clock-frequency = <10>;
The clock frequency is board specific. This should come from da850-evm.dts.
Also, I think you can merge the auxdata addition with this patch. Can
you post the i2c0 support separate from this series, so it can be taken
independently? Looks like other patches have some dependencies/acks needed.
Thanks,
Sekhar
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH V2 3/6] ARM: davinci: da850: add DT node for I2C0
On Sun, Feb 03, 2013 at 18:19:05, Nori, Sekhar wrote:
On 1/29/2013 1:08 PM, Vishwanathrao Badarkhe, Manish wrote:
Add I2C0 device tree node information to da850-evm.
Also, add I2C0 pin muxing information in da850-evm.
Signed-off-by: Vishwanathrao Badarkhe, Manish manish...@ti.com
---
Changes since V1:
- Updated i2c0 node names in dts and dtsi file.
- Removed interrupt parent from i2c0 node.
- Handled i2c0 pin mux inside i2c0 node.
:100644 100644 433027f... c9ed683... M arch/arm/boot/dts/da850-evm.dts
:100644 100644 5e0eb5c... 245ab9a... M arch/arm/boot/dts/da850.dtsi
arch/arm/boot/dts/da850-evm.dts |5 +
arch/arm/boot/dts/da850.dtsi| 15 +++
2 files changed, 20 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts
b/arch/arm/boot/dts/da850-evm.dts index 433027f..c9ed683 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -27,6 +27,11 @@
serial2: serial@1d0d000 {
status = okay;
};
+ i2c0: i2c0@1c22000 {
This should be:
i2c0: i2c@1c22000
inline with what has been done in rest of this file.
Ok, I will update this name everywhere.
+ status = okay;
+ pinctrl-names = default;
+ pinctrl-0 = i2c0_pins;
+ };
};
nand_cs3@6200 {
status = okay;
diff --git a/arch/arm/boot/dts/da850.dtsi
b/arch/arm/boot/dts/da850.dtsi index 5e0eb5c..245ab9a 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -56,6 +56,12 @@
0x30 0x0110 0x0ff0
;
};
+ i2c0_pins: pinmux_i2c0_pins {
+ pinctrl-single,bits =
+ /* I2C0_SDA,I2C0_SCL */
+ 0x10 0x2200 0xff00
+ ;
+ };
};
serial0: serial@1c42000 {
compatible = ns16550a;
@@ -81,6 +87,15 @@
interrupts = 61;
status = disabled;
};
+ i2c0: i2c0@1c22000 {
+ compatible = ti,davinci-i2c;
+ reg = 0x22000 0x1000;
+ clock-frequency = 10;
The clock frequency is board specific. This should come from da850-evm.dts.
Ok, I will add this board specific information in da850-evm.dts.
Also, I think you can merge the auxdata addition with this patch. Can you
post the i2c0 support separate from this series, so it can be taken
independently? Looks like other patches have some dependencies/acks needed.
Sure, I will merge auxdata addition with this patch and post this patch
independently from this series.
Thanks,
Sekhar
Regards,
Manish
N�r��yb�X��ǧv�^�){.n�+{zX����ܨ}���Ơz�j:+v���zZ+��+zf���h���~i���z� �w���?��)ߢf��^jǫy�m��@A�a���
0��h���i
Re: [PATCH V2 3/6] ARM: davinci: da850: add DT node for I2C0
On 1/29/2013 1:08 PM, Vishwanathrao Badarkhe, Manish wrote:
Add I2C0 device tree node information to da850-evm.
Also, add I2C0 pin muxing information in da850-evm.
Signed-off-by: Vishwanathrao Badarkhe, Manish manish...@ti.com
---
Changes since V1:
- Updated i2c0 node names in dts and dtsi file.
- Removed interrupt parent from i2c0 node.
- Handled i2c0 pin mux inside i2c0 node.
:100644 100644 433027f... c9ed683... March/arm/boot/dts/da850-evm.dts
:100644 100644 5e0eb5c... 245ab9a... March/arm/boot/dts/da850.dtsi
arch/arm/boot/dts/da850-evm.dts |5 +
arch/arm/boot/dts/da850.dtsi| 15 +++
2 files changed, 20 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 433027f..c9ed683 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -27,6 +27,11 @@
serial2: serial@1d0d000 {
status = okay;
};
+ i2c0: i2c0@1c22000 {
This should be:
i2c0: i2c@1c22000
inline with what has been done in rest of this file.
+ status = okay;
+ pinctrl-names = default;
+ pinctrl-0 = i2c0_pins;
+ };
};
nand_cs3@6200 {
status = okay;
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 5e0eb5c..245ab9a 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -56,6 +56,12 @@
0x30 0x0110 0x0ff0
;
};
+ i2c0_pins: pinmux_i2c0_pins {
+ pinctrl-single,bits =
+ /* I2C0_SDA,I2C0_SCL */
+ 0x10 0x2200 0xff00
+ ;
+ };
};
serial0: serial@1c42000 {
compatible = ns16550a;
@@ -81,6 +87,15 @@
interrupts = 61;
status = disabled;
};
+ i2c0: i2c0@1c22000 {
+ compatible = ti,davinci-i2c;
+ reg = 0x22000 0x1000;
+ clock-frequency = 10;
The clock frequency is board specific. This should come from da850-evm.dts.
Also, I think you can merge the auxdata addition with this patch. Can
you post the i2c0 support separate from this series, so it can be taken
independently? Looks like other patches have some dependencies/acks needed.
Thanks,
Sekhar
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
