Skip to contentSkip to conversation

mtd: parsers: add support for Sercomm partitions

PATCH Applied Mikhail Zhilkin 4 participants 12 messages May 3, 2022 – May 16, 2022

Subsystems (2)

v3 of 6 Applied Latest is v6 →

lore
Share & tools
lore
#0 lore raw

v3 cover mtd: parsers: add support for Sercomm partitions

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
Signature · 1 line
2.25.1
#1 lore raw

PATCH v3 1/3 dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix

Update Documentation/devicetree/bindings/vendor-prefixes.yaml to include "sercomm" as a vendor prefix for "Sercomm (Suzhou) Corporation". Company website:

---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)
Documentation/devicetree/bindings/vendor-prefixes.yaml +2 -0
index 01430973ecec..65ff22364fb3 100644
@@ -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,.*":
Signature · 1 line
2.25.1
#2 lore raw

Re: [PATCH v3 1/3] dt-bindings: Add Sercomm (Suzhou) Corporation vendor prefix

Quoted 1 line · Mikhail Zhilkin

Update Documentation/devicetree/bindings/vendor-prefixes.yaml

This is visible from the diff, so just "Add sercomm vendor prefix for ...."

to include

Quoted 8 lines

"sercomm" as a vendor prefix for "Sercomm (Suzhou) Corporation". Company website:

---
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 1 file changed, 2 insertions(+)

Best regards,
Krzysztof

#3 lore raw

PATCH v3 2/3 dt-bindings: mtd: partitions: Extend fixed-partitions binding

In reply to Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

---
 .../mtd/partitions/fixed-partitions.yaml      | 55 ++++++++++++++++++-
 1 file changed, 53 insertions(+), 2 deletions(-)
Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.yaml +53 -2
index ea4cace6a955..ad3ccd250802 100644
@@ -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;
+        };
+    };
Signature · 6 lines
2.25.1


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
maintainer Applied
#5 lore raw

Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding

In reply to Mikhail Zhilkin
Quoted 5 lines · Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks.

Miquel

#6 lore raw

Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding

Quoted 9 lines · Miquel Raynal
Quoted 5 lines · Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

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

maintainer
#7 lore raw

Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding

Hi,

[email protected] wrote on Fri, 13 May 2022 09:12:03 -0500:

Quoted 7 lines
Quoted 5 lines · Miquel Raynal
Quoted 3 lines · Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

Quoted 7 lines
Quoted 4 lines
Quoted 2 lines

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.

Quoted 171 lines

/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

Quoted 1 line

Rob

Thanks,
Miquèl

#8 lore raw

Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding

Hi, Miquel!

Quoted 10 lines · Miquel Raynal

Hi,

[email protected] wrote on Fri, 13 May 2022 09:12:03 -0500:

Quoted 6 lines
Quoted 4 lines · Miquel Raynal
Quoted 3 lines · Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Quoted 2 lines

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/[email protected]/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 <[email protected]> (maintainer:OPEN FIRMWARE AND FLATTENED
DEVICE TREE BINDINGS,commit_signer:59/65=91%,in file)
Krzysztof Kozlowski <[email protected]> (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 <[email protected]> (commit_signer:7/65=11%)
Stanislav Jakubek <[email protected]>
(commit_signer:6/65=9%,authored:6/65=9%,added_lines:12/139=9%)
Sam Ravnborg <[email protected]> (commit_signer:5/65=8%)
Linus Walleij <[email protected]> (added_lines:10/139=7%)
Hao Fang <[email protected]> (removed_lines:1/2=50%)
[email protected] (open list:OPEN FIRMWARE AND FLATTENED DEVICE
TREE BINDINGS)
[email protected] (open list)
Quoted 175 lines
Quoted 171 lines

/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

Quoted 1 line

Rob

Thanks,
Miquèl

Signature · 2 lines
Best regards,
Mikhail
#9 lore raw

Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend fixed-partitions binding

Quoted 20 lines · Mikhail Zhilkin

Hi, Miquel!

Quoted 17 lines · Miquel Raynal

Hi,

[email protected] wrote on Fri, 13 May 2022 09:12:03 -0500:

Quoted 11 lines
Quoted 7 lines · Miquel Raynal
Quoted 5 lines · Mikhail Zhilkin

Extend fixed-partitions binding for support of Sercomm partition parser (add "sercomm,sc-partitions" compatible).

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.

Quoted 8 lines
Quoted 1 line

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/[email protected]/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

#10 lore raw

PATCH v3 3/3 mtd: parsers: add support for Sercomm partitions

In reply to Mikhail Zhilkin

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:

---
 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
drivers/mtd/parsers/Kconfig +9 -0
index 23763d16e4f9..851f4670d89b 100644
@@ -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.
drivers/mtd/parsers/Makefile +1 -0
index 2e98aa048278..2fcf0ab9e7da 100644
@@ -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
drivers/mtd/parsers/scpart.c new file +240 -0

240 lines not on this page · raw message

Signature · 1 line
2.25.1
maintainer Applied
#11 lore raw
Quoted 7 lines · Mikhail Zhilkin

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.

Quoted 5 lines

This is essentially the same code as proposed by NOGUCHI Hiroshi:

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks.

Miquel