[PATCH] mtd: parsers: trx: parse "firmware" MTD partitions only

Rafał Miłecki zajec5 at gmail.com
Tue Apr 13 17:37:00 BST 2021


From: Rafał Miłecki <rafal at milecki.pl>

Parsing every partition with "compatible" set to "brcm,trx" results in
parsing both: firmware partition and failsafe partition on devices that
implement failsafe booting. This affects e.g. Linksys EA9500 which has:

partition at 200000 {
	reg = <0x0200000 0x01d00000>;
	compatible = "linksys,ns-firmware", "brcm,trx";
};

partition at 1f00000 {
	reg = <0x01f00000 0x01d00000>;
	compatible = "linksys,ns-firmware", "brcm,trx";
};

Check for MTD partition name "firmware" before parsing. Recently added
ofpart_linksys_ns.c creates "firmware" and "failsafe" depending on
bootloader setup.

Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
---
Vivek has recently reported this problem to me and this is soltuion I
came up with.

One alternative I thought of could be marking "failsafe" MTD partition
node as disabled (using of_update_property() + "status" + "disabled") and
then using of_device_is_available() in the parser_trx.c.

Let me know if you prefer the other (or any other) solution over this
patch.
---
 drivers/mtd/parsers/parser_trx.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/mtd/parsers/parser_trx.c b/drivers/mtd/parsers/parser_trx.c
index 8541182134d4..0063791e164d 100644
--- a/drivers/mtd/parsers/parser_trx.c
+++ b/drivers/mtd/parsers/parser_trx.c
@@ -58,6 +58,10 @@ static int parser_trx_parse(struct mtd_info *mtd,
 	uint8_t curr_part = 0, i = 0;
 	int err;
 
+	/* Don't parse any failsafe / backup partitions */
+	if (strcmp(mtd->name, "firmware"))
+		return -EINVAL;
+
 	parts = kcalloc(TRX_PARSER_MAX_PARTS, sizeof(struct mtd_partition),
 			GFP_KERNEL);
 	if (!parts)
-- 
2.26.2




More information about the linux-mtd mailing list