Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread John Crispin
>> How about we do the following. >> >> config LANTIQ >> select PINCTRL >> >> config PINCTRL_LANTIQ >> def_bool y >> depends on LANTIQ >> >> config PINCTRL_LANTIQ_XWAY >> def_bool y >> depends on SOC_TYPE_XWAY >> >> This would auto select the right symbols, have all the de

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread Stephen Warren
On 05/08/2012 09:39 AM, John Crispin wrote: >>> I don't think anyone else is going to want to compile >>> this. >> >> This Kconfig option is selected by the ARCH Kconfig, so only selected at >> the right time. The user won't get prompted for it since there's no >> string after "bool". I think this

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread John Crispin
>> I don't think anyone else is going to want to compile >> this. > > This Kconfig option is selected by the ARCH Kconfig, so only selected at > the right time. The user won't get prompted for it since there's no > string after "bool". I think this is OK. Tegra's pinctrl Kconfig option > doesn't ha

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread Stephen Warren
On 05/08/2012 07:21 AM, Linus Walleij wrote: > On Fri, May 4, 2012 at 2:18 PM, John Crispin wrote: > >> Implement support for pinctrl on lantiq/xway socs. The IO core found on these >> socs has the registers for pinctrl, pinconf and gpio mixed up in the same >> register range. As the gpio_chip ha

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread John Crispin
Hi Linus, Thanks. I will fold your comments with Stephen's into a V2 and resend it. > Shouldn't this be: > > depends on SOC_TYPE_XWAY > depends on PINCTRL_LANTIQ > > ? > > So LANTIQ selects it's pinctrl driver, the the xway SoC > selects its driver and they both are dependent on their > respectiv

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-08 Thread Linus Walleij
On Fri, May 4, 2012 at 2:18 PM, John Crispin wrote: > Implement support for pinctrl on lantiq/xway socs. The IO core found on these > socs has the registers for pinctrl, pinconf and gpio mixed up in the same > register range. As the gpio_chip handling is only a few lines, the driver also > implem

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-04 Thread John Crispin
Hi Stephen, Thanks for the comments > Overall looks pretty reasonable. Some comments below. > > There doesn't appear to be binding documentation. Something is needed to > describe how to instantiate the pinctrl driver, and the format of the > "pin configuration nodes". See > Documentation/devicetr

Re: [PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-04 Thread Stephen Warren
On 05/04/2012 06:18 AM, John Crispin wrote: > Implement support for pinctrl on lantiq/xway socs. The IO core found on these > socs has the registers for pinctrl, pinconf and gpio mixed up in the same > register range. As the gpio_chip handling is only a few lines, the driver also > implements the g

[PATCH 04/14] OF: pinctrl: MIPS: lantiq: implement lantiq/xway pinctrl support

2012-05-04 Thread John Crispin
Implement support for pinctrl on lantiq/xway socs. The IO core found on these socs has the registers for pinctrl, pinconf and gpio mixed up in the same register range. As the gpio_chip handling is only a few lines, the driver also implements the gpio functionality. This obseletes the old gpio drive