Re: [PATCH v2] nvmem: core: don't consider subnodes not matching binding

[Srinivas Kandagatla]

On 15/04/2020 09:05, Ahmad Fatoum wrote:
> > - If the cell has a compatible node, it must be "nvmem-cell".
> >    Otherwise, a cell with a compatible node is skipped.
This one sounds more sensible!
If we go this way, it needs to be properly documented in the bindings too.


> > - Adding a nvmem cell container with a compatible and making support for
> >    the old binding configurable

Older bindings should be still supported but we should encourage to use 
compatible string for new additions.

> > - Skipping cells that are malformed and lack a reg = < > property_without_
> >    error
Currently we only support Offset and Size with address cell and size 
cells of 1. Adding an extra check here to see if its malformed makes 
sense too!


Thanks,
srini

> gentle ping. I can prepare another patch if you indicate which approach
> you'd prefer.