Re: [PATCH v3 1/6] dt-bindings: nvme: Document nvme-card compatible

Christian Marangi
2024-08-06T12:55:20Z

On Tue, Aug 06, 2024 at 02:42:24PM +0200, Christoph Hellwig wrote:
> On Tue, Aug 06, 2024 at 01:41:11PM +0200, Christian Marangi wrote:
> > Document new nvme-card compatible to permit defining fixed-partition in
> > DT by the use of the block2mtd module to use block devices as MTD.
>
> What does nvme card mean? Is this about nvmem or nvme? If this is nvme,
> are you talking about nvme-pci? Why would that needs a device binding
> when it is a PCI device?
>

It's similar to how it's done with mmc and it's to keep the property
consistent with block devices.

emmc have something like

mmc {
mmc-card {
specific-property;

partitions...
};
};

The same will be with nvme with

nvme {
compatible = "pci_id"

nvme-card {
quirks maybe in the future?
partitions...
};
};

The following implementation permits in block2mtd to not complicate the
implementation with all the add_disk functions works with parenting
struct and how they are initialized.

(alternative is to have in block2mtd all kind of extra logic with switch
case to check for major block ID that deviates from a common schema)

--
Ansuel