From mboxrd@z Thu Jan 1 00:00:00 1970 From: Emmanuel Vadot Subject: Re: [PATCH] ARM: dts: sunxi: Enable spi1 and spi2 for Olimex A20 SOM EVB Date: Thu, 5 Jan 2017 19:04:50 +0100 Message-ID: <20170105190450.d29f27a40978bdf0faff2e65@bidouilliste.com> References: <20161226175349.17712-1-manu@bidouilliste.com> <20170105171601.us4wffupptilvvqp@lukather> <20170105183734.c56cf3e0be4ad4ac21028851@bidouilliste.com> <20170105180151.gooxj3zy53hgojah@lukather> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170105180151.gooxj3zy53hgojah@lukather> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Maxime Ripard Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, linux@armlinux.org.uk, linux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Thu, 5 Jan 2017 19:01:51 +0100 Maxime Ripard wrote: > On Thu, Jan 05, 2017 at 06:37:34PM +0100, Emmanuel Vadot wrote: > > > > Hi, > > > > On Thu, 5 Jan 2017 18:16:01 +0100 > > Maxime Ripard wrote: > > > > > Hi, > > > > > > On Mon, Dec 26, 2016 at 06:53:49PM +0100, Emmanuel Vadot wrote: > > > > Enable the spi1 and spi2 node since the pins are exposed on the UEXT > > > > connectors. > > > > > > > > Signed-off-by: Emmanuel Vadot > > > > --- > > > > arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts b/arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts > > > > index 669a1c338c76..fa8c6f60552b 100644 > > > > --- a/arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts > > > > +++ b/arch/arm/boot/dts/sun7i-a20-olimex-som-evb.dts > > > > @@ -300,12 +300,14 @@ > > > > pinctrl-names = "default"; > > > > pinctrl-0 = <&spi1_pins_a>, > > > > <&spi1_cs0_pins_a>; > > > > + status = "okay"; > > > > }; > > > > > > > > &spi2 { > > > > pinctrl-names = "default"; > > > > pinctrl-0 = <&spi2_pins_a>, > > > > <&spi2_cs0_pins_a>; > > > > + status = "okay"; > > > > }; > > > > > > Those nodes don't exist unfortunately. Maybe you forgot to send one > > > patch? > > > > > > Thanks! > > > Maxime > > > > > > -- > > > Maxime Ripard, Free Electrons > > > Embedded Linux and Kernel engineering > > > http://free-electrons.com > > > > It's based on a previous sent patch : > > http://lists.infradead.org/pipermail/linux-arm-kernel/2016-November/469288.html > > > > You said you'll squash the two commits. > > Hmmm, indeed, I might have made a mistake on this one and ended up > dropping it... :/ > > I reapplied both and squashed them together, thanks (and sorry again)! > > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com No problem, thank you! -- Emmanuel Vadot