[PATCH 09/10] ARM: dts: sun9i: Add pinmux settings for mmc2

[Maxime Ripard]

On Sun, Dec 07, 2014 at 11:11:06PM +0800, Chen-Yu Tsai wrote:
> On Sun, Dec 7, 2014 at 11:03 PM, Maxime Ripard
> <maxime.ripard at free-electrons.com> wrote:
> > On Wed, Dec 03, 2014 at 02:36:04PM +0800, Chen-Yu Tsai wrote:
> >> mmc2 is available on port C. Add a pinmux setting for usual 4 bit
> >> wide bus, as well as one with the 4 extra pins for 8 bit wide bus.
> >>
> >> Signed-off-by: Chen-Yu Tsai <wens at csie.org>
> >> Signed-off-by: Andreas Färber <afaerber at suse.de>
> >> ---
> >>  arch/arm/boot/dts/sun9i-a80.dtsi | 15 +++++++++++++++
> >>  1 file changed, 15 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi
> >> index f5b5645..fc866d8 100644
> >> --- a/arch/arm/boot/dts/sun9i-a80.dtsi
> >> +++ b/arch/arm/boot/dts/sun9i-a80.dtsi
> >> @@ -466,6 +466,21 @@
> >>                               allwinner,pull = <0>;
> >>                       };
> >>
> >> +                     mmc2_pins: mmc2 {
> >> +                             allwinner,pins = "PC6","PC7","PC8","PC9",
> >> +                                              "PC10","PC11";
> >> +                             allwinner,function = "mmc2";
> >> +                             allwinner,drive = <2>;
> >> +                             allwinner,pull = <1>;
> >> +                     };
> >> +
> >> +                     mmc2_8bit_pins: mmc2_8bit {
> >> +                             allwinner,pins = "PC12","PC13","PC14","PC15";
> >> +                             allwinner,function = "mmc2";
> >> +                             allwinner,drive = <2>;
> >> +                             allwinner,pull = <1>;
> >> +                     };
> >> +
> >
> > Hmmm, how is that 8 bits? There's only 4 pins here.
> 
> The 8bit setting only includes the upper 4 bits.
> Using both settings together you get an 8 bit bus,
> along with CMD and CLK pins, for a total of 10 pins.
> 
> Or is this the wrong way?

The way it's usually done is that you have two standalone nodes. Just
like what's done for the UARTs for example.

Maxime


-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20141207/f30b618c/attachment-0001.sig>