public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Daniel Golle <daniel@makrotopia.org>
Cc: "Rob Herring (Arm)" <robh@kernel.org>,
	"Chad Monroe" <chad.monroe@adtran.com>,
	"Richard Weinberger" <richard@nod.at>,
	"Vignesh Raghavendra" <vigneshr@ti.com>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"Kamal Dasu" <kdasu.kdev@gmail.com>,
	"William Zhang" <william.zhang@broadcom.com>,
	"Nick Terrell" <terrelln@fb.com>,
	"David Sterba" <dsterba@suse.com>,
	"Rafał Miłecki" <rafal@milecki.pl>,
	"Simon Glass" <sjg@chromium.org>,
	"Linus Walleij" <linusw@kernel.org>,
	"Ulf Hansson" <ulf.hansson@linaro.org>,
	"Marcus Folkesson" <marcus.folkesson@gmail.com>,
	"Tony Lindgren" <tony@atomide.com>,
	"Roger Quadros" <rogerq@kernel.org>,
	"Hauke Mehrtens" <hauke@hauke-m.de>,
	linux-mtd@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org,
	"Krzysztof Kozlowski" <krzysztof.kozlowski@oss.qualcomm.com>
Subject: Re: [PATCH v3 06/10] dt-bindings: mtd: partitions: Drop partitions.yaml
Date: Thu, 22 Jan 2026 11:31:54 +0100	[thread overview]
Message-ID: <87tsweos4l.fsf@bootlin.com> (raw)
In-Reply-To: <aXFCws0XUToQ03UH@makrotopia.org> (Daniel Golle's message of "Wed, 21 Jan 2026 21:18:58 +0000")

On 21/01/2026 at 21:18:58 GMT, Daniel Golle <daniel@makrotopia.org> wrote:

> On Wed, Jan 21, 2026 at 01:56:39PM -0600, Rob Herring (Arm) 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".
>
> It would actually be great to also allow 'gpt-partitions' as compatible
> type with #address-cells = <0> and #size-cells = <0> and allow matching
> on partition UUID, name or index. This has previously been discussed and
> would avoid having to extract MAC addresses and WiFi EEPROM data in
> userspace on many devices which rely on such conventions.

Out of curiosity, why not exposing this data through an NVMEM cell
instead? Anyway, this (re?)addition can probably be part of a follow-up
series and is almost orthogonal to this cleanup IMO.

Thanks,
Miquèl

  reply	other threads:[~2026-01-22 10:32 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-21 19:56 [PATCH v3 00/10] dt-bindings: mtd: Partition binding fixes and restructuring Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 01/10] dt-bindings: mtd: brcm,brcmnand: Drop "brcm,brcmnand" compatible for iProc Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 02/10] dt-bindings: mtd: fixed-partitions: Move "compression" to partition node Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 03/10] dt-bindings: mtd: partitions: Move "sercomm,scpart-id" to partition.yaml Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 04/10] dt-bindings: mtd: partitions: Allow "nvmem-layout" in generic partition nodes Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 05/10] dt-bindings: mtd: partitions: Define "#{address,size}-cells" in specific schemas Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 06/10] dt-bindings: mtd: partitions: Drop partitions.yaml Rob Herring (Arm)
2026-01-21 21:18   ` Daniel Golle
2026-01-22 10:31     ` Miquel Raynal [this message]
2026-01-22 16:42       ` Daniel Golle
2026-01-21 19:56 ` [PATCH v3 07/10] dt-bindings: mtd: Ensure partition node properties are documented Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 08/10] dt-bindings: mtd: fixed-partitions: Restrict undefined properties Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 09/10] dt-bindings: mtd: partitions: Convert brcm,trx to DT schema Rob Herring (Arm)
2026-01-21 19:56 ` [PATCH v3 10/10] dt-bindings: mtd: partitions: Combine simple partition bindings Rob Herring (Arm)
2026-01-22  7:56   ` Krzysztof Kozlowski
2026-01-22 14:09 ` [PATCH v3 00/10] dt-bindings: mtd: Partition binding fixes and restructuring Miquel Raynal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tsweos4l.fsf@bootlin.com \
    --to=miquel.raynal@bootlin.com \
    --cc=chad.monroe@adtran.com \
    --cc=computersforpeace@gmail.com \
    --cc=conor+dt@kernel.org \
    --cc=daniel@makrotopia.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dsterba@suse.com \
    --cc=hauke@hauke-m.de \
    --cc=kdasu.kdev@gmail.com \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@oss.qualcomm.com \
    --cc=linusw@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=marcus.folkesson@gmail.com \
    --cc=rafal@milecki.pl \
    --cc=richard@nod.at \
    --cc=robh@kernel.org \
    --cc=rogerq@kernel.org \
    --cc=sjg@chromium.org \
    --cc=terrelln@fb.com \
    --cc=tony@atomide.com \
    --cc=ulf.hansson@linaro.org \
    --cc=vigneshr@ti.com \
    --cc=william.zhang@broadcom.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox