mbox series

[v2,0/2] dt-bindings: mtd: partitions: Add binding for Sercomm parser

Message ID 20220406195557.1956-1-csharper2005@gmail.com
Headers show
Series dt-bindings: mtd: partitions: Add binding for Sercomm parser | expand

Message

Mikhail Zhilkin April 6, 2022, 7:55 p.m. UTC 
MTD partition parser for the Sercomm partition table that is used in some
Beeline, Netgear and Sercomm routers.

The Sercomm partition map table contains real partition offsets, which
may differ from device to device depending on the number and location of
bad blocks on NAND.

Changes since:
v1:
 - Add dt-binding in a separate patch
 - Remove redundant braces and logical NOT operator
 - Define pr_fmt
 - Replace kcalloc by kzalloc
 - Use of_get_child_count() and alloc big enough array before the
   for_each_child_of_node()

Mikhail Zhilkin (2):
  dt-bindings: mtd: partitions: Add binding for Sercomm parser
  mtd: parsers: add support for Sercomm partitions

 .../devicetree/bindings/mtd/partitions/sercomm,sc-partitions.yaml          |  70 ++++++++++
 drivers/mtd/parsers/Kconfig                                                |   9 ++
 drivers/mtd/parsers/Makefile                                               |   1 +
 drivers/mtd/parsers/scpart.c                                               | 240 ++++++++++++++++++++++++++++++++++
 4 files changed, 320 insertions(+)

Comments

Mikhail Zhilkin April 9, 2022, 12:35 p.m. UTC | #1 
Hello Rob,

On 4/7/2022 4:50 PM, Rob Herring wrote:
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure 'yamllint' is installed and dt-schema is up to
> date:
>
> pip3 install dtschema --upgrade
>
> Please check and re-submit.

Thanks for your great explanation how to test! I found and fixed a mistake. 
How I have only one WARNING: 
"added, moved or deleted file(s), does MAINTAINERS need updating?"

I hope it doesn't require additional change. What do you think?


Best regards,
Mikhail