Re: [PATCH v4 1/7] dt-bindings: mtd: jedec,spi-nor: allow the SFDP to be exposed via NVMEM

[Linus Walleij]

Hi Manikandan,

thanks for your patch!

On Tue, Jun 30, 2026 at 11:24 AM Manikandan Muralidharan
<manikandan.m@xxxxxxxxxxxxx> wrote:

> Add an optional "sfdp" child node (compatible "jedec,sfdp") that
> describes the SFDP as a read-only NVMEM provider via nvmem.yaml, so its
> contents (e.g. a vendor EUI-48/EUI-64) can be read through NVMEM cells.
>
> Signed-off-by: Manikandan Muralidharan <manikandan.m@xxxxxxxxxxxxx>

I would expect it to follow nvmem conventions like this, notice
compatibles specific-to-general with sfdp first:

sfdp {
    /* NVMEM provided by SFDP */
    compatible = "jedec,sfdp", "nvmem-cells";
    label = "SFDP";
    read-only;
    #address-cells = <1>;
    #size-cells = <1>;

    mac0: macaddr@0x00 {
        reg = <0x00 0x06>;
    };
    mac1: macaddr@0x06 {
        reg = <0x06 0x06>;
    };
};

Your example should definitely be more elaborate like this,
just an opaque sfdp node will not suffice. Maybe a separate
example?

Yours,
Linus Walleij