Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding

[Oleksij Rempel]

On Wed, Apr 12, 2017 at 08:23:53AM +0200, Oleksij Rempel wrote:
> On Tue, Apr 11, 2017 at 08:35:00AM -0500, Rob Herring wrote:
> > On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempel <ore@xxxxxxxxxxxxxx> wrote:
> > > Hi,
> > > On 04/10/2017 08:22 PM, Rob Herring wrote:
> > >>
> > >> On Thu, Apr 06, 2017 at 09:31:07AM +0200, Oleksij Rempel wrote:
> > >>>
> > >>> Documenation bindings for the Low Power General Purpose Registe
> > >>
> > > Jes, it is refereed by other driver.
> > 
> > What I mean is snvs-lpgpr referenced elsewhere in DT, not by some
> > driver? You are not using the nvmem binding here so it doesn't seem
> > like it is. If that's the case, then you don't need this node. The
> > only information here is the offset which can be part of a driver for
> > the parent node. To put it another way, we don't want to fill DT with
> > a node per register.
> 
> This node is referenced insight of product DT. Since the system has
> or may have more then one nvmem node we need to assigne aliase to it.

Hi, are there still some doubts about this patch set?

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |