ar71xx: Fix header offset for newer WRT160NL models
authorGabor Juhos <juhosg@openwrt.org>
Tue, 31 May 2011 22:53:20 +0000 (22:53 +0000)
committerGabor Juhos <juhosg@openwrt.org>
Tue, 31 May 2011 22:53:20 +0000 (22:53 +0000)
Newer WRT160NLs have a flash chip with 4K erase blocks instead of 64K,
resulting in miscalculated partition sizes.
Since the actual sizes did not change, hardcode them to their current
sizes, and make sure they are at least one erase block big (in case Cisco
decides to start to use chips with 128K erase blocks).

Signed-off-by: Jonas Gorski <jonas.gorski+openwrt@gmail.com>
SVN-Revision: 27049

target/linux/ar71xx/files/drivers/mtd/wrt160nl_part.c

index c8233ce59a6d8f964977f469d8540987345f1768..eabb47e02982895f2d08a8e7c671b58375183ff8 100644 (file)
@@ -78,6 +78,10 @@ struct wrt160nl_header {
 
 static struct mtd_partition trx_parts[TRX_PARTS];
 
 
 static struct mtd_partition trx_parts[TRX_PARTS];
 
+#define WRT160NL_UBOOT_LEN     0x40000
+#define WRT160NL_ART_LEN       0x10000
+#define WRT160NL_NVRAM_LEN     0x10000
+
 static int wrt160nl_parse_partitions(struct mtd_info *master,
                                     struct mtd_partition **pparts,
                                     unsigned long origin)
 static int wrt160nl_parse_partitions(struct mtd_info *master,
                                     struct mtd_partition **pparts,
                                     unsigned long origin)
@@ -87,6 +91,9 @@ static int wrt160nl_parse_partitions(struct mtd_info *master,
        struct uimage_header *uheader;
        size_t retlen;
        unsigned int kernel_len;
        struct uimage_header *uheader;
        size_t retlen;
        unsigned int kernel_len;
+       unsigned int uboot_len = max(master->erasesize, WRT160NL_UBOOT_LEN);
+       unsigned int nvram_len = max(master->erasesize, WRT160NL_NVRAM_LEN);
+       unsigned int art_len = max(master->erasesize, WRT160NL_ART_LEN);
        int ret;
 
        header = vmalloc(sizeof(*header));
        int ret;
 
        header = vmalloc(sizeof(*header));
@@ -95,7 +102,7 @@ static int wrt160nl_parse_partitions(struct mtd_info *master,
                goto out;
        }
 
                goto out;
        }
 
-       ret = master->read(master, 4 * master->erasesize, sizeof(*header),
+       ret = master->read(master, uboot_len, sizeof(*header),
                           &retlen, (void *) header);
        if (ret)
                goto free_hdr;
                           &retlen, (void *) header);
        if (ret)
                goto free_hdr;
@@ -128,7 +135,7 @@ static int wrt160nl_parse_partitions(struct mtd_info *master,
 
        trx_parts[0].name = "u-boot";
        trx_parts[0].offset = 0;
 
        trx_parts[0].name = "u-boot";
        trx_parts[0].offset = 0;
-       trx_parts[0].size = 4 * master->erasesize;
+       trx_parts[0].size = uboot_len;
        trx_parts[0].mask_flags = MTD_WRITEABLE;
 
        trx_parts[1].name = "kernel";
        trx_parts[0].mask_flags = MTD_WRITEABLE;
 
        trx_parts[1].name = "kernel";
@@ -138,23 +145,23 @@ static int wrt160nl_parse_partitions(struct mtd_info *master,
 
        trx_parts[2].name = "rootfs";
        trx_parts[2].offset = trx_parts[1].offset + trx_parts[1].size;
 
        trx_parts[2].name = "rootfs";
        trx_parts[2].offset = trx_parts[1].offset + trx_parts[1].size;
-       trx_parts[2].size = master->size - 6 * master->erasesize -
+       trx_parts[2].size = master->size - uboot_len - nvram_len - art_len -
                trx_parts[1].size;
        trx_parts[2].mask_flags = 0;
 
        trx_parts[3].name = "nvram";
                trx_parts[1].size;
        trx_parts[2].mask_flags = 0;
 
        trx_parts[3].name = "nvram";
-       trx_parts[3].offset = master->size - 2 * master->erasesize;
-       trx_parts[3].size = master->erasesize;
+       trx_parts[3].offset = master->size - nvram_len - art_len;
+       trx_parts[3].size = nvram_len;
        trx_parts[3].mask_flags = MTD_WRITEABLE;
 
        trx_parts[4].name = "art";
        trx_parts[3].mask_flags = MTD_WRITEABLE;
 
        trx_parts[4].name = "art";
-       trx_parts[4].offset = master->size - master->erasesize;
-       trx_parts[4].size = master->erasesize;
+       trx_parts[4].offset = master->size - art_len;
+       trx_parts[4].size = art_len;
        trx_parts[4].mask_flags = MTD_WRITEABLE;
 
        trx_parts[5].name = "firmware";
        trx_parts[4].mask_flags = MTD_WRITEABLE;
 
        trx_parts[5].name = "firmware";
-       trx_parts[5].offset = 4 * master->erasesize;
-       trx_parts[5].size = master->size - 6 * master->erasesize;
+       trx_parts[5].offset = uboot_len;
+       trx_parts[5].size = master->size - uboot_len - nvram_len - art_len;
        trx_parts[5].mask_flags = 0;
 
        *pparts = trx_parts;
        trx_parts[5].mask_flags = 0;
 
        *pparts = trx_parts;