On Tue, Aug 06, 2024 at 03:00:03PM +0200, Christoph Hellwig wrote: > I don't see how this works. There is absolutely nothing nvme specific > in there, and we really should not bring OF into NVMe. > Thing is that OF for PCIe devices is already a thing so NVMe already have OF support. And that is totally OK if the PCIe is not removable. So it's really a simple subnode to keep things organized in DT and permit common code. Why this would be problematic, the implications of adding support for this are really none, only additional support for the new devices that are taking this path. (you would be surprised by the amount of hacks they use downstream so about times to stabilize this the correct way) > > (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) > > What common scheme? > The emmc one and also nand attached to spi. They all follow this. controller { card { }; }; -- Ansuel