Skip to content

mtd: read partitions compatible prop for "ofpart" type

Rafał Miłecki linux-devicetree linux-mtd 1 version Applied

Status from this message: Applied, finally. I'm still convinced putting this information in the

Get series with b4
b4 am [email protected]

Versions newest first

  1. v10 Latest3 patches 2 repliesApplied

Patches in v10 Applied

#Subject and filesSizeRepliesApplied
1/3mtd: partitions: add of_match_table parser matching for the "ofpart" typedrivers/mtd/mtdpart.c include/linux/mtd/partitions.h+108 -90 replies
2/3mtd: rename "ofpart" parser to "fixed-partitions" as it fits it betterdrivers/mtd/mtdpart.c drivers/mtd/ofpart.c+8 -70 replies
3/3mtd: ofpart: add of_match_table with "fixed-partitions"drivers/mtd/ofpart.c+7 -00 replies

Reviews trailers per patch

1 of 3 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchPeter RosinBrian NorrisRichard Weinberger
SeriesR-b
1/3T-b
2/3
3/3R-b T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v10 thread

Cover letter [PATCH V10 0/3] mtd: read partitions compatible prop for "ofpart" type

From: Rafał Miłecki <[email protected]>

This patchset provides a proper support for flash device DT node with "partitions" subnode using "compatible" property. It's already documented in the: Documentation/devicetree/bindings/mtd/partition.txt

We believed that version 7 was ready to go, but soon after landing in the linux-next we got a regression report from Peter. Later versions takes a safe path by:
1) Respecting parsers order as specified in the default/driver-provided list.
2) Looking at "compatible" property only when "ofpart" type gets speciied.

Version 9 was successfully tested by Peter and version 10 just squashes two commits into a one.

I double-checked the code and cannot think of any regression this could cause. I also hope this design (roughly discussed with Boris) can be acceptable for the mtd subsystem.

The most important patch from this patchset (1/3) was reviewed by Rob in the
[PATCH V6 1/2] mtd: partitions: add of_match_table parser matching
https://patchwork.ozlabs.org/comment/1831551/

If possible this is a 4.17 material.

Rafał Miłecki (3):
mtd: partitions: add of_match_table parser matching for the "ofpart"
type
mtd: rename "ofpart" parser to "fixed-partitions" as it fits it better
mtd: ofpart: add of_match_table with "fixed-partitions"

 drivers/mtd/mtdpart.c          | 116 +++++++++++++++++++++++++++++++++++++----
 drivers/mtd/ofpart.c           |  18 +++++--
 include/linux/mtd/partitions.h |   1 +
 3 files changed, 121 insertions(+), 14 deletions(-)
Signature · 6 lines
2.11.0


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/