On 11:12 Thu 08 Mar     , Rob Herring wrote:
> On 03/08/2012 02:50 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > For now on use i2c-gpio driver on the same pin as the hardware IP.
> > 
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
> > Cc: Nicolas Ferre <[email protected]>
> > Cc: [email protected]
> > ---
> > v3:
> > 
> >     update i2c binding (Rob comments)
> > 
> > Best Regards,
> > J.
> >  arch/arm/boot/dts/at91sam9x5.dtsi |   39 
> > +++++++++++++++++++++++++++++++++++++
> >  1 files changed, 39 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi 
> > b/arch/arm/boot/dts/at91sam9x5.dtsi
> > index c294657..fdd1ac6 100644
> > --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> > @@ -188,4 +188,43 @@
> >                     status = "disabled";
> >             };
> >     };
> > +
> > +   i2c-gpio@0 {
> > +           compatible = "i2c-gpio";
> > +           gpios = <&pioA 30 0 /* sda */
> > +                    &pioA 31 0 /* scl */
> > +                   >;
> > +           i2c-gpio,sda-open-drain;
> > +           i2c-gpio,scl-open-drain;
> > +           i2c-gpio,delay-us = <2>;        /* ~100 kHz */
> > +           #address-cells = <1>;
> > +           #size-cells = <0>;
> > +           status = "disabled";
> > +   };
> > +
> > +   i2c-gpio@1 {
> > +           compatible = "i2c-gpio";
> > +           gpios = <&pioC 0 0 /* sda */
> > +                    &pioC 1 0 /* scl */
> > +                   >;
> > +           i2c-gpio,sda-open-drain;
> > +           i2c-gpio,scl-open-drain;
> > +           i2c-gpio,delay-us = <2>;        /* ~100 kHz */
> > +           #address-cells = <1>;
> > +           #size-cells = <0>;
> > +           status = "disabled";
> > +   };
> > +
> > +   i2c-gpio@2 {
> > +           compatible = "i2c-gpio";
> > +           gpios = <&pioB 4 0 /* sda */
> > +                    &pioB 5 0 /* scl */
> > +                   >;
> > +           i2c-gpio,sda-open-drain;
> > +           i2c-gpio,scl-open-drain;
> > +           i2c-gpio,delay-us = <2>;        /* ~100 kHz */
> > +           #address-cells = <1>;
> > +           #size-cells = <0>;
> > +           status = "disabled";
> > +   };
> 
> If these are just any random gpio lines, it seems strange to define
> these in a SOC dtsi and then disable them.
they are not, they are the same pin as the i2c hardware IP.

Today the hw driver is not ready or will not work correctly (IP issue)

On hardware design we use the I2C pin but can not use the hw driver

Best Regards,
J.
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to