RE: [PATCH 1/2] arm/dts: Add AM33XX basic pinctrl support

2012-07-09 Thread AnilKumar, Chimata 
Hi Tony,

Thanks for reviewing.

On Fri, Jul 06, 2012 at 13:53:44, Tony Lindgren wrote:
> * AnilKumar Ch  [120705 02:18]:
> > Adds basic pinctrl support for AM33XX family of devices. This patch
> > is based on the pinctrl-simple driver submitted by Tony Lindgren's
> > here: http://lwn.net/Articles/496075/
> > 
> > Signed-off-by: AnilKumar Ch 
> > ---
> >  arch/arm/boot/dts/am33xx.dtsi |   12 
> >  1 file changed, 12 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> > index 59509c4..85def31 100644
> > --- a/arch/arm/boot/dts/am33xx.dtsi
> > +++ b/arch/arm/boot/dts/am33xx.dtsi
> > @@ -40,6 +40,18 @@
> > };
> > };
> >  
> > +   am3358_pinmux: pinmux@44E10800 {
> > +   compatible = "ti,omap4-padconf";
> > +   reg = <0x44E10800 0x0338>;
> > +   #address-cells = <1>;
> > +   #size-cells = <0>;
> > +   #pinctrl-cells = <2>;
> > +   pinctrl-simple,register-width = <32>;
> > +   pinctrl-simple,function-mask = <0x7>;
> > +   pinctrl-simple,function-off = <0x>;
> > +   pinctrl-simple,pinconf-mask = <0x78>;
> > +   };
> > +
> 
> You might want to update to the latest version, which is now called
> pinctrl-single instead of pinctrl-simple. Should be easy to update,
> note that we're still waiting on people to comment on the binding,
> so you might want to wait a bit before reposting so we have the
> driver merged.

In that case, I will submit the patches with respect to pinctrl-single
after it is available in linux-next tree.

Thanks
AnilKumar

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] arm/dts: Add AM33XX basic pinctrl support

2012-07-06 Thread Tony Lindgren 
* AnilKumar Ch  [120705 02:18]:
> Adds basic pinctrl support for AM33XX family of devices. This patch
> is based on the pinctrl-simple driver submitted by Tony Lindgren's
> here: http://lwn.net/Articles/496075/
> 
> Signed-off-by: AnilKumar Ch 
> ---
>  arch/arm/boot/dts/am33xx.dtsi |   12 
>  1 file changed, 12 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index 59509c4..85def31 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -40,6 +40,18 @@
>   };
>   };
>  
> + am3358_pinmux: pinmux@44E10800 {
> + compatible = "ti,omap4-padconf";
> + reg = <0x44E10800 0x0338>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + #pinctrl-cells = <2>;
> + pinctrl-simple,register-width = <32>;
> + pinctrl-simple,function-mask = <0x7>;
> + pinctrl-simple,function-off = <0x>;
> + pinctrl-simple,pinconf-mask = <0x78>;
> + };
> +

You might want to update to the latest version, which is now called
pinctrl-single instead of pinctrl-simple. Should be easy to update,
note that we're still waiting on people to comment on the binding,
so you might want to wait a bit before reposting so we have the
driver merged.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/2] arm/dts: Add AM33XX basic pinctrl support

2012-07-05 Thread AnilKumar Ch 
Adds basic pinctrl support for AM33XX family of devices. This patch
is based on the pinctrl-simple driver submitted by Tony Lindgren's
here: http://lwn.net/Articles/496075/

Signed-off-by: AnilKumar Ch 
---
 arch/arm/boot/dts/am33xx.dtsi |   12 
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 59509c4..85def31 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -40,6 +40,18 @@
};
};
 
+   am3358_pinmux: pinmux@44E10800 {
+   compatible = "ti,omap4-padconf";
+   reg = <0x44E10800 0x0338>;
+   #address-cells = <1>;
+   #size-cells = <0>;
+   #pinctrl-cells = <2>;
+   pinctrl-simple,register-width = <32>;
+   pinctrl-simple,function-mask = <0x7>;
+   pinctrl-simple,function-off = <0x>;
+   pinctrl-simple,pinconf-mask = <0x78>;
+   };
+
/*
 * XXX: Use a flat representation of the AM33XX interconnect.
 * The real AM33XX interconnect network is quite complex.Since
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html