[PATCH] ARM: Kirkwood: convert ns2 boards to pinctrl
Simon Guinot
simon.guinot at sequanux.org
Tue Jan 8 10:09:07 EST 2013
On Tue, Jan 08, 2013 at 03:49:54PM +0100, Andrew Lunn wrote:
> On Tue, Jan 08, 2013 at 03:36:18PM +0100, Simon Guinot wrote:
> > Note that the pinctrl conversion also fixes GPIO support for ns2 boards.
> > Since commit f9e75922: "ARM: Kirkwood: Make use of mvebu pincltl and
> > gpio", the mvbu_gpio driver is used for DT boards. As mvbu_gpio relies
> > on the pinctrl driver, then a pinctrl definition must be given to allow
> > the GPIO configuration.
> >
> > Signed-off-by: Simon Guinot <simon.guinot at sequanux.org>
> > ---
> > arch/arm/boot/dts/kirkwood-ns2-common.dtsi | 16 ++++++++++++
> > arch/arm/mach-kirkwood/board-ns2.c | 38 ----------------------------
> > 2 files changed, 16 insertions(+), 38 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/kirkwood-ns2-common.dtsi b/arch/arm/boot/dts/kirkwood-ns2-common.dtsi
> > index 9bc6785..77d21ab 100644
> > --- a/arch/arm/boot/dts/kirkwood-ns2-common.dtsi
> > +++ b/arch/arm/boot/dts/kirkwood-ns2-common.dtsi
> > @@ -1,4 +1,5 @@
> > /include/ "kirkwood.dtsi"
> > +/include/ "kirkwood-6281.dtsi"
> >
> > / {
> > chosen {
> > @@ -6,6 +7,21 @@
> > };
> >
> > ocp at f1000000 {
> > + pinctrl: pinctrl at 10000 {
> > + pinctrl-0 = < &pmx_spi &pmx_twsi0 &pmx_uart0
> > + &pmx_ns2_sata0 &pmx_ns2_sata1>;
> > + pinctrl-names = "default";
> > +
> > + pmx_ns2_sata0: pmx-ns2-sata0 {
> > + marvell,pins = "mpp21";
> > + marvell,function = "sata0";
> > + };
> > + pmx_ns2_sata1: pmx-ns2-sata1 {
> > + marvell,pins = "mpp20";
> > + marvell,function = "sata1";
> > + };
> > + };
>
> Hi Simon
>
> Its normal to also list all the gpio pins here as well.
I was not sure about that. As gpio_request triggers the mpp
configuration at run time, IMHO there is no point for an initial and
statical configuration, except for a documentation purpose. As GPIOs
are also referenced in their dedicated driver nodes, I think that the
informations about the pin are quite exhaustive. All the non-GPIO MPPs
are listed in the pinctrl node. The other MPPs are GPIOs if requested
by a driver. It is simple :)
Do you think that the full MPP/GPIO list is really needed ?
Simon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130108/5c06e391/attachment-0001.sig>
More information about the linux-arm-kernel
mailing list