| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAPDyKFo_4jiX7HjqzRc6r_uLG5Chto8aP_e5R9qv=jUjczr2Hg@mail.gmail.com> Date: Tue, 20 Jan 2026 11:40:13 +0100 From: Ulf Hansson <ulf.hansson@...aro.org> To: "Rob Herring (Arm)" <robh@...nel.org> Cc: Miquel Raynal <miquel.raynal@...tlin.com>, Richard Weinberger <richard@....at>, Vignesh Raghavendra <vigneshr@...com>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Brian Norris <computersforpeace@...il.com>, Kamal Dasu <kdasu.kdev@...il.com>, William Zhang <william.zhang@...adcom.com>, Nick Terrell <terrelln@...com>, David Sterba <dsterba@...e.com>, Rafał Miłecki <rafal@...ecki.pl>, Simon Glass <sjg@...omium.org>, Linus Walleij <linusw@...nel.org>, Marcus Folkesson <marcus.folkesson@...il.com>, Tony Lindgren <tony@...mide.com>, Roger Quadros <rogerq@...nel.org>, Hauke Mehrtens <hauke@...ke-m.de>, linux-mtd@...ts.infradead.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-mmc@...r.kernel.org Subject: Re: [PATCH v2 06/10] dt-bindings: mtd: partitions: Drop partitions.yaml On Tue, 20 Jan 2026 at 02:49, Rob Herring (Arm) <robh@...nel.org> wrote: > > The partitions.yaml schema is an unusual structure in that it includes > all possible partition types, and it disables the normal matching by > compatible strings. As partitions.yaml has nothing to match on, it is > only applied when explicitly referenced. The use of "oneOf" also results > in misleading warnings which are difficult to understand. Drop > partitions.yaml and rely on the standard compatible matching instead. > > The "mmc-card" case previously allowed any partition type, but now only > allows "fixed-partitions". There aren't any users and the original > intent appeared to be only for "fixed-partitions". > > Signed-off-by: Rob Herring (Arm) <robh@...nel.org> Acked-by: Ulf Hansson <ulf.hansson@...aro.org> # For MMC Kind regards Uffe > --- > .../devicetree/bindings/mmc/mmc-card.yaml | 20 ++++-------- > Documentation/devicetree/bindings/mtd/mtd.yaml | 2 +- > .../mtd/partitions/arm,arm-firmware-suite.yaml | 2 -- > .../mtd/partitions/brcm,bcm4908-partitions.yaml | 2 -- > .../partitions/brcm,bcm947xx-cfe-partitions.yaml | 2 -- > .../mtd/partitions/linksys,ns-partitions.yaml | 2 -- > .../bindings/mtd/partitions/partitions.yaml | 36 ---------------------- > .../devicetree/bindings/mtd/ti,davinci-nand.yaml | 4 ++- > 8 files changed, 10 insertions(+), 60 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/mmc-card.yaml b/Documentation/devicetree/bindings/mmc/mmc-card.yaml > index 1d91d4272de0..a61d6c96df75 100644 > --- a/Documentation/devicetree/bindings/mmc/mmc-card.yaml > +++ b/Documentation/devicetree/bindings/mmc/mmc-card.yaml > @@ -32,21 +32,13 @@ properties: > > patternProperties: > "^partitions(-boot[12]|-gp[14])?$": > - $ref: /schemas/mtd/partitions/partitions.yaml > + type: object > + additionalProperties: true > > - patternProperties: > - "^partition@[0-9a-f]+$": > - $ref: /schemas/mtd/partitions/partition.yaml > - > - properties: > - reg: > - description: Must be multiple of 512 as it's converted > - internally from bytes to SECTOR_SIZE (512 bytes) > - > - required: > - - reg > - > - unevaluatedProperties: false > + properties: > + compatible: > + contains: > + const: fixed-partitions > > required: > - compatible > diff --git a/Documentation/devicetree/bindings/mtd/mtd.yaml b/Documentation/devicetree/bindings/mtd/mtd.yaml > index bbb56216a4e2..e56dba83f00a 100644 > --- a/Documentation/devicetree/bindings/mtd/mtd.yaml > +++ b/Documentation/devicetree/bindings/mtd/mtd.yaml > @@ -30,7 +30,7 @@ properties: > deprecated: true > > partitions: > - $ref: /schemas/mtd/partitions/partitions.yaml > + type: object > > required: > - compatible > diff --git a/Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.yaml b/Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.yaml > index 97618847ee35..76c88027b6d2 100644 > --- a/Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.yaml > +++ b/Documentation/devicetree/bindings/mtd/partitions/arm,arm-firmware-suite.yaml > @@ -9,8 +9,6 @@ title: ARM Firmware Suite (AFS) Partitions > maintainers: > - Linus Walleij <linus.walleij@...aro.org> > > -select: false > - > description: | > The ARM Firmware Suite is a flash partitioning system found on the > ARM reference designs: Integrator AP, Integrator CP, Versatile AB, > diff --git a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml > index 94f0742b375c..159b32d12803 100644 > --- a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml > +++ b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml > @@ -17,8 +17,6 @@ description: | > maintainers: > - Rafał Miłecki <rafal@...ecki.pl> > > -select: false > - > properties: > compatible: > const: brcm,bcm4908-partitions > diff --git a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm947xx-cfe-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm947xx-cfe-partitions.yaml > index 939e7b50db22..3484e06d6bcb 100644 > --- a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm947xx-cfe-partitions.yaml > +++ b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm947xx-cfe-partitions.yaml > @@ -35,8 +35,6 @@ description: | > maintainers: > - Rafał Miłecki <rafal@...ecki.pl> > > -select: false > - > properties: > compatible: > const: brcm,bcm947xx-cfe-partitions > diff --git a/Documentation/devicetree/bindings/mtd/partitions/linksys,ns-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/linksys,ns-partitions.yaml > index c5fa78ff7125..02ecb51fcece 100644 > --- a/Documentation/devicetree/bindings/mtd/partitions/linksys,ns-partitions.yaml > +++ b/Documentation/devicetree/bindings/mtd/partitions/linksys,ns-partitions.yaml > @@ -18,8 +18,6 @@ description: | > maintainers: > - Rafał Miłecki <rafal@...ecki.pl> > > -select: false > - > properties: > compatible: > const: linksys,ns-partitions > diff --git a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml > deleted file mode 100644 > index 76199506d690..000000000000 > --- a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml > +++ /dev/null > @@ -1,36 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > -%YAML 1.2 > ---- > -$id: http://devicetree.org/schemas/mtd/partitions/partitions.yaml# > -$schema: http://devicetree.org/meta-schemas/core.yaml# > - > -title: Partitions > - > -description: | > - This binding is generic and describes the content of the partitions container > - node. All partition parsers must be referenced here. > - > -maintainers: > - - Miquel Raynal <miquel.raynal@...tlin.com> > - > -oneOf: > - - $ref: arm,arm-firmware-suite.yaml > - - $ref: brcm,bcm4908-partitions.yaml > - - $ref: brcm,bcm947xx-cfe-partitions.yaml > - - $ref: fixed-partitions.yaml > - - $ref: linksys,ns-partitions.yaml > - - $ref: qcom,smem-part.yaml > - - $ref: redboot-fis.yaml > - - $ref: tplink,safeloader-partitions.yaml > - > -properties: > - compatible: true > - > -patternProperties: > - "^partition(-.+|@[0-9a-f]+)$": > - $ref: partition.yaml > - > -required: > - - compatible > - > -unevaluatedProperties: false > diff --git a/Documentation/devicetree/bindings/mtd/ti,davinci-nand.yaml b/Documentation/devicetree/bindings/mtd/ti,davinci-nand.yaml > index ed24b0ea86e5..7619b19e7a04 100644 > --- a/Documentation/devicetree/bindings/mtd/ti,davinci-nand.yaml > +++ b/Documentation/devicetree/bindings/mtd/ti,davinci-nand.yaml > @@ -24,7 +24,9 @@ properties: > - description: AEMIF control registers. > > partitions: > - $ref: /schemas/mtd/partitions/partitions.yaml > + type: object > + required: > + - compatible > > ti,davinci-chipselect: > description: > > -- > 2.51.0 >
Powered by blists - more mailing lists