ARM: dts: da850-lcdk: increase the size of the u-boot nand partition

PATCH In review Bartosz Golaszewski 2 participants 2 messages May 4, 2018 Device tree bindings

lore

Share & tools

lore

Bartosz Golaszewski

author

2018-05-04 08:08 UTC #0 lore raw

PATCH 1/1 ARM: dts: da850-lcdk: increase the size of the u-boot nand partition

From: Bartosz Golaszewski <[email protected]>

Current mainline u-boot builds images that are bigger than the 0x80000 bytes u-boot partition allocated from the NAND memory. Using the free space partition results in an unbootable system. Increase the partition size to 0xa0000 bytes.

Signed-off-by Bartosz Golaszewski <[email protected]>

---
 arch/arm/boot/dts/da850-lcdk.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

arch/arm/boot/dts/da850-lcdk.dts +2 -2

index 7de302f890d7..fc8667445892 100644
@@ -336,11 +336,11 @@
 				partition at 20000 {
 					/* The LCDK defaults to booting from this partition */
 					label = "u-boot";
-					reg = <0x020000 0x080000>;
+					reg = <0x020000 0x0a0000>;
 				};
 				partition at a0000 {
 					label = "free space";
-					reg = <0x0a0000 0>;
+					reg = <0x0c0000 0>;
 				};
 			};
 		};

Signature · 1 line

2.17.0

Sekhar Nori

2018-05-04 08:23 UTC #1 lore raw

Hi Bartosz,

Quoted 17 lines · Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>
Current mainline u-boot builds images that are bigger than the 0x80000 bytes u-boot partition allocated from the NAND memory. Using the free space partition results in an unbootable system. Increase the partition size to 0xa0000 bytes.
Signed-off-by Bartosz Golaszewski <[email protected]>
---
 arch/arm/boot/dts/da850-lcdk.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
arch/arm/boot/dts/da850-lcdk.dts +0 -0
index 7de302f890d7..fc8667445892 100644

arch/arm/boot/dts/da850-lcdk.dts line 343

Quoted 10 lines

arch/arm/boot/dts/da850-lcdk.dts +2 -2

@@ -336,11 +336,11 @@
 				partition at 20000 {
 					/* The LCDK defaults to booting from this partition */
 					label = "u-boot";
-					reg = <0x020000 0x080000>;
+					reg = <0x020000 0x0a0000>;
 				};
 				partition at a0000 {
 					label = "free space";
-					reg = <0x0a0000 0>;
+					reg = <0x0c0000 0>;

Unfortunately, we cannot do this because it introduces incompatibility (data loss) for users who may have some data stored in "free space".

I should have insisted on a lot of padding for the u-boot partition or asked not to add partitions to DT at all. For now the only solution is to use command line partitions. They take precedence over DT partitions.

Thanks,
Sekhar