diff options
| author | INAGAKI Hiroshi | 2025-11-08 05:35:30 +0000 |
|---|---|---|
| committer | Hauke Mehrtens | 2025-11-08 20:36:29 +0000 |
| commit | 5aab294726f35d8515945484cbee951655cdfe90 (patch) | |
| tree | 151103bb226d185eff7c7fcccf0d81dd5e35a7d2 | |
| parent | 0334564ea663174823547d751312843c52b33696 (diff) | |
| download | openwrt-5aab294726f35d8515945484cbee951655cdfe90.tar.gz | |
kernel: generic: mtdsplit_seil: return 0 instead of -ENODEV
Return 0 if the current mtd is inactive or no valid header/rootfs found,
instead of -ENODEV.
Linux Kernel 6.7 and later versions handle all errors returned by mtd
parsers, including -ENODEV as error. So '0' needs to be returned if no
child partitions were not parsed.
Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
Link: https://github.com/openwrt/openwrt/pull/20697
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
| -rw-r--r-- | target/linux/generic/files/drivers/mtd/mtdsplit/mtdsplit_seil.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/target/linux/generic/files/drivers/mtd/mtdsplit/mtdsplit_seil.c b/target/linux/generic/files/drivers/mtd/mtdsplit/mtdsplit_seil.c index e58bb49b23..97bee9d268 100644 --- a/target/linux/generic/files/drivers/mtd/mtdsplit/mtdsplit_seil.c +++ b/target/linux/generic/files/drivers/mtd/mtdsplit/mtdsplit_seil.c @@ -112,7 +112,7 @@ static int mtdsplit_parse_seil_fw(struct mtd_info *master, u64 id; if (!seil_bootdev_is_active(np)) - return -ENODEV; + return 0; ret = of_property_read_u64(np, "iij,seil-id", &id); if (ret) { @@ -137,7 +137,7 @@ static int mtdsplit_parse_seil_fw(struct mtd_info *master, if (be64_to_cpu(header.id) != id || be32_to_cpu(header.vfmt) != SEIL_VFMT) { pr_debug("no valid seil image found in \"%s\"\n", master->name); - ret = -ENODEV; + ret = 0; goto err_free_parts; } @@ -154,7 +154,7 @@ static int mtdsplit_parse_seil_fw(struct mtd_info *master, if (ret || (master->size - rootfs_offset) == 0) { pr_debug("no rootfs after seil image in \"%s\"\n", master->name); - ret = -ENODEV; + ret = 0; goto err_free_parts; } |