Re: [PATCH v11 07/10] mtd: spi-nor: Add stacked memories support in spi-nor

Miquel Raynal
2024-08-12T08:40:08Z

Hi Michael,

michael@walle.cc wrote on Mon, 12 Aug 2024 09:37:06 +0200:

> Hi,
>
> > > > > The first round of patches were really invasive regarding the core
> > > > > code. So if there is a clean layering approach which can be enabled
> > > > > as a module and you are maintaining it I'm fine with that (even if
> > > > > the core code needs some changes then like hooks or so, not sure).
> > > >
> > > > That discussion started with Miquel some years ago when he was trying to to
> > > > solve description in DT which is merged for a while in the kernel.
> > >
> > > What's your point here? From what I can tell the DT binding is wrong
> > > and needs to be reworked anyway.
> >
> > I'm sorry I'm now catching up, can you point at the thread explaining
> > what is wrong in the bindings? I didn't find where this was detailed. Or
> > otherwise summarize quickly what needs to change?
>
> Somewhere in this mega thread Tudor had some remarks about the
> bindings. Amit also mentioned it here:
>
> https://lore.kernel.org/r/IA0PR12MB769944254171C39FF4171B52DCB42@IA0PR12MB7699.namprd12.prod.outlook.com/

Great. I jumped-in there. Thanks!

Miquèl