Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1621267

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

Show all headers | View raw

On Mon, Apr 10, 2017 at 11:36 PM, Oleksij Rempel <ore@pengutronix.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
>>
>>
>> s/Registe/Register/
>>
>>> available on i.MX6 SoCs in the Secure Non-Volatile Storage.
>>>
>>> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
>>> Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>>> Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
>>> Cc: Rob Herring <robh+dt@kernel.org>
>>> Cc: Mark Rutland <mark.rutland@arm.com>
>>> Cc: devicetree@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 000000000000..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 = <&snvs>;
>>> +               offset = <0x68>;
>>
>>
>> Why does this need to be in DT? Is something going to refer to this
>> node? If not, the &snvs 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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding Rob Herring <robh@kernel.org> - 2017-04-10 20:30 +0200
  Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding Oleksij Rempel <ore@pengutronix.de> - 2017-04-11 06:40 +0200
    Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding Rob Herring <robh@kernel.org> - 2017-04-11 15:40 +0200
      Re: [PATCH v1 3/3] nvmem: dt: document SNVS LPGPR binding Oleksij Rempel <ore@pengutronix.de> - 2017-04-12 08:30 +0200
  [PATCH v2 1/3] nvmem: add snvs_lpgpr driver Oleksij Rempel <o.rempel@pengutronix.de> - 2017-04-11 07:30 +0200
  [PATCH v2 2/3] ARM: dts: imx6qdl.dtsi: add "fsl,imx6q-snvs-lpgpr" node Oleksij Rempel <o.rempel@pengutronix.de> - 2017-04-11 07:30 +0200
  [PATCH v2 3/3] nvmem: dt: document SNVS LPGPR binding Oleksij Rempel <o.rempel@pengutronix.de> - 2017-04-11 07:30 +0200

csiph-web