From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH V3 4/4] SPEAr3xx: Add pinctrl support for boards Date: Mon, 16 Apr 2012 09:07:47 +0530 Message-ID: <4F8B940B.7030700@st.com> References: <81b04bd4fe751633802e539d724facce80e600a4.1334207964.git.viresh.kumar@st.com> <4F8851E5.7050509@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4F8851E5.7050509-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Stephen Warren Cc: "devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.org" , spear-devel , "viresh.linux-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org On 4/13/2012 9:48 PM, Stephen Warren wrote: > On 04/11/2012 11:24 PM, Viresh Kumar wrote: >> Signed-off-by: Viresh Kumar > > Just a couple of minor comments below. No Probs. >> diff --git a/arch/arm/boot/dts/spear300-evb.dts b/arch/arm/boot/dts/spear300-evb.dts > >> + pinmux@99000000 { >> + st,pinmux-mode=<2>; > > spaces around = here for consistency? Ok. >> + state_default: pinmux { >> + i2c0 { >> + st,pins = "i2c0_grp"; >> + st,function = "i2c0"; >> + }; > > Do the group names need the "_grp" suffix? If that's what's in your TRM, > it's fine by me, but I'd guess that isn't the case, and if so I'd say > either drop the _grp suffix here, or enhance the binding documentation > to mention the need to add this suffix when writing the .dts file. Updated Documentation clearly for this. As you have spent a lot of time getting these reviewed, i would add your Reviewed-by on all these patches. Hope that would be fine? Will fix all these, resend them and send a pull request to Arnd. -- viresh From mboxrd@z Thu Jan 1 00:00:00 1970 From: viresh.kumar@st.com (Viresh Kumar) Date: Mon, 16 Apr 2012 09:07:47 +0530 Subject: [PATCH V3 4/4] SPEAr3xx: Add pinctrl support for boards In-Reply-To: <4F8851E5.7050509@wwwdotorg.org> References: <81b04bd4fe751633802e539d724facce80e600a4.1334207964.git.viresh.kumar@st.com> <4F8851E5.7050509@wwwdotorg.org> Message-ID: <4F8B940B.7030700@st.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 4/13/2012 9:48 PM, Stephen Warren wrote: > On 04/11/2012 11:24 PM, Viresh Kumar wrote: >> Signed-off-by: Viresh Kumar > > Just a couple of minor comments below. No Probs. >> diff --git a/arch/arm/boot/dts/spear300-evb.dts b/arch/arm/boot/dts/spear300-evb.dts > >> + pinmux at 99000000 { >> + st,pinmux-mode=<2>; > > spaces around = here for consistency? Ok. >> + state_default: pinmux { >> + i2c0 { >> + st,pins = "i2c0_grp"; >> + st,function = "i2c0"; >> + }; > > Do the group names need the "_grp" suffix? If that's what's in your TRM, > it's fine by me, but I'd guess that isn't the case, and if so I'd say > either drop the _grp suffix here, or enhance the binding documentation > to mention the need to add this suffix when writing the .dts file. Updated Documentation clearly for this. As you have spent a lot of time getting these reviewed, i would add your Reviewed-by on all these patches. Hope that would be fine? Will fix all these, resend them and send a pull request to Arnd. -- viresh