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> > 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- |
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 > > 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- |
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Tue, Apr 11, 2017 at 08:35:00AM -0500, Rob Herring wrote: > On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempelwrote: > > 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. -- 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- |
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Tue, Apr 11, 2017 at 08:35:00AM -0500, Rob Herring wrote: > On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempel 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. -- 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- |
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempelwrote: > 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 >> >> >> s/Registe/Register/ >> >>> available on i.MX6 SoCs in the Secure Non-Volatile Storage. >>> >>> Signed-off-by: Oleksij Rempel >>> Cc: Srinivas Kandagatla >>> Cc: Maxime Ripard >>> Cc: Rob Herring >>> Cc: Mark Rutland >>> Cc: devicet...@vger.kernel.org >>> Cc: linux-kernel@vger.kernel.org >>> --- >>> Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 >>> +++ >>> 1 file changed, 15 insertions(+) >>> create mode 100644 >>> Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> >>> diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> new file mode 100644 >>> index ..9a8be1a2d12e >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> @@ -0,0 +1,15 @@ >>> +Device tree bindings for Low Power General Purpose Registe found in >>> i.MX6Q/D >>> +Secure Non-Volatile Storage. >>> + >>> +Required properties: >>> +- compatible: should be one of >>> + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). >>> +- offset: Should contain the offset relative to syscon parrent node. >> >> >> typo > > > ok. > >> +- regmap: Should contain a phandle pointing to syscon. >>> >>> + >>> +Example: >>> + snvs_lpgpr: snvs-lpgpr { >>> + compatible = "fsl,imx6q-snvs-lpgpr"; >>> + regmap = <>; >>> + offset = <0x68>; >> >> >> Why does this need to be in DT? Is something going to refer to this >> node? If not, the node should be enough information for the OS. > > > 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. Rob
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempel 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 >> >> >> s/Registe/Register/ >> >>> available on i.MX6 SoCs in the Secure Non-Volatile Storage. >>> >>> Signed-off-by: Oleksij Rempel >>> Cc: Srinivas Kandagatla >>> Cc: Maxime Ripard >>> Cc: Rob Herring >>> Cc: Mark Rutland >>> Cc: devicet...@vger.kernel.org >>> Cc: linux-kernel@vger.kernel.org >>> --- >>> Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 >>> +++ >>> 1 file changed, 15 insertions(+) >>> create mode 100644 >>> Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> >>> diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> new file mode 100644 >>> index ..9a8be1a2d12e >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt >>> @@ -0,0 +1,15 @@ >>> +Device tree bindings for Low Power General Purpose Registe found in >>> i.MX6Q/D >>> +Secure Non-Volatile Storage. >>> + >>> +Required properties: >>> +- compatible: should be one of >>> + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). >>> +- offset: Should contain the offset relative to syscon parrent node. >> >> >> typo > > > ok. > >> +- regmap: Should contain a phandle pointing to syscon. >>> >>> + >>> +Example: >>> + snvs_lpgpr: snvs-lpgpr { >>> + compatible = "fsl,imx6q-snvs-lpgpr"; >>> + regmap = <>; >>> + offset = <0x68>; >> >> >> Why does this need to be in DT? Is something going to refer to this >> node? If not, the node should be enough information for the OS. > > > 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. Rob
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
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 s/Registe/Register/ available on i.MX6 SoCs in the Secure Non-Volatile Storage. Signed-off-by: Oleksij RempelCc: Srinivas Kandagatla Cc: Maxime Ripard Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt new file mode 100644 index ..9a8be1a2d12e --- /dev/null +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt @@ -0,0 +1,15 @@ +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D +Secure Non-Volatile Storage. + +Required properties: +- compatible: should be one of + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). +- offset: Should contain the offset relative to syscon parrent node. typo ok. +- regmap: Should contain a phandle pointing to syscon. + +Example: + snvs_lpgpr: snvs-lpgpr { + compatible = "fsl,imx6q-snvs-lpgpr"; + regmap = <>; + offset = <0x68>; Why does this need to be in DT? Is something going to refer to this node? If not, the node should be enough information for the OS. Jes, it is refereed by other driver. Thank you.
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
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 s/Registe/Register/ available on i.MX6 SoCs in the Secure Non-Volatile Storage. Signed-off-by: Oleksij Rempel Cc: Srinivas Kandagatla Cc: Maxime Ripard Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt new file mode 100644 index ..9a8be1a2d12e --- /dev/null +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt @@ -0,0 +1,15 @@ +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D +Secure Non-Volatile Storage. + +Required properties: +- compatible: should be one of + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). +- offset: Should contain the offset relative to syscon parrent node. typo ok. +- regmap: Should contain a phandle pointing to syscon. + +Example: + snvs_lpgpr: snvs-lpgpr { + compatible = "fsl,imx6q-snvs-lpgpr"; + regmap = <>; + offset = <0x68>; Why does this need to be in DT? Is something going to refer to this node? If not, the node should be enough information for the OS. Jes, it is refereed by other driver. Thank you.
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Thu, Apr 06, 2017 at 09:31:07AM +0200, Oleksij Rempel wrote: > Documenation bindings for the Low Power General Purpose Registe s/Registe/Register/ > available on i.MX6 SoCs in the Secure Non-Volatile Storage. > > Signed-off-by: Oleksij Rempel> Cc: Srinivas Kandagatla > Cc: Maxime Ripard > Cc: Rob Herring > Cc: Mark Rutland > Cc: devicet...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ > 1 file changed, 15 insertions(+) > create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > > diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > new file mode 100644 > index ..9a8be1a2d12e > --- /dev/null > +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > @@ -0,0 +1,15 @@ > +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D > +Secure Non-Volatile Storage. > + > +Required properties: > +- compatible: should be one of > + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). > +- offset: Should contain the offset relative to syscon parrent node. typo +- regmap: Should contain a phandle pointing to syscon. > + > +Example: > + snvs_lpgpr: snvs-lpgpr { > + compatible = "fsl,imx6q-snvs-lpgpr"; > + regmap = <>; > + offset = <0x68>; Why does this need to be in DT? Is something going to refer to this node? If not, the node should be enough information for the OS. Rob
Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
On Thu, Apr 06, 2017 at 09:31:07AM +0200, Oleksij Rempel wrote: > Documenation bindings for the Low Power General Purpose Registe s/Registe/Register/ > available on i.MX6 SoCs in the Secure Non-Volatile Storage. > > Signed-off-by: Oleksij Rempel > Cc: Srinivas Kandagatla > Cc: Maxime Ripard > Cc: Rob Herring > Cc: Mark Rutland > Cc: devicet...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ > 1 file changed, 15 insertions(+) > create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > > diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > new file mode 100644 > index ..9a8be1a2d12e > --- /dev/null > +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt > @@ -0,0 +1,15 @@ > +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D > +Secure Non-Volatile Storage. > + > +Required properties: > +- compatible: should be one of > + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). > +- offset: Should contain the offset relative to syscon parrent node. typo +- regmap: Should contain a phandle pointing to syscon. > + > +Example: > + snvs_lpgpr: snvs-lpgpr { > + compatible = "fsl,imx6q-snvs-lpgpr"; > + regmap = <>; > + offset = <0x68>; Why does this need to be in DT? Is something going to refer to this node? If not, the node should be enough information for the OS. Rob
[PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
Documenation bindings for the Low Power General Purpose Registe available on i.MX6 SoCs in the Secure Non-Volatile Storage. Signed-off-by: Oleksij RempelCc: Srinivas Kandagatla Cc: Maxime Ripard Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt new file mode 100644 index ..9a8be1a2d12e --- /dev/null +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt @@ -0,0 +1,15 @@ +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D +Secure Non-Volatile Storage. + +Required properties: +- compatible: should be one of + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). +- offset: Should contain the offset relative to syscon parrent node. +- regmap: Should contain a phandle pointing to syscon. + +Example: + snvs_lpgpr: snvs-lpgpr { + compatible = "fsl,imx6q-snvs-lpgpr"; + regmap = <>; + offset = <0x68>; + }; -- 2.11.0
[PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding
Documenation bindings for the Low Power General Purpose Registe available on i.MX6 SoCs in the Secure Non-Volatile Storage. Signed-off-by: Oleksij Rempel Cc: Srinivas Kandagatla Cc: Maxime Ripard Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt diff --git a/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt new file mode 100644 index ..9a8be1a2d12e --- /dev/null +++ b/Documentation/devicetree/bindings/nvmem/snvs-lpgpr.txt @@ -0,0 +1,15 @@ +Device tree bindings for Low Power General Purpose Registe found in i.MX6Q/D +Secure Non-Volatile Storage. + +Required properties: +- compatible: should be one of + "fsl,imx6q-snvs-lpgpr" (i.MX6Q/D/DL/S). +- offset: Should contain the offset relative to syscon parrent node. +- regmap: Should contain a phandle pointing to syscon. + +Example: + snvs_lpgpr: snvs-lpgpr { + compatible = "fsl,imx6q-snvs-lpgpr"; + regmap = <>; + offset = <0x68>; + }; -- 2.11.0