From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Sun, 11 May 2014 18:17:27 +0000 Subject: Re: [PATCH] ARM: shmobile: henninger: add QSPI DT support Message-Id: <536FBEB7.9080605@cogentembedded.com> List-Id: References: <201405060245.32952.sergei.shtylyov@cogentembedded.com> In-Reply-To: <201405060245.32952.sergei.shtylyov@cogentembedded.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Hello. On 05/06/2014 02:45 AM, Sergei Shtylyov wrote: > Define the Henninger board dependent part of the QSPI device node. > Add device nodes for Spansion S25FL512S SPI flash and MTD partitions on it. > Signed-off-by: Sergei Shtylyov > --- > This patch is against 'renesas-devel-v3.15-rc3-20140502' tag of Simon Horman's > 'renesas.git' repo and SDHI0/2 DT support patch posted last weekend. > arch/arm/boot/dts/r8a7791-henninger.dts | 38 ++++++++++++++++++++++++++++++++ > 1 file changed, 38 insertions(+) > Index: renesas/arch/arm/boot/dts/r8a7791-henninger.dts > =================================> --- renesas.orig/arch/arm/boot/dts/r8a7791-henninger.dts > +++ renesas/arch/arm/boot/dts/r8a7791-henninger.dts [...] > @@ -158,3 +163,36 @@ > cd-gpios = <&gpio6 22 GPIO_ACTIVE_LOW>; > status = "okay"; > }; > + > +&qspi { > + pinctrl-0 = <&qspi_pins>; > + pinctrl-names = "default"; > + > + status = "okay"; > + > + flash@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "spansion,s25fl512s"; > + reg = <0>; > + spi-max-frequency = <30000000>; > + spi-tx-bus-width = <4>; > + spi-rx-bus-width = <4>; > + m25p,fast-read; > + > + partition@0 { > + label = "loader_prg"; > + reg = <0x00000000 0x00040000>; > + read-only; > + }; > + partition@40000 { > + label = "user_prg"; > + reg = <0x00040000 0x00400000>; > + read-only; Oops, there shouldn't be "read-only" here, I forgot to refresh the patch as I'm now seeing... too bad it's too late to resend now. :-( WBR, Sergei