Skip to content

mtd: Fix partition handling code

Miquel Raynal u-boot 1 version New

Get series with b4
b4 am 20181031114318.2cb4e4f3@xps13

Versions newest first

  1. v1 Latest0 of 3 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 3 patches the cover letter announces are in this archive.

Reviews trailers per patch

PatchMiquel Raynal
SeriesR-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [U-Boot] [PATCH 0/3] mtd: Fix partition handling code

Hi Boris,

Boris Brezillon <[email protected]> wrote on Wed, 31 Oct 2018 11:07:33 +0100:

Quoted 12 lines

Hello,

Stefan recently reported a bug when using ubi part on a spi-nand device and proposed this fix[1]. While his solution while working I proposed a different approach to kill the CMD_UBI dependency on CMD_MTDPARTS. This is what patches 1 and 2 are doing.

Patch 3 is just removing the duplicate Kconfig MTD_PARTITIONS entry.

Jagan, can you queue those patches to one of your branch and send a fixes PR, unless Tom prefers to take them directly.

Quoted 8 lines

Regards,

Boris

Boris Brezillon (3):
mtd: Use default mtdparts/mtids when not defined in the environment
cmd: ubi: Remove useless call to mtdparts_init()
mtd: Drop duplicate MTD_PARTITIONS Kconfig option

Strange, we already had a discussion about it. But indeed this patch is needed.

Quoted 7 lines
 cmd/Kconfig             |  1 -
 cmd/ubi.c               |  5 ----
 drivers/mtd/Kconfig     |  6 ----
 drivers/mtd/mtd_uboot.c | 62 +++++++++++++++++++++++++++++++++++++++--
 4 files changed, 60 insertions(+), 14 deletions(-)
Reviewed-by Miquel Raynal <[email protected]>

Thanks,
Miquèl

Read the cover letter in its thread