mtd: read partitions compatible prop for "ofpart" type
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
- v10 Latest3 patches 2 repliesApplied
Patches in v10 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | mtd: partitions: add of_match_table parser matching for the "ofpart" typedrivers/ include/ | +108 | 0 replies | |
| 2/3 | mtd: rename "ofpart" parser to "fixed-partitions" as it fits it betterdrivers/ drivers/ | +8 | 0 replies | |
| 3/3 | mtd: ofpart: add of_match_table with "fixed-partitions"drivers/ | +7 | 0 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.
| Patch | Peter Rosin | Brian Norris | Richard Weinberger |
|---|---|---|---|
| Series | R-b | ||
| 1/3 | T-b | ||
| 2/3 | |||
| 3/3 | R-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.
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/