On 10/26/2020 6:13 AM, Vivek Unune wrote:
This router has dual paritions to store trx firmware image and
dual partitions for nvram. The second one in each of these cases acts
as a backup store.

When tested with OpenWrt, the default partition parser causes two issues:

1. It labels both nvram partitions as nvram. In factory, second one is
labeled devinfo.
2. It parses second trx image and tries to create second 'linux' partition
and fails with - cannot create duplicate 'linux' partition. I've set this
partition to read-only for now

The following patch works around both of these issues.

Signed-off-by: Vivek Unune <[email protected]>
---
  .../boot/dts/bcm47094-linksys-panamera.dts    | 41 +++++++++++++++++++
  1 file changed, 41 insertions(+)

diff --git a/arch/arm/boot/dts/bcm47094-linksys-panamera.dts 
b/arch/arm/boot/dts/bcm47094-linksys-panamera.dts
index 5d5930edfb9d..13da16c5de68 100644
--- a/arch/arm/boot/dts/bcm47094-linksys-panamera.dts
+++ b/arch/arm/boot/dts/bcm47094-linksys-panamera.dts
@@ -292,3 +292,44 @@ fixed-link {
  &usb3_phy {
        status = "okay";
  };
+
+&nandcs {
+       partitions {
+               compatible = "fixed-partitions";
+               #address-cells = <1>;
+               #size-cells = <1>;
+
+               partition@0 {
+                       label = "boot";
+                       reg = <0x0000000 0x0080000>;
+                       read-only;
+               };
+
+               partition@80000 {
+                       label = "nvram";
+                       reg = <0x080000 0x0100000>;
+               };
+
+               partition@180000{
+                       label = "devinfo";
+                       reg = <0x0180000 0x080000>;
+               };
+
+               partition@200000 {
+                       label = "firmware";
+                       reg = <0x0200000 0x01D00000>;
+                       compatible = "brcm,trx";
+               };
+
+               partition@1F00000 {
+                       label = "failsafe";
+                       reg = <0x01F00000 0x01D00000>;
+                       read-only;
+               };
+
+               partition@0x5200000 {

You would need to remove the 0x from the hex number here.

+                       label = "brcmnand";

Not sure how useful naming this partition brcmnand is, can we find a better name for it?
--
Florian

Reply via email to