On Wed, May 26, 2021 at 5:39 PM Marek Behún <marek.be...@nic.cz> wrote: > > Hello, > > this is v4 of patchset that adds support for U-Boot to parse MTD > partitions from device-tree, and also improves support for SPI NOR > access via the `mtd` command. > > Small rebase was needed since last version. > > Finally passing CI since LTO is now merged and can optimize away the > code increase. CI at https://github.com/u-boot/u-boot/pull/55 > > Changes since v3: > - rebased against current master (removed first patch, not needed > anymore) > - check for CONFIG_OF_CONTROL in addition to CONFIG_DM, since we are > also using ofnode_* functions > - match mtd's name in a separate function to make code more readable. > Also add non-DM version of this name matching function, since #if > macro must be used (otherwise CI will fail for configurations with > disabled DM) > - addressed Simon's comments about using IS_ENABLED instead of #ifdefs > - added Miquel's Reviewed-by and Patrice's Tested-by to the whole series > > Changes since v2: > - addressed Pali's comments in patch that adds partition parsing (4/7 in > this version): no check for whether the `compatible` property is > present in a partition node and added comment explaining mask flags) > - added 4 more patches: > 1) adding ofnode_get_path() function > 2) printing OF path in `mtd list` > 3) in `mtd read <name> ...`, <name> can now also be DM's device name > or OF path > 4) the fact from 3) is added to `mtd help` > > Changes since v1: > - added tests of ofnode_get_addr_size_index() and > ofnode_get_addr_size_index_notrans() as requested by Simon > - the last patch now probes SPI NORs in both versions of > mtd_probe_devices(), that is when MTDPARTS is enabled or disabled > > Marek > > Cc: Jagan Teki <ja...@amarulasolutions.com> > Cc: Priyanka Jain <priyanka.j...@nxp.com> > Cc: Simon Glass <s...@chromium.org> > Cc: Heiko Schocher <h...@denx.de> > Cc: Jagan Teki <ja...@amarulasolutions.com> > Cc: Patrick Delaunay <patrick.delau...@st.com> > Cc: Patrice CHOTARD <patrice.chot...@foss.st.com> > Cc: Miquel Raynal <miquel.ray...@bootlin.com> > > Marek Behún (10): > dm: core: add non-translating version of ofnode_get_addr_size_index() > dm: core: add ofnode_get_path() > mtd: add support for parsing partitions defined in OF > mtd: spi-nor: allow registering multiple MTDs when DM is enabled > mtd: spi-nor: fill-in mtd->dev member > mtd: remove mtd_probe() function > mtd: probe SPI NOR devices in mtd_probe_devices() > cmd: mtd: print device OF path in listing > mtd: compare also with OF path and device name in get_mtd_device_nm() > cmd: mtd: expand <name> argument definition in command help > > cmd/mtd.c | 9 ++- > drivers/core/ofnode.c | 44 ++++++++++- > drivers/mtd/mtd-uclass.c | 15 ---- > drivers/mtd/mtd_uboot.c | 129 ++++++++++++++++++++------------- > drivers/mtd/mtdcore.c | 35 +++++++++ > drivers/mtd/mtdpart.c | 63 ++++++++++++++++ > drivers/mtd/spi/sf_internal.h | 4 +- > drivers/mtd/spi/sf_mtd.c | 19 ++++- > drivers/mtd/spi/sf_probe.c | 6 +- > drivers/mtd/spi/spi-nor-core.c | 1 + > drivers/mtd/spi/spi-nor-tiny.c | 1 + > include/dm/ofnode.h | 27 +++++++ > include/linux/mtd/mtd.h | 10 +++ > include/mtd.h | 1 - > test/dm/ofnode.c | 26 +++++++ > 15 files changed, 315 insertions(+), 75 deletions(-)
This series have some conflicts wrt my series about MTD UCLASS migration. Does this bypass that series? Jagan.