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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 17 May 2017 09:37:45 +0200
From:   Oleksij Rempel <ore@...gutronix.de>
To:     Rob Herring <robh@...nel.org>
Cc:     Oleksij Rempel <o.rempel@...gutronix.de>,
        Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
        Maxime Ripard <maxime.ripard@...e-electrons.com>,
        Mark Rutland <mark.rutland@....com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        kernel@...gutrnoix.de
Subject: Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding

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@...gutronix.de> 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 |