* [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
@ 2013-06-20 21:08 ` Anna-Maria Gleixner
0 siblings, 0 replies; 6+ messages in thread
From: Anna-Maria Gleixner @ 2013-06-20 21:08 UTC (permalink / raw)
To: linux-tegra-u79uwXL29TY76Z2rM5mHXA
Cc: Joseph Lo, Stephen Warren, linux-kernel-u79uwXL29TY76Z2rM5mHXA
The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
tristate. This results in non working i2c, because the i2c pins are
not actively driven. Set the entries to "driven".
Signed-off-by: Anna-Maria Gleixner <anna-maria-WyZXDcvM26CzQB+pC5nmwQ@public.gmane.org>
diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
index 4441620..ad699bb 100644
--- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
+++ b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
@@ -86,13 +86,13 @@
nvidia,pins = "rm";
nvidia,function = "i2c1";
nvidia,pull = <0>;
- nvidia,tristate = <1>;
+ nvidia,tristate = <0>;
};
i2c3 {
nvidia,pins = "dtf";
nvidia,function = "i2c3";
nvidia,pull = <0>;
- nvidia,tristate = <1>;
+ nvidia,tristate = <0>;
};
i2cddc {
nvidia,pins = "ddc";
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
@ 2013-06-20 21:08 ` Anna-Maria Gleixner
0 siblings, 0 replies; 6+ messages in thread
From: Anna-Maria Gleixner @ 2013-06-20 21:08 UTC (permalink / raw)
To: linux-tegra; +Cc: Joseph Lo, Stephen Warren, linux-kernel
The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
tristate. This results in non working i2c, because the i2c pins are
not actively driven. Set the entries to "driven".
Signed-off-by: Anna-Maria Gleixner <anna-maria@glx-um.de>
diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
index 4441620..ad699bb 100644
--- a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
+++ b/arch/arm/boot/dts/tegra20-colibri-512.dtsi
@@ -86,13 +86,13 @@
nvidia,pins = "rm";
nvidia,function = "i2c1";
nvidia,pull = <0>;
- nvidia,tristate = <1>;
+ nvidia,tristate = <0>;
};
i2c3 {
nvidia,pins = "dtf";
nvidia,function = "i2c3";
nvidia,pull = <0>;
- nvidia,tristate = <1>;
+ nvidia,tristate = <0>;
};
i2cddc {
nvidia,pins = "ddc";
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
2013-06-20 21:08 ` Anna-Maria Gleixner
@ 2013-06-21 16:24 ` Stephen Warren
-1 siblings, 0 replies; 6+ messages in thread
From: Stephen Warren @ 2013-06-21 16:24 UTC (permalink / raw)
To: Lucas Stach
Cc: Anna-Maria Gleixner, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
Joseph Lo, linux-kernel-u79uwXL29TY76Z2rM5mHXA
On 06/20/2013 03:08 PM, Anna-Maria Gleixner wrote:
> The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
> tristate. This results in non working i2c, because the i2c pins are
> not actively driven. Set the entries to "driven".
Lucas, can you please comment on this patch? It sounds correct to me,
but I would have assumed you'd tested I2C already when you wrote the
original DT?
Oh actually, now that I look further, I see that tegra20-iris-512.dts
includes this file, and overrides some of the tristate values there.
What's the thinking behind that, and does this patch fit into it?
> diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> nvidia,pins = "rm";
> nvidia,function = "i2c1";
> nvidia,pull = <0>;
> - nvidia,tristate = <1>;
> + nvidia,tristate = <0>;
> };
> i2c3 {
> nvidia,pins = "dtf";
> nvidia,function = "i2c3";
> nvidia,pull = <0>;
> - nvidia,tristate = <1>;
> + nvidia,tristate = <0>;
> };
> i2cddc {
> nvidia,pins = "ddc";
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
@ 2013-06-21 16:24 ` Stephen Warren
0 siblings, 0 replies; 6+ messages in thread
From: Stephen Warren @ 2013-06-21 16:24 UTC (permalink / raw)
To: Lucas Stach; +Cc: Anna-Maria Gleixner, linux-tegra, Joseph Lo, linux-kernel
On 06/20/2013 03:08 PM, Anna-Maria Gleixner wrote:
> The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
> tristate. This results in non working i2c, because the i2c pins are
> not actively driven. Set the entries to "driven".
Lucas, can you please comment on this patch? It sounds correct to me,
but I would have assumed you'd tested I2C already when you wrote the
original DT?
Oh actually, now that I look further, I see that tegra20-iris-512.dts
includes this file, and overrides some of the tristate values there.
What's the thinking behind that, and does this patch fit into it?
> diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
> nvidia,pins = "rm";
> nvidia,function = "i2c1";
> nvidia,pull = <0>;
> - nvidia,tristate = <1>;
> + nvidia,tristate = <0>;
> };
> i2c3 {
> nvidia,pins = "dtf";
> nvidia,function = "i2c3";
> nvidia,pull = <0>;
> - nvidia,tristate = <1>;
> + nvidia,tristate = <0>;
> };
> i2cddc {
> nvidia,pins = "ddc";
^ permalink raw reply [flat|nested] 6+ messages in thread[parent not found: <51C47E41.6050609-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]
* Re: [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
2013-06-21 16:24 ` Stephen Warren
@ 2013-06-21 17:05 ` Lucas Stach
-1 siblings, 0 replies; 6+ messages in thread
From: Lucas Stach @ 2013-06-21 17:05 UTC (permalink / raw)
To: Stephen Warren
Cc: Anna-Maria Gleixner, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
Joseph Lo, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Am Freitag, den 21.06.2013, 10:24 -0600 schrieb Stephen Warren:
> On 06/20/2013 03:08 PM, Anna-Maria Gleixner wrote:
> > The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
> > tristate. This results in non working i2c, because the i2c pins are
> > not actively driven. Set the entries to "driven".
>
> Lucas, can you please comment on this patch? It sounds correct to me,
> but I would have assumed you'd tested I2C already when you wrote the
> original DT?
>
This is a NACK from me. For robustness reasons all pins/busses not
actually used on the Colibri module itself are tristated in order to
avoid any invalid configurations when placed on a baseboard.
Baseboards using particular busses should untristate those pins in their
respective dts. So if this change is about iris, I would be ok with
enabling i2c1, as this is actually routed on this baseboard, but not
i2c3.
Regards,
Lucas
> Oh actually, now that I look further, I see that tegra20-iris-512.dts
> includes this file, and overrides some of the tristate values there.
> What's the thinking behind that, and does this patch fit into it?
>
> > diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
>
> > nvidia,pins = "rm";
> > nvidia,function = "i2c1";
> > nvidia,pull = <0>;
> > - nvidia,tristate = <1>;
> > + nvidia,tristate = <0>;
> > };
> > i2c3 {
> > nvidia,pins = "dtf";
> > nvidia,function = "i2c3";
> > nvidia,pull = <0>;
> > - nvidia,tristate = <1>;
> > + nvidia,tristate = <0>;
> > };
> > i2cddc {
> > nvidia,pins = "ddc";
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux
@ 2013-06-21 17:05 ` Lucas Stach
0 siblings, 0 replies; 6+ messages in thread
From: Lucas Stach @ 2013-06-21 17:05 UTC (permalink / raw)
To: Stephen Warren; +Cc: Anna-Maria Gleixner, linux-tegra, Joseph Lo, linux-kernel
Am Freitag, den 21.06.2013, 10:24 -0600 schrieb Stephen Warren:
> On 06/20/2013 03:08 PM, Anna-Maria Gleixner wrote:
> > The nvidia,tristate entry for the pinmux of i2c1 an i2c3 was set to
> > tristate. This results in non working i2c, because the i2c pins are
> > not actively driven. Set the entries to "driven".
>
> Lucas, can you please comment on this patch? It sounds correct to me,
> but I would have assumed you'd tested I2C already when you wrote the
> original DT?
>
This is a NACK from me. For robustness reasons all pins/busses not
actually used on the Colibri module itself are tristated in order to
avoid any invalid configurations when placed on a baseboard.
Baseboards using particular busses should untristate those pins in their
respective dts. So if this change is about iris, I would be ok with
enabling i2c1, as this is actually routed on this baseboard, but not
i2c3.
Regards,
Lucas
> Oh actually, now that I look further, I see that tegra20-iris-512.dts
> includes this file, and overrides some of the tristate values there.
> What's the thinking behind that, and does this patch fit into it?
>
> > diff --git a/arch/arm/boot/dts/tegra20-colibri-512.dtsi
>
> > nvidia,pins = "rm";
> > nvidia,function = "i2c1";
> > nvidia,pull = <0>;
> > - nvidia,tristate = <1>;
> > + nvidia,tristate = <0>;
> > };
> > i2c3 {
> > nvidia,pins = "dtf";
> > nvidia,function = "i2c3";
> > nvidia,pull = <0>;
> > - nvidia,tristate = <1>;
> > + nvidia,tristate = <0>;
> > };
> > i2cddc {
> > nvidia,pins = "ddc";
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-06-21 17:14 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-20 21:08 [PATCH] ARM:tegra20-colibri-512.dts: Change tristate status for i2c pinmux Anna-Maria Gleixner
2013-06-20 21:08 ` Anna-Maria Gleixner
2013-06-21 16:24 ` Stephen Warren
2013-06-21 16:24 ` Stephen Warren
[not found] ` <51C47E41.6050609-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-06-21 17:05 ` Lucas Stach
2013-06-21 17:05 ` Lucas Stach
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.