* [PATCH v3 0/3] mtd: parsers: add support for Sercomm partitions @ 2022-05-03 15:42 ` Mikhail Zhilkin 0 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:42 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: devicetree, linux-kernel, linux-mtd, Karim, M, Mikhail Zhilkin This patch series add support for Sercomm mtd partition table parser. It's 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: v2: - Fix mistakes in dt-binding - Add patch for new vendor prefix - Add vendor prefix to scpart-id property 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 (3): dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix dt-bindings: mtd: partitions: Extend fixed-partitions binding mtd: parsers: add support for Sercomm partitions .../mtd/partitions/fixed-partitions.yaml | 55 +++- .../devicetree/bindings/vendor-prefixes.yaml | 2 + drivers/mtd/parsers/Kconfig | 9 + drivers/mtd/parsers/Makefile | 1 + drivers/mtd/parsers/scpart.c | 240 ++++++++++++++++++ 5 files changed, 305 insertions(+), 2 deletions(-) create mode 100644 drivers/mtd/parsers/scpart.c -- 2.25.1 ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v3 0/3] mtd: parsers: add support for Sercomm partitions @ 2022-05-03 15:42 ` Mikhail Zhilkin 0 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:42 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: devicetree, linux-kernel, linux-mtd, Karim, M, Mikhail Zhilkin This patch series add support for Sercomm mtd partition table parser. It's 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: v2: - Fix mistakes in dt-binding - Add patch for new vendor prefix - Add vendor prefix to scpart-id property 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 (3): dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix dt-bindings: mtd: partitions: Extend fixed-partitions binding mtd: parsers: add support for Sercomm partitions .../mtd/partitions/fixed-partitions.yaml | 55 +++- .../devicetree/bindings/vendor-prefixes.yaml | 2 + drivers/mtd/parsers/Kconfig | 9 + drivers/mtd/parsers/Makefile | 1 + drivers/mtd/parsers/scpart.c | 240 ++++++++++++++++++ 5 files changed, 305 insertions(+), 2 deletions(-) create mode 100644 drivers/mtd/parsers/scpart.c -- 2.25.1 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v3 1/3] dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix 2022-05-03 15:42 ` Mikhail Zhilkin (?) @ 2022-05-03 15:47 ` Mikhail Zhilkin 2022-05-04 15:07 ` Krzysztof Kozlowski -1 siblings, 1 reply; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:47 UTC (permalink / raw) To: Rob Herring, Krzysztof Kozlowski Cc: devicetree, linux-kernel, Karim, M, Mikhail Zhilkin Update Documentation/devicetree/bindings/vendor-prefixes.yaml to include "sercomm" as a vendor prefix for "Sercomm (Suzhou) Corporation". Company website: Link: https://www.sercomm.com/ Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 01430973ecec..65ff22364fb3 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -1082,6 +1082,8 @@ patternProperties: description: Sensirion AG "^sensortek,.*": description: Sensortek Technology Corporation + "^sercomm,.*": + description: Sercomm (Suzhou) Corporation "^sff,.*": description: Small Form Factor Committee "^sgd,.*": -- 2.25.1 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v3 1/3] dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix 2022-05-03 15:47 ` [PATCH v3 1/3] dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix Mikhail Zhilkin @ 2022-05-04 15:07 ` Krzysztof Kozlowski 0 siblings, 0 replies; 22+ messages in thread From: Krzysztof Kozlowski @ 2022-05-04 15:07 UTC (permalink / raw) To: Mikhail Zhilkin, Rob Herring, Krzysztof Kozlowski Cc: devicetree, linux-kernel, Karim, M On 03/05/2022 17:47, Mikhail Zhilkin wrote: > Update Documentation/devicetree/bindings/vendor-prefixes.yaml This is visible from the diff, so just "Add sercomm vendor prefix for ...." to include > "sercomm" as a vendor prefix for "Sercomm (Suzhou) Corporation". > Company website: > Link: https://www.sercomm.com/ > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > --- > Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ > 1 file changed, 2 insertions(+) Best regards, Krzysztof ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-03 15:42 ` Mikhail Zhilkin @ 2022-05-03 15:50 ` Mikhail Zhilkin -1 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:50 UTC (permalink / raw) To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M, Mikhail Zhilkin Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible). Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> --- .../mtd/partitions/fixed-partitions.yaml | 55 ++++++++++++++++++- 1 file changed, 53 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml index ea4cace6a955..ad3ccd250802 100644 --- a/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml +++ b/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml @@ -19,7 +19,11 @@ maintainers: properties: compatible: - const: fixed-partitions + oneOf: + - const: fixed-partitions + - items: + - const: sercomm,sc-partitions + - const: fixed-partitions "#address-cells": true @@ -27,7 +31,24 @@ properties: patternProperties: "@[0-9a-f]+$": - $ref: "partition.yaml#" + allOf: + - $ref: "partition.yaml#" + - if: + properties: + compatible: + contains: + const: sercomm,sc-partitions + then: + properties: + sercomm,scpart-id: + description: Partition id in Sercomm partition map. Mtd + parser uses this id to find a record in the partition map + containing offset and size of the current partition. The + values from partition map overrides partition offset and + size defined in reg property of the dts. Frequently these + values are the same, but may differ if device has bad + eraseblocks on a flash. + $ref: /schemas/types.yaml#/definitions/uint32 required: - "#address-cells" @@ -52,6 +73,7 @@ examples: reg = <0x0100000 0x200000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -64,6 +86,7 @@ examples: reg = <0x00000000 0x1 0x00000000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -82,6 +105,7 @@ examples: reg = <0x2 0x00000000 0x1 0x00000000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -119,3 +143,30 @@ examples: }; }; }; + + - | + partitions { + compatible = "sercomm,sc-partitions", "fixed-partitions"; + #address-cells = <1>; + #size-cells = <1>; + + partition@0 { + label = "u-boot"; + reg = <0x0 0x100000>; + sercomm,scpart-id = <0>; + read-only; + }; + + partition@100000 { + label = "dynamic partition map"; + reg = <0x100000 0x100000>; + sercomm,scpart-id = <1>; + }; + + partition@200000 { + label = "Factory"; + reg = <0x200000 0x100000>; + sercomm,scpart-id = <2>; + read-only; + }; + }; -- 2.25.1 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-03 15:50 ` Mikhail Zhilkin 0 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:50 UTC (permalink / raw) To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M, Mikhail Zhilkin Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible). Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> --- .../mtd/partitions/fixed-partitions.yaml | 55 ++++++++++++++++++- 1 file changed, 53 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml index ea4cace6a955..ad3ccd250802 100644 --- a/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml +++ b/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml @@ -19,7 +19,11 @@ maintainers: properties: compatible: - const: fixed-partitions + oneOf: + - const: fixed-partitions + - items: + - const: sercomm,sc-partitions + - const: fixed-partitions "#address-cells": true @@ -27,7 +31,24 @@ properties: patternProperties: "@[0-9a-f]+$": - $ref: "partition.yaml#" + allOf: + - $ref: "partition.yaml#" + - if: + properties: + compatible: + contains: + const: sercomm,sc-partitions + then: + properties: + sercomm,scpart-id: + description: Partition id in Sercomm partition map. Mtd + parser uses this id to find a record in the partition map + containing offset and size of the current partition. The + values from partition map overrides partition offset and + size defined in reg property of the dts. Frequently these + values are the same, but may differ if device has bad + eraseblocks on a flash. + $ref: /schemas/types.yaml#/definitions/uint32 required: - "#address-cells" @@ -52,6 +73,7 @@ examples: reg = <0x0100000 0x200000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -64,6 +86,7 @@ examples: reg = <0x00000000 0x1 0x00000000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -82,6 +105,7 @@ examples: reg = <0x2 0x00000000 0x1 0x00000000>; }; }; + - | partitions { compatible = "fixed-partitions"; @@ -119,3 +143,30 @@ examples: }; }; }; + + - | + partitions { + compatible = "sercomm,sc-partitions", "fixed-partitions"; + #address-cells = <1>; + #size-cells = <1>; + + partition@0 { + label = "u-boot"; + reg = <0x0 0x100000>; + sercomm,scpart-id = <0>; + read-only; + }; + + partition@100000 { + label = "dynamic partition map"; + reg = <0x100000 0x100000>; + sercomm,scpart-id = <1>; + }; + + partition@200000 { + label = "Factory"; + reg = <0x200000 0x100000>; + sercomm,scpart-id = <2>; + read-only; + }; + }; -- 2.25.1 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-03 15:50 ` Mikhail Zhilkin @ 2022-05-04 15:07 ` Krzysztof Kozlowski -1 siblings, 0 replies; 22+ messages in thread From: Krzysztof Kozlowski @ 2022-05-04 15:07 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M On 03/05/2022 17:50, Mikhail Zhilkin wrote: > Extend fixed-partitions binding for support of Sercomm partition parser > (add "sercomm,sc-partitions" compatible). > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-04 15:07 ` Krzysztof Kozlowski 0 siblings, 0 replies; 22+ messages in thread From: Krzysztof Kozlowski @ 2022-05-04 15:07 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M On 03/05/2022 17:50, Mikhail Zhilkin wrote: > Extend fixed-partitions binding for support of Sercomm partition parser > (add "sercomm,sc-partitions" compatible). > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-03 15:50 ` Mikhail Zhilkin @ 2022-05-12 15:27 ` Miquel Raynal -1 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-12 15:27 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > Extend fixed-partitions binding for support of Sercomm partition parser > (add "sercomm,sc-partitions" compatible). > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Miquel ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-12 15:27 ` Miquel Raynal 0 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-12 15:27 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski Cc: Rafał Miłecki, linux-mtd, devicetree, linux-kernel, Karim, M On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > Extend fixed-partitions binding for support of Sercomm partition parser > (add "sercomm,sc-partitions" compatible). > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Miquel ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-12 15:27 ` Miquel Raynal @ 2022-05-13 14:12 ` Rob Herring -1 siblings, 0 replies; 22+ messages in thread From: Rob Herring @ 2022-05-13 14:12 UTC (permalink / raw) To: Miquel Raynal Cc: Mikhail Zhilkin, Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M On Thu, May 12, 2022 at 10:27 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > > Extend fixed-partitions binding for support of Sercomm partition parser > > (add "sercomm,sc-partitions" compatible). > > > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Patch 1 is a dependency, please apply it too. Without it, we now get in linux-next: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: partition@0: 'sercomm,scpart-id' does not match any of the regexes: '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', '^(pinctrl-single|#pinctrl-single|PowerPC),.*', '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', '^zkmagic,.*', '^zte,.*', '^zyxel,.*' From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml Rob ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-13 14:12 ` Rob Herring 0 siblings, 0 replies; 22+ messages in thread From: Rob Herring @ 2022-05-13 14:12 UTC (permalink / raw) To: Miquel Raynal Cc: Mikhail Zhilkin, Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M On Thu, May 12, 2022 at 10:27 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote: > > On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > > Extend fixed-partitions binding for support of Sercomm partition parser > > (add "sercomm,sc-partitions" compatible). > > > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Patch 1 is a dependency, please apply it too. Without it, we now get in linux-next: /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: partition@0: 'sercomm,scpart-id' does not match any of the regexes: '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', '^(pinctrl-single|#pinctrl-single|PowerPC),.*', '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', '^zkmagic,.*', '^zte,.*', '^zyxel,.*' From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml Rob ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-13 14:12 ` Rob Herring @ 2022-05-16 8:21 ` Miquel Raynal -1 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-16 8:21 UTC (permalink / raw) To: Rob Herring Cc: Mikhail Zhilkin, Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M Hi, robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: > On Thu, May 12, 2022 at 10:27 AM Miquel Raynal > <miquel.raynal@bootlin.com> wrote: > > > > On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > > > Extend fixed-partitions binding for support of Sercomm partition parser > > > (add "sercomm,sc-partitions" compatible). > > > > > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > > > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. > > Patch 1 is a dependency, please apply it too. Without it, we now get > in linux-next: Only patch 3 was sent to me/the mtd ML. I'll drop the patch. Mikhail, please resend the series including us on all your patches. > /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: > partition@0: 'sercomm,scpart-id' does not match any of the regexes: > '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', > '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', > '^(pinctrl-single|#pinctrl-single|PowerPC),.*', > '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', > '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', > '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', > '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', > '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', > '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', > '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', > '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', > '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', > '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', > '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', > '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', > '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', > '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', > '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', > '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', > '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', > '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', > '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', > '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', > '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', > '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', > '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', > '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', > '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', > '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', > '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', > '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', > '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', > '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', > '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', > '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', > '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', > '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', > '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', > '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', > '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', > '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', > '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', > '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', > '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', > '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', > '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', > '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', > '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', > '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', > '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', > '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', > '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', > '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', > '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', > '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', > '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', > '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', > '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', > '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', > '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', > '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', > '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', > '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', > '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', > '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', > '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', > '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', > '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', > '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', > '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', > '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', > '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', > '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', > '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', > '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', > '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', > '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', > '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', > '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', > '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', > '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', > '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', > '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', > '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', > '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', > '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', > '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', > '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', > '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', > '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', > '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', > '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', > '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', > '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', > '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', > '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', > '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', > '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', > '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', > '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', > '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', > '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', > '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', > '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', > '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', > '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', > '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', > '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', > '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', > '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', > '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', > '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', > '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', > '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', > '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', > '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', > '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', > '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', > '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', > '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', > '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', > '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', > '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', > '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', > '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', > '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', > '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', > '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', > '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', > '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', > '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', > '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', > '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', > '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', > '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', > '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', > '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', > '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', > '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', > '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', > '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', > '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', > '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', > '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', > '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', > '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', > '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', > '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', > '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', > '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', > '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', > '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', > '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', > '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', > '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', > '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', > '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', > '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', > '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', > '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', > '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', > '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', > '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', > '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', > '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', > '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', > '^zkmagic,.*', '^zte,.*', '^zyxel,.*' > From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml > > Rob Thanks, Miquèl ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-16 8:21 ` Miquel Raynal 0 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-16 8:21 UTC (permalink / raw) To: Rob Herring Cc: Mikhail Zhilkin, Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M Hi, robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: > On Thu, May 12, 2022 at 10:27 AM Miquel Raynal > <miquel.raynal@bootlin.com> wrote: > > > > On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: > > > Extend fixed-partitions binding for support of Sercomm partition parser > > > (add "sercomm,sc-partitions" compatible). > > > > > > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > > > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. > > Patch 1 is a dependency, please apply it too. Without it, we now get > in linux-next: Only patch 3 was sent to me/the mtd ML. I'll drop the patch. Mikhail, please resend the series including us on all your patches. > /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: > partition@0: 'sercomm,scpart-id' does not match any of the regexes: > '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', > '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', > '^(pinctrl-single|#pinctrl-single|PowerPC),.*', > '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', > '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', > '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', > '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', > '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', > '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', > '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', > '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', > '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', > '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', > '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', > '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', > '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', > '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', > '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', > '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', > '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', > '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', > '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', > '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', > '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', > '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', > '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', > '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', > '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', > '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', > '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', > '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', > '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', > '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', > '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', > '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', > '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', > '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', > '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', > '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', > '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', > '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', > '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', > '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', > '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', > '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', > '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', > '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', > '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', > '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', > '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', > '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', > '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', > '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', > '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', > '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', > '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', > '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', > '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', > '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', > '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', > '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', > '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', > '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', > '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', > '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', > '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', > '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', > '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', > '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', > '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', > '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', > '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', > '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', > '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', > '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', > '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', > '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', > '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', > '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', > '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', > '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', > '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', > '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', > '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', > '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', > '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', > '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', > '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', > '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', > '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', > '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', > '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', > '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', > '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', > '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', > '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', > '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', > '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', > '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', > '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', > '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', > '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', > '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', > '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', > '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', > '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', > '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', > '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', > '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', > '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', > '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', > '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', > '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', > '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', > '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', > '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', > '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', > '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', > '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', > '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', > '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', > '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', > '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', > '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', > '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', > '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', > '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', > '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', > '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', > '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', > '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', > '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', > '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', > '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', > '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', > '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', > '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', > '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', > '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', > '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', > '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', > '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', > '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', > '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', > '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', > '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', > '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', > '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', > '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', > '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', > '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', > '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', > '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', > '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', > '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', > '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', > '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', > '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', > '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', > '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', > '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', > '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', > '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', > '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', > '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', > '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', > '^zkmagic,.*', '^zte,.*', '^zyxel,.*' > From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml > > Rob Thanks, Miquèl ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-16 8:21 ` Miquel Raynal @ 2022-05-16 15:34 ` Mikhail Zhilkin -1 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-16 15:34 UTC (permalink / raw) To: Miquel Raynal, Rob Herring Cc: Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M Hi, Miquel! On 5/16/2022 11:21 AM, Miquel Raynal wrote: > Hi, > > robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: > >> On Thu, May 12, 2022 at 10:27 AM Miquel Raynal >> <miquel.raynal@bootlin.com> wrote: >>> On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: >>>> Extend fixed-partitions binding for support of Sercomm partition parser >>>> (add "sercomm,sc-partitions" compatible). >>>> >>>> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> >>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. >> Patch 1 is a dependency, please apply it too. Without it, we now get >> in linux-next: > Only patch 3 was sent to me/the mtd ML. I'll drop the patch. > > Mikhail, please resend the series including us on all your patches. Done! I sent the series to you and mtd https://lore.kernel.org/linux-mtd/20220516151228.885222-1-csharper2005@gmail.com/T/#t P.S. get_maintainer.pl didn't recommend to add you and mtd that and this time. Do I need to send each patch from the series to all maintainers of all the patches in the future? $ scripts/get_maintainer.pl v5-0001-dt-bindings-Add-Sercomm-Suzhou-Corporation-vendor.patch Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,commit_signer:59/65=91%,in file) Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,commit_signer:12/65=18%,added_lines:7/139=5%,removed_lines:1/2=50%) Shawn Guo <shawnguo@kernel.org> (commit_signer:7/65=11%) Stanislav Jakubek <stano.jakubek@gmail.com> (commit_signer:6/65=9%,authored:6/65=9%,added_lines:12/139=9%) Sam Ravnborg <sam@ravnborg.org> (commit_signer:5/65=8%) Linus Walleij <linus.walleij@linaro.org> (added_lines:10/139=7%) Hao Fang <fanghao11@huawei.com> (removed_lines:1/2=50%) devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS) linux-kernel@vger.kernel.org (open list) >> /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: >> partition@0: 'sercomm,scpart-id' does not match any of the regexes: >> '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', >> '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', >> '^(pinctrl-single|#pinctrl-single|PowerPC),.*', >> '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', >> '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', >> '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', >> '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', >> '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', >> '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', >> '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', >> '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', >> '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', >> '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', >> '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', >> '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', >> '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', >> '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', >> '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', >> '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', >> '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', >> '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', >> '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', >> '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', >> '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', >> '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', >> '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', >> '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', >> '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', >> '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', >> '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', >> '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', >> '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', >> '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', >> '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', >> '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', >> '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', >> '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', >> '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', >> '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', >> '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', >> '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', >> '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', >> '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', >> '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', >> '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', >> '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', >> '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', >> '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', >> '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', >> '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', >> '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', >> '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', >> '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', >> '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', >> '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', >> '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', >> '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', >> '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', >> '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', >> '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', >> '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', >> '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', >> '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', >> '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', >> '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', >> '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', >> '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', >> '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', >> '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', >> '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', >> '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', >> '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', >> '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', >> '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', >> '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', >> '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', >> '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', >> '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', >> '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', >> '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', >> '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', >> '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', >> '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', >> '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', >> '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', >> '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', >> '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', >> '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', >> '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', >> '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', >> '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', >> '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', >> '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', >> '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', >> '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', >> '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', >> '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', >> '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', >> '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', >> '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', >> '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', >> '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', >> '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', >> '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', >> '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', >> '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', >> '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', >> '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', >> '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', >> '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', >> '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', >> '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', >> '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', >> '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', >> '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', >> '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', >> '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', >> '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', >> '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', >> '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', >> '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', >> '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', >> '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', >> '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', >> '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', >> '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', >> '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', >> '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', >> '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', >> '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', >> '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', >> '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', >> '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', >> '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', >> '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', >> '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', >> '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', >> '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', >> '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', >> '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', >> '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', >> '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', >> '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', >> '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', >> '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', >> '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', >> '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', >> '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', >> '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', >> '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', >> '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', >> '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', >> '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', >> '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', >> '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', >> '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', >> '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', >> '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', >> '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', >> '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', >> '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', >> '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', >> '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', >> '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', >> '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', >> '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', >> '^zkmagic,.*', '^zte,.*', '^zyxel,.*' >> From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml >> >> Rob > > Thanks, > Miquèl -- Best regards, Mikhail ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-16 15:34 ` Mikhail Zhilkin 0 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-16 15:34 UTC (permalink / raw) To: Miquel Raynal, Rob Herring Cc: Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M Hi, Miquel! On 5/16/2022 11:21 AM, Miquel Raynal wrote: > Hi, > > robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: > >> On Thu, May 12, 2022 at 10:27 AM Miquel Raynal >> <miquel.raynal@bootlin.com> wrote: >>> On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: >>>> Extend fixed-partitions binding for support of Sercomm partition parser >>>> (add "sercomm,sc-partitions" compatible). >>>> >>>> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> >>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. >> Patch 1 is a dependency, please apply it too. Without it, we now get >> in linux-next: > Only patch 3 was sent to me/the mtd ML. I'll drop the patch. > > Mikhail, please resend the series including us on all your patches. Done! I sent the series to you and mtd https://lore.kernel.org/linux-mtd/20220516151228.885222-1-csharper2005@gmail.com/T/#t P.S. get_maintainer.pl didn't recommend to add you and mtd that and this time. Do I need to send each patch from the series to all maintainers of all the patches in the future? $ scripts/get_maintainer.pl v5-0001-dt-bindings-Add-Sercomm-Suzhou-Corporation-vendor.patch Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,commit_signer:59/65=91%,in file) Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,commit_signer:12/65=18%,added_lines:7/139=5%,removed_lines:1/2=50%) Shawn Guo <shawnguo@kernel.org> (commit_signer:7/65=11%) Stanislav Jakubek <stano.jakubek@gmail.com> (commit_signer:6/65=9%,authored:6/65=9%,added_lines:12/139=9%) Sam Ravnborg <sam@ravnborg.org> (commit_signer:5/65=8%) Linus Walleij <linus.walleij@linaro.org> (added_lines:10/139=7%) Hao Fang <fanghao11@huawei.com> (removed_lines:1/2=50%) devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS) linux-kernel@vger.kernel.org (open list) >> /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb: >> partition@0: 'sercomm,scpart-id' does not match any of the regexes: >> '^#.*', '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*', >> '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', >> '^(pinctrl-single|#pinctrl-single|PowerPC),.*', >> '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', >> '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^100ask,.*', >> '^70mai,.*', '^8dev,.*', '^GEFanuc,.*', '^ORCL,.*', '^SUNW,.*', >> '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', >> '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', >> '^abracon,.*', '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', >> '^active-semi,.*', '^ad,.*', '^adafruit,.*', '^adapteva,.*', >> '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*', >> '^aeroflexgaisler,.*', '^aesop,.*', '^airoha,.*', '^al,.*', >> '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*', >> '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*', >> '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', >> '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', >> '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*', >> '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', >> '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*', >> '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*', >> '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*', >> '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', >> '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', >> '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*', >> '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*', >> '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*', >> '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*', >> '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*', >> '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', >> '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*', >> '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*', >> '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*', >> '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', >> '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*', >> '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*', >> '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*', >> '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*', >> '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*', >> '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*', >> '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', >> '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', >> '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*', >> '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*', >> '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*', >> '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*', >> '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*', >> '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*', >> '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', >> '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*', >> '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', >> '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*', >> '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*', >> '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*', >> '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*', >> '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*', >> '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*', >> '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*', >> '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', >> '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', >> '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*', >> '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*', >> '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', >> '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', >> '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*', >> '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*', >> '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', >> '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*', >> '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', >> '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*', >> '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*', >> '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*', >> '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*', >> '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*', >> '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*', >> '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*', >> '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*', >> '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*', >> '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', >> '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', >> '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*', >> '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', >> '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*', >> '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*', >> '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^macnica,.*', >> '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', >> '^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', >> '^mediatek,.*', '^megachips,.*', '^mele,.*', '^melexis,.*', >> '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*', '^mentor,.*', >> '^meraki,.*', '^merrii,.*', '^micrel,.*', '^microchip,.*', >> '^microcrystal,.*', '^micron,.*', '^microsoft,.*', '^microsys,.*', >> '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*', >> '^miramems,.*', '^mitsubishi,.*', '^miyoo,.*', '^mntre,.*', >> '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*', >> '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*', >> '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*', >> '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*', >> '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', >> '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*', >> '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*', >> '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*', >> '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*', >> '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*', >> '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*', >> '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*', >> '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*', >> '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', >> '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*', >> '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*', >> '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', >> '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', >> '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', >> '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*', >> '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*', >> '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', >> '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', >> '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*', >> '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*', >> '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', >> '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*', >> '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*', >> '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', >> '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*', >> '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', >> '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', >> '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*', >> '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*', >> '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*', >> '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', >> '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*', >> '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*', >> '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', >> '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*', >> '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*', >> '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', >> '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*', >> '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*', >> '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*', >> '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*', >> '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*', >> '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*', >> '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', >> '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*', >> '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*', >> '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*', >> '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', >> '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*', >> '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*', >> '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*', >> '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*', >> '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*', >> '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*', >> '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*', >> '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*', >> '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*', >> '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*', >> '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*', >> '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*', >> '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*', >> '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*', >> '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', >> '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*', >> '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*', >> '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*', >> '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*', >> '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*', >> '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*', >> '^zkmagic,.*', '^zte,.*', '^zyxel,.*' >> From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml >> >> Rob > > Thanks, > Miquèl -- Best regards, Mikhail ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding 2022-05-16 15:34 ` Mikhail Zhilkin @ 2022-05-16 15:39 ` Krzysztof Kozlowski -1 siblings, 0 replies; 22+ messages in thread From: Krzysztof Kozlowski @ 2022-05-16 15:39 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Rob Herring Cc: Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M On 16/05/2022 17:34, Mikhail Zhilkin wrote: > Hi, Miquel! > > On 5/16/2022 11:21 AM, Miquel Raynal wrote: >> Hi, >> >> robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: >> >>> On Thu, May 12, 2022 at 10:27 AM Miquel Raynal >>> <miquel.raynal@bootlin.com> wrote: >>>> On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: >>>>> Extend fixed-partitions binding for support of Sercomm partition parser >>>>> (add "sercomm,sc-partitions" compatible). >>>>> >>>>> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> >>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. >>> Patch 1 is a dependency, please apply it too. Without it, we now get >>> in linux-next: >> Only patch 3 was sent to me/the mtd ML. I'll drop the patch. >> >> Mikhail, please resend the series including us on all your patches. > > Done! I sent the series to you and mtd > https://lore.kernel.org/linux-mtd/20220516151228.885222-1-csharper2005@gmail.com/T/#t > > P.S. get_maintainer.pl didn't recommend to add you and mtd that and this > time. Do I need to send each patch from the series to all maintainers of > all the patches in the future? Yes, entire patchset should be sent to the same folks, same maintainers. Otherwise maintainer should assume patches are independent, which is not true. Exception is for large patchsets where we do not want to spam everyone and these require different approaches. It's not the case here. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding @ 2022-05-16 15:39 ` Krzysztof Kozlowski 0 siblings, 0 replies; 22+ messages in thread From: Krzysztof Kozlowski @ 2022-05-16 15:39 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Rob Herring Cc: Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski, Rafał Miłecki, MTD Maling List, devicetree, linux-kernel, Karim, M On 16/05/2022 17:34, Mikhail Zhilkin wrote: > Hi, Miquel! > > On 5/16/2022 11:21 AM, Miquel Raynal wrote: >> Hi, >> >> robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500: >> >>> On Thu, May 12, 2022 at 10:27 AM Miquel Raynal >>> <miquel.raynal@bootlin.com> wrote: >>>> On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote: >>>>> Extend fixed-partitions binding for support of Sercomm partition parser >>>>> (add "sercomm,sc-partitions" compatible). >>>>> >>>>> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> >>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. >>> Patch 1 is a dependency, please apply it too. Without it, we now get >>> in linux-next: >> Only patch 3 was sent to me/the mtd ML. I'll drop the patch. >> >> Mikhail, please resend the series including us on all your patches. > > Done! I sent the series to you and mtd > https://lore.kernel.org/linux-mtd/20220516151228.885222-1-csharper2005@gmail.com/T/#t > > P.S. get_maintainer.pl didn't recommend to add you and mtd that and this > time. Do I need to send each patch from the series to all maintainers of > all the patches in the future? Yes, entire patchset should be sent to the same folks, same maintainers. Otherwise maintainer should assume patches are independent, which is not true. Exception is for large patchsets where we do not want to spam everyone and these require different approaches. It's not the case here. Best regards, Krzysztof ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v3 3/3] mtd: parsers: add support for Sercomm partitions 2022-05-03 15:42 ` Mikhail Zhilkin @ 2022-05-03 15:51 ` Mikhail Zhilkin -1 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:51 UTC (permalink / raw) To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: Mikhail Zhilkin, Hauke Mehrtens, NOGUCHI Hiroshi, INAGAKI Hiroshi, linux-kernel, linux-mtd, Karim, M This adds an 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. This is essentially the same code as proposed by NOGUCHI Hiroshi: Link: https://github.com/openwrt/openwrt/pull/1318#issuecomment-420607394 Signed-off-by: NOGUCHI Hiroshi <drvlabo@gmail.com> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> --- drivers/mtd/parsers/Kconfig | 9 ++ drivers/mtd/parsers/Makefile | 1 + drivers/mtd/parsers/scpart.c | 240 +++++++++++++++++++++++++++++++++++ 3 files changed, 250 insertions(+) create mode 100644 drivers/mtd/parsers/scpart.c diff --git a/drivers/mtd/parsers/Kconfig b/drivers/mtd/parsers/Kconfig index 23763d16e4f9..851f4670d89b 100644 --- a/drivers/mtd/parsers/Kconfig +++ b/drivers/mtd/parsers/Kconfig @@ -186,3 +186,12 @@ config MTD_QCOMSMEM_PARTS help This provides support for parsing partitions from Shared Memory (SMEM) for NAND and SPI flash on Qualcomm platforms. + +config MTD_SERCOMM_PARTS + tristate "Sercomm partition table parser" + depends on MTD + help + This provides partitions table parser for devices with Sercomm + partition map. This partition table contains real partition + offsets, which may differ from device to device depending on the + number and location of bad blocks on NAND. diff --git a/drivers/mtd/parsers/Makefile b/drivers/mtd/parsers/Makefile index 2e98aa048278..2fcf0ab9e7da 100644 --- a/drivers/mtd/parsers/Makefile +++ b/drivers/mtd/parsers/Makefile @@ -10,6 +10,7 @@ ofpart-$(CONFIG_MTD_OF_PARTS_LINKSYS_NS)+= ofpart_linksys_ns.o obj-$(CONFIG_MTD_PARSER_IMAGETAG) += parser_imagetag.o obj-$(CONFIG_MTD_AFS_PARTS) += afs.o obj-$(CONFIG_MTD_PARSER_TRX) += parser_trx.o +obj-$(CONFIG_MTD_SERCOMM_PARTS) += scpart.o obj-$(CONFIG_MTD_SHARPSL_PARTS) += sharpslpart.o obj-$(CONFIG_MTD_REDBOOT_PARTS) += redboot.o obj-$(CONFIG_MTD_QCOMSMEM_PARTS) += qcomsmempart.o diff --git a/drivers/mtd/parsers/scpart.c b/drivers/mtd/parsers/scpart.c new file mode 100644 index 000000000000..d7bb736c9f2a --- /dev/null +++ b/drivers/mtd/parsers/scpart.c @@ -0,0 +1,240 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * drivers/mtd/scpart.c: Sercomm Partition Parser + * + * Copyright (C) 2018 NOGUCHI Hiroshi + * Copyright (C) 2022 Mikhail Zhilkin + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/mtd/mtd.h> +#include <linux/mtd/partitions.h> +#include <linux/module.h> + +#define MOD_NAME "scpart" + +#ifdef pr_fmt +#undef pr_fmt +#endif + +#define pr_fmt(fmt) MOD_NAME ": " fmt + +static const char sc_part_magic[] = { + 'S', 'C', 'F', 'L', 'M', 'A', 'P', 'O', 'K', '\0', +}; +#define PART_MAGIC_LEN sizeof(sc_part_magic) + +/* assumes that all fields are set by CPU native endian */ +struct sc_part_desc { + uint32_t part_id; + uint32_t part_offs; + uint32_t part_bytes; +}; +#define ID_ALREADY_FOUND 0xFFFFFFFFUL + +#define MAP_OFFS_IN_BLK 0x800 + +#define MAP_MIRROR_NUM 2 + +static int scpart_desc_is_valid(struct sc_part_desc *pdesc) +{ + return ((pdesc->part_id != 0xFFFFFFFFUL) && + (pdesc->part_offs != 0xFFFFFFFFUL) && + (pdesc->part_bytes != 0xFFFFFFFFUL)); +} + +static int scpart_scan_partmap(struct mtd_info *master, loff_t partmap_offs, + struct sc_part_desc **ppdesc) +{ + uint8_t *buf; + loff_t offs; + size_t retlen; + struct sc_part_desc *pdesc = NULL; + struct sc_part_desc *tmpdesc; + int cnt = 0; + int res2; + int res = 0; + + buf = kzalloc(master->erasesize, GFP_KERNEL); + if (!buf) { + res = -ENOMEM; + goto out; + } + + res2 = mtd_read(master, partmap_offs, master->erasesize, &retlen, buf); + if (res2 || retlen != master->erasesize) { + res = -EIO; + goto free; + } + + offs = MAP_OFFS_IN_BLK; + while (offs < (master->erasesize - sizeof(*tmpdesc))) { + tmpdesc = (struct sc_part_desc *)&(buf[offs]); + if (!scpart_desc_is_valid(tmpdesc)) + break; + cnt++; + offs += sizeof(*tmpdesc); + } + + if (cnt > 0) { + int bytes = cnt * sizeof(*pdesc); + + pdesc = kcalloc(cnt, sizeof(*pdesc), GFP_KERNEL); + if (!pdesc) { + res = -ENOMEM; + goto free; + } + memcpy(pdesc, &(buf[MAP_OFFS_IN_BLK]), bytes); + + *ppdesc = pdesc; + res = cnt; + } + +free: + kfree(buf); + +out: + return res; +} + +static int scpart_find_partmap(struct mtd_info *master, + struct sc_part_desc **ppdesc) +{ + loff_t offs; + uint8_t rdbuf[PART_MAGIC_LEN]; + size_t retlen; + int magic_found = 0; + int res2; + int res = 0; + + offs = 0; + while ((magic_found < MAP_MIRROR_NUM) && + (offs < master->size) && !mtd_block_isbad(master, offs)) { + res2 = mtd_read(master, offs, PART_MAGIC_LEN, &retlen, rdbuf); + if (res2 || (retlen != PART_MAGIC_LEN)) { + res = -EIO; + goto out; + } + if (!memcmp(rdbuf, sc_part_magic, PART_MAGIC_LEN)) { + pr_debug("Signature found at 0x%llx\n", offs); + magic_found++; + res = scpart_scan_partmap(master, offs, ppdesc); + if (res > 0) + goto out; + } + offs += master->erasesize; + } + +out: + if (res > 0) + pr_info("Valid 'SC PART MAP' (%d partitions) found at 0x%llx\n", res, offs); + else + pr_info("No valid 'SC PART MAP' was found\n"); + + return res; +} + +static int scpart_parse(struct mtd_info *master, + const struct mtd_partition **pparts, + struct mtd_part_parser_data *data) +{ + struct sc_part_desc *scpart_map = NULL; + struct mtd_partition *parts = NULL; + struct device_node *mtd_node; + struct device_node *ofpart_node; + struct device_node *pp; + const char *partname; + int nr_scparts; + int nr_parts = 0; + int n; + int res = 0; + + mtd_node = mtd_get_of_node(master); + if (!mtd_node) + goto out; + + ofpart_node = of_get_child_by_name(mtd_node, "partitions"); + if (!ofpart_node) + goto out; + + nr_scparts = scpart_find_partmap(master, &scpart_map); + if (nr_scparts <= 0) { + res = nr_scparts; + goto free; + } + + parts = kcalloc(of_get_child_count(ofpart_node), sizeof(*parts), + GFP_KERNEL); + if (!parts) { + res = -ENOMEM; + goto out; + } + + for_each_child_of_node(ofpart_node, pp) { + u32 scpart_id; + + if (of_property_read_u32(pp, "sercomm,scpart-id", &scpart_id)) + continue; + + for (n = 0 ; n < nr_scparts ; n++) + if ((scpart_map[n].part_id != ID_ALREADY_FOUND) && + (scpart_id == scpart_map[n].part_id)) + break; + if (n >= nr_scparts) + /* not match */ + continue; + + /* add the partition found in OF into MTD partition array */ + parts[nr_parts].offset = scpart_map[n].part_offs; + parts[nr_parts].size = scpart_map[n].part_bytes; + parts[nr_parts].of_node = pp; + + if (!of_property_read_string(pp, "label", &partname)) + parts[nr_parts].name = partname; + if (of_property_read_bool(pp, "read-only")) + parts[nr_parts].mask_flags |= MTD_WRITEABLE; + if (of_property_read_bool(pp, "lock")) + parts[nr_parts].mask_flags |= MTD_POWERUP_LOCK; + + /* mark as 'done' */ + scpart_map[n].part_id = ID_ALREADY_FOUND; + + nr_parts++; + } + + if (nr_parts > 0) { + *pparts = parts; + res = nr_parts; + } else + pr_info("No partition in OF matches partition ID with 'SC PART MAP'.\n"); + + of_node_put(pp); + +free: + kfree(scpart_map); + if (res <= 0) + kfree(parts); + +out: + return res; +} + +static const struct of_device_id scpart_parser_of_match_table[] = { + { .compatible = "sercomm,sc-partitions" }, + {}, +}; +MODULE_DEVICE_TABLE(of, scpart_parser_of_match_table); + +static struct mtd_part_parser scpart_parser = { + .parse_fn = scpart_parse, + .name = "scpart", + .of_match_table = scpart_parser_of_match_table, +}; +module_mtd_part_parser(scpart_parser); + +/* mtd parsers will request the module by parser name */ +MODULE_ALIAS("scpart"); +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("NOGUCHI Hiroshi <drvlabo@gmail.com>"); +MODULE_DESCRIPTION("Sercomm partition parser"); -- 2.25.1 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v3 3/3] mtd: parsers: add support for Sercomm partitions @ 2022-05-03 15:51 ` Mikhail Zhilkin 0 siblings, 0 replies; 22+ messages in thread From: Mikhail Zhilkin @ 2022-05-03 15:51 UTC (permalink / raw) To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: Mikhail Zhilkin, Hauke Mehrtens, NOGUCHI Hiroshi, INAGAKI Hiroshi, linux-kernel, linux-mtd, Karim, M This adds an 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. This is essentially the same code as proposed by NOGUCHI Hiroshi: Link: https://github.com/openwrt/openwrt/pull/1318#issuecomment-420607394 Signed-off-by: NOGUCHI Hiroshi <drvlabo@gmail.com> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> --- drivers/mtd/parsers/Kconfig | 9 ++ drivers/mtd/parsers/Makefile | 1 + drivers/mtd/parsers/scpart.c | 240 +++++++++++++++++++++++++++++++++++ 3 files changed, 250 insertions(+) create mode 100644 drivers/mtd/parsers/scpart.c diff --git a/drivers/mtd/parsers/Kconfig b/drivers/mtd/parsers/Kconfig index 23763d16e4f9..851f4670d89b 100644 --- a/drivers/mtd/parsers/Kconfig +++ b/drivers/mtd/parsers/Kconfig @@ -186,3 +186,12 @@ config MTD_QCOMSMEM_PARTS help This provides support for parsing partitions from Shared Memory (SMEM) for NAND and SPI flash on Qualcomm platforms. + +config MTD_SERCOMM_PARTS + tristate "Sercomm partition table parser" + depends on MTD + help + This provides partitions table parser for devices with Sercomm + partition map. This partition table contains real partition + offsets, which may differ from device to device depending on the + number and location of bad blocks on NAND. diff --git a/drivers/mtd/parsers/Makefile b/drivers/mtd/parsers/Makefile index 2e98aa048278..2fcf0ab9e7da 100644 --- a/drivers/mtd/parsers/Makefile +++ b/drivers/mtd/parsers/Makefile @@ -10,6 +10,7 @@ ofpart-$(CONFIG_MTD_OF_PARTS_LINKSYS_NS)+= ofpart_linksys_ns.o obj-$(CONFIG_MTD_PARSER_IMAGETAG) += parser_imagetag.o obj-$(CONFIG_MTD_AFS_PARTS) += afs.o obj-$(CONFIG_MTD_PARSER_TRX) += parser_trx.o +obj-$(CONFIG_MTD_SERCOMM_PARTS) += scpart.o obj-$(CONFIG_MTD_SHARPSL_PARTS) += sharpslpart.o obj-$(CONFIG_MTD_REDBOOT_PARTS) += redboot.o obj-$(CONFIG_MTD_QCOMSMEM_PARTS) += qcomsmempart.o diff --git a/drivers/mtd/parsers/scpart.c b/drivers/mtd/parsers/scpart.c new file mode 100644 index 000000000000..d7bb736c9f2a --- /dev/null +++ b/drivers/mtd/parsers/scpart.c @@ -0,0 +1,240 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * drivers/mtd/scpart.c: Sercomm Partition Parser + * + * Copyright (C) 2018 NOGUCHI Hiroshi + * Copyright (C) 2022 Mikhail Zhilkin + */ + +#include <linux/kernel.h> +#include <linux/slab.h> +#include <linux/mtd/mtd.h> +#include <linux/mtd/partitions.h> +#include <linux/module.h> + +#define MOD_NAME "scpart" + +#ifdef pr_fmt +#undef pr_fmt +#endif + +#define pr_fmt(fmt) MOD_NAME ": " fmt + +static const char sc_part_magic[] = { + 'S', 'C', 'F', 'L', 'M', 'A', 'P', 'O', 'K', '\0', +}; +#define PART_MAGIC_LEN sizeof(sc_part_magic) + +/* assumes that all fields are set by CPU native endian */ +struct sc_part_desc { + uint32_t part_id; + uint32_t part_offs; + uint32_t part_bytes; +}; +#define ID_ALREADY_FOUND 0xFFFFFFFFUL + +#define MAP_OFFS_IN_BLK 0x800 + +#define MAP_MIRROR_NUM 2 + +static int scpart_desc_is_valid(struct sc_part_desc *pdesc) +{ + return ((pdesc->part_id != 0xFFFFFFFFUL) && + (pdesc->part_offs != 0xFFFFFFFFUL) && + (pdesc->part_bytes != 0xFFFFFFFFUL)); +} + +static int scpart_scan_partmap(struct mtd_info *master, loff_t partmap_offs, + struct sc_part_desc **ppdesc) +{ + uint8_t *buf; + loff_t offs; + size_t retlen; + struct sc_part_desc *pdesc = NULL; + struct sc_part_desc *tmpdesc; + int cnt = 0; + int res2; + int res = 0; + + buf = kzalloc(master->erasesize, GFP_KERNEL); + if (!buf) { + res = -ENOMEM; + goto out; + } + + res2 = mtd_read(master, partmap_offs, master->erasesize, &retlen, buf); + if (res2 || retlen != master->erasesize) { + res = -EIO; + goto free; + } + + offs = MAP_OFFS_IN_BLK; + while (offs < (master->erasesize - sizeof(*tmpdesc))) { + tmpdesc = (struct sc_part_desc *)&(buf[offs]); + if (!scpart_desc_is_valid(tmpdesc)) + break; + cnt++; + offs += sizeof(*tmpdesc); + } + + if (cnt > 0) { + int bytes = cnt * sizeof(*pdesc); + + pdesc = kcalloc(cnt, sizeof(*pdesc), GFP_KERNEL); + if (!pdesc) { + res = -ENOMEM; + goto free; + } + memcpy(pdesc, &(buf[MAP_OFFS_IN_BLK]), bytes); + + *ppdesc = pdesc; + res = cnt; + } + +free: + kfree(buf); + +out: + return res; +} + +static int scpart_find_partmap(struct mtd_info *master, + struct sc_part_desc **ppdesc) +{ + loff_t offs; + uint8_t rdbuf[PART_MAGIC_LEN]; + size_t retlen; + int magic_found = 0; + int res2; + int res = 0; + + offs = 0; + while ((magic_found < MAP_MIRROR_NUM) && + (offs < master->size) && !mtd_block_isbad(master, offs)) { + res2 = mtd_read(master, offs, PART_MAGIC_LEN, &retlen, rdbuf); + if (res2 || (retlen != PART_MAGIC_LEN)) { + res = -EIO; + goto out; + } + if (!memcmp(rdbuf, sc_part_magic, PART_MAGIC_LEN)) { + pr_debug("Signature found at 0x%llx\n", offs); + magic_found++; + res = scpart_scan_partmap(master, offs, ppdesc); + if (res > 0) + goto out; + } + offs += master->erasesize; + } + +out: + if (res > 0) + pr_info("Valid 'SC PART MAP' (%d partitions) found at 0x%llx\n", res, offs); + else + pr_info("No valid 'SC PART MAP' was found\n"); + + return res; +} + +static int scpart_parse(struct mtd_info *master, + const struct mtd_partition **pparts, + struct mtd_part_parser_data *data) +{ + struct sc_part_desc *scpart_map = NULL; + struct mtd_partition *parts = NULL; + struct device_node *mtd_node; + struct device_node *ofpart_node; + struct device_node *pp; + const char *partname; + int nr_scparts; + int nr_parts = 0; + int n; + int res = 0; + + mtd_node = mtd_get_of_node(master); + if (!mtd_node) + goto out; + + ofpart_node = of_get_child_by_name(mtd_node, "partitions"); + if (!ofpart_node) + goto out; + + nr_scparts = scpart_find_partmap(master, &scpart_map); + if (nr_scparts <= 0) { + res = nr_scparts; + goto free; + } + + parts = kcalloc(of_get_child_count(ofpart_node), sizeof(*parts), + GFP_KERNEL); + if (!parts) { + res = -ENOMEM; + goto out; + } + + for_each_child_of_node(ofpart_node, pp) { + u32 scpart_id; + + if (of_property_read_u32(pp, "sercomm,scpart-id", &scpart_id)) + continue; + + for (n = 0 ; n < nr_scparts ; n++) + if ((scpart_map[n].part_id != ID_ALREADY_FOUND) && + (scpart_id == scpart_map[n].part_id)) + break; + if (n >= nr_scparts) + /* not match */ + continue; + + /* add the partition found in OF into MTD partition array */ + parts[nr_parts].offset = scpart_map[n].part_offs; + parts[nr_parts].size = scpart_map[n].part_bytes; + parts[nr_parts].of_node = pp; + + if (!of_property_read_string(pp, "label", &partname)) + parts[nr_parts].name = partname; + if (of_property_read_bool(pp, "read-only")) + parts[nr_parts].mask_flags |= MTD_WRITEABLE; + if (of_property_read_bool(pp, "lock")) + parts[nr_parts].mask_flags |= MTD_POWERUP_LOCK; + + /* mark as 'done' */ + scpart_map[n].part_id = ID_ALREADY_FOUND; + + nr_parts++; + } + + if (nr_parts > 0) { + *pparts = parts; + res = nr_parts; + } else + pr_info("No partition in OF matches partition ID with 'SC PART MAP'.\n"); + + of_node_put(pp); + +free: + kfree(scpart_map); + if (res <= 0) + kfree(parts); + +out: + return res; +} + +static const struct of_device_id scpart_parser_of_match_table[] = { + { .compatible = "sercomm,sc-partitions" }, + {}, +}; +MODULE_DEVICE_TABLE(of, scpart_parser_of_match_table); + +static struct mtd_part_parser scpart_parser = { + .parse_fn = scpart_parse, + .name = "scpart", + .of_match_table = scpart_parser_of_match_table, +}; +module_mtd_part_parser(scpart_parser); + +/* mtd parsers will request the module by parser name */ +MODULE_ALIAS("scpart"); +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("NOGUCHI Hiroshi <drvlabo@gmail.com>"); +MODULE_DESCRIPTION("Sercomm partition parser"); -- 2.25.1 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v3 3/3] mtd: parsers: add support for Sercomm partitions 2022-05-03 15:51 ` Mikhail Zhilkin @ 2022-05-12 15:27 ` Miquel Raynal -1 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-12 15:27 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: Hauke Mehrtens, NOGUCHI Hiroshi, INAGAKI Hiroshi, linux-kernel, linux-mtd, Karim, M On Tue, 2022-05-03 at 15:51:40 UTC, Mikhail Zhilkin wrote: > This adds an 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. > > This is essentially the same code as proposed by NOGUCHI Hiroshi: > Link: https://github.com/openwrt/openwrt/pull/1318#issuecomment-420607394 > > Signed-off-by: NOGUCHI Hiroshi <drvlabo@gmail.com> > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Miquel ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3 3/3] mtd: parsers: add support for Sercomm partitions @ 2022-05-12 15:27 ` Miquel Raynal 0 siblings, 0 replies; 22+ messages in thread From: Miquel Raynal @ 2022-05-12 15:27 UTC (permalink / raw) To: Mikhail Zhilkin, Miquel Raynal, Richard Weinberger, Vignesh Raghavendra Cc: Hauke Mehrtens, NOGUCHI Hiroshi, INAGAKI Hiroshi, linux-kernel, linux-mtd, Karim, M On Tue, 2022-05-03 at 15:51:40 UTC, Mikhail Zhilkin wrote: > This adds an 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. > > This is essentially the same code as proposed by NOGUCHI Hiroshi: > Link: https://github.com/openwrt/openwrt/pull/1318#issuecomment-420607394 > > Signed-off-by: NOGUCHI Hiroshi <drvlabo@gmail.com> > Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks. Miquel ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2022-05-16 15:40 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-05-03 15:42 [PATCH v3 0/3] mtd: parsers: add support for Sercomm partitions Mikhail Zhilkin 2022-05-03 15:42 ` Mikhail Zhilkin 2022-05-03 15:47 ` [PATCH v3 1/3] dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix Mikhail Zhilkin 2022-05-04 15:07 ` Krzysztof Kozlowski 2022-05-03 15:50 ` [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding Mikhail Zhilkin 2022-05-03 15:50 ` Mikhail Zhilkin 2022-05-04 15:07 ` Krzysztof Kozlowski 2022-05-04 15:07 ` Krzysztof Kozlowski 2022-05-12 15:27 ` Miquel Raynal 2022-05-12 15:27 ` Miquel Raynal 2022-05-13 14:12 ` Rob Herring 2022-05-13 14:12 ` Rob Herring 2022-05-16 8:21 ` Miquel Raynal 2022-05-16 8:21 ` Miquel Raynal 2022-05-16 15:34 ` Mikhail Zhilkin 2022-05-16 15:34 ` Mikhail Zhilkin 2022-05-16 15:39 ` Krzysztof Kozlowski 2022-05-16 15:39 ` Krzysztof Kozlowski 2022-05-03 15:51 ` [PATCH v3 3/3] mtd: parsers: add support for Sercomm partitions Mikhail Zhilkin 2022-05-03 15:51 ` Mikhail Zhilkin 2022-05-12 15:27 ` Miquel Raynal 2022-05-12 15:27 ` Miquel Raynal
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.