RE: [PATCH 4/4] powerpc/85xx: add the P1020UTM-PC DTS support

Thanks
Jerry Huang

-----Original Message-----
From: Wood Scott-B07421
Sent: Thursday, March 15, 2012 2:50 AM
To: Huang Changming-R66093
Cc: linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH 4/4] powerpc/85xx: add the P1020UTM-PC DTS support
=20
On 03/14/2012 04:08 AM, Chang-Ming.Huang@freescale.com wrote:

quoted

+&lbc {
+	nor@0,0 {
+		#address-cells =3D <1>;
+		#size-cells =3D <1>;
+		compatible =3D "cfi-flash";
+		reg =3D <0x0 0x0 0x2000000>;
+		bank-width =3D <2>;
+		device-width =3D <1>;
+
+		partition@0 {
+			/* 256KB for DTB Image */
+			reg =3D <0x0 0x00040000>;
+			label =3D "NOR (RO) DTB Image";
+			read-only;
+		};
+
+		partition@40000 {
+			/* 3.75 MB for Linux Kernel Image */
+			reg =3D <0x00040000 0x003c0000>;
+			label =3D "NOR (RO) Linux Kernel Image";
+			read-only;
+		};
+
+		partition@400000 {
+			/* 27MB for Root file System */
+			reg =3D <0x00400000 0x01b00000>;
+			label =3D "NOR (RW) Root File System";
+		};
+
+		partition@1f00000 {
+			/* This location must not be altered  */
+			/* 512KB for u-boot Bootloader Image */
+			/* 512KB for u-boot Environment Variables */
+			reg =3D <0x01f00000 0x00100000>;
+			label =3D "NOR (RO) U-Boot Image";
+			read-only;
+		};

=20
Don't put (RW) or (RO) in the label; that information is already there in
the read-only property.

But for the user, after booting the kernel, he don't know these partition a=
re read-only or can be wrote.=20
If add (RW) or (RO), user can know which partition can be wrote or can't di=
rectly.

The DTB and kernel image should not be read-only.