From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH v9 05/17] ARM: dts: sun4i: Add pin-muxing info for the mmc0 controller Date: Tue, 29 Apr 2014 10:20:39 +0200 Message-ID: <535F60D7.1080500@redhat.com> References: <1398507381-5614-1-git-send-email-hdegoede@redhat.com> <1398507381-5614-6-git-send-email-hdegoede@redhat.com> <20140428193210.GF3134@lukather> Reply-To: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <20140428193210.GF3134@lukather> List-Post: , List-Help: , List-Archive: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Subscribe: , List-Unsubscribe: , To: Maxime Ripard Cc: Chris Ball , Ulf Hansson , Emilio Lopez , Mike Turquette , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, =?ISO-8859-1?Q?David_Lanzend=F6rfer?= , linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree List-Id: devicetree@vger.kernel.org Hi, On 04/28/2014 09:32 PM, Maxime Ripard wrote: > Hi Hans, > > On Sat, Apr 26, 2014 at 12:16:09PM +0200, Hans de Goede wrote: >> mmc0 is the only controller actually being used on boards, so limit the >> pin-muxing options to that. >> >> Signed-off-by: Hans de Goede >> --- >> arch/arm/boot/dts/sun4i-a10.dtsi | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi >> index dce074b..29fd4f5 100644 >> --- a/arch/arm/boot/dts/sun4i-a10.dtsi >> +++ b/arch/arm/boot/dts/sun4i-a10.dtsi >> @@ -565,6 +565,20 @@ >> allwinner,drive = <0>; >> allwinner,pull = <0>; >> }; >> + >> + mmc0_pins_a: mmc0@0 { >> + allwinner,pins = "PF0","PF1","PF2","PF3","PF4","PF5"; >> + allwinner,function = "mmc0"; >> + allwinner,drive = <2>; >> + allwinner,pull = <0>; >> + }; >> + >> + mmc0_cd_pin_a: mmc0_cd_pin@0 { >> + allwinner,pins = "PH1"; >> + allwinner,function = "gpio_in"; >> + allwinner,drive = <0>; >> + allwinner,pull = <1>; >> + }; > > Actually, I like very much the reference_pin mention you had in your > bindings documentation. Could you use it for this and the other > patches? Ok will fix this up for the next version. Regards, Hans