diff mbox

arm64: dts: marvell: Enable Armada-8040-DB CPS SPI1

Message ID 20170921073643.5677-1-miquel.raynal@free-electrons.com (mailing list archive)
State New, archived
Headers show

Commit Message

Miquel Raynal Sept. 21, 2017, 7:36 a.m. UTC 
From: Christine Gharzuzi <chrisg@marvell.com>

Add the DT node enabling Armada-8040-DB CPS SPI controller driver.

Add the SPI NAND flash device connected on the bus. Fill the MTD
partitions layout.

Signed-off-by: Christine Gharzuzi <chrisg@marvell.com>
Signed-off-by: Miquel Raynal <miquel.raynal@free-electrons.com>
---
 arch/arm64/boot/dts/marvell/armada-8040-db.dts | 31 ++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

Comments

Antoine Tenart Sept. 21, 2017, 7:52 a.m. UTC | #1 
Hi Miquel,

On Thu, Sep 21, 2017 at 09:36:43AM +0200, Miquel Raynal wrote:
> From: Christine Gharzuzi <chrisg@marvell.com>
> +&cps_spi1 {
> +	status = "okay";
> +
> +	spi-flash@0 {
> +		#address-cells = <0x1>;
> +		#size-cells = <0x1>;
> +		compatible = "jedec,spi-nor";
> +		reg = <0x0>;
> +		spi-max-frequency = <20000000>;
> +
> +		partitions {
> +			compatible = "fixed-partitions";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +
> +			partition@0 {
> +				label = "Boot";
> +				reg = <0x0 0x200000>;
> +			};
> +			partition@200000 {
> +				label = "Filesystem";
> +				reg = <0x200000 0xd00000>;
> +			};
> +			partition@d00000 {

The node should be 'partition@f00000'.

> +				label = "Boot_2nd";
> +				reg = <0xf00000 0x100000>;
> +			};
> +		};
> +	};
> +};

Thanks,
Antoine
Miquel Raynal Sept. 22, 2017, 12:47 p.m. UTC | #2 
Hello Antoine,

On Thu, 21 Sep 2017 09:52:39 +0200
Antoine Tenart <antoine.tenart@free-electrons.com> wrote:

> Hi Miquel,
> 
> On Thu, Sep 21, 2017 at 09:36:43AM +0200, Miquel Raynal wrote:
> > From: Christine Gharzuzi <chrisg@marvell.com>
> > +&cps_spi1 {
> > +	status = "okay";
> > +
> > +	spi-flash@0 {
> > +		#address-cells = <0x1>;
> > +		#size-cells = <0x1>;
> > +		compatible = "jedec,spi-nor";
> > +		reg = <0x0>;
> > +		spi-max-frequency = <20000000>;
> > +
> > +		partitions {
> > +			compatible = "fixed-partitions";
> > +			#address-cells = <1>;
> > +			#size-cells = <1>;
> > +
> > +			partition@0 {
> > +				label = "Boot";
> > +				reg = <0x0 0x200000>;
> > +			};
> > +			partition@200000 {
> > +				label = "Filesystem";
> > +				reg = <0x200000 0xd00000>;
> > +			};
> > +			partition@d00000 {  
> 
> The node should be 'partition@f00000'.

Absolutely, thanks for poiting it!

Regards,
Miquèl

> 
> > +				label = "Boot_2nd";
> > +				reg = <0xf00000 0x100000>;
> > +			};
> > +		};
> > +	};
> > +};  
> 
> Thanks,
> Antoine
>
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/marvell/armada-8040-db.dts b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
index 0d7b2ae46610..f5be6ffe0106 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-db.dts
@@ -216,6 +216,37 @@ 
 	clock-frequency = <100000>;
 };
 
+&cps_spi1 {
+	status = "okay";
+
+	spi-flash@0 {
+		#address-cells = <0x1>;
+		#size-cells = <0x1>;
+		compatible = "jedec,spi-nor";
+		reg = <0x0>;
+		spi-max-frequency = <20000000>;
+
+		partitions {
+			compatible = "fixed-partitions";
+			#address-cells = <1>;
+			#size-cells = <1>;
+
+			partition@0 {
+				label = "Boot";
+				reg = <0x0 0x200000>;
+			};
+			partition@200000 {
+				label = "Filesystem";
+				reg = <0x200000 0xd00000>;
+			};
+			partition@d00000 {
+				label = "Boot_2nd";
+				reg = <0xf00000 0x100000>;
+			};
+		};
+	};
+};
+
 /* CON4 on CP1 expansion */
 &cps_sata0 {
 	status = "okay";