lkml.org 
[lkml]   [2013]   [Dec]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH V2 1/2] pinctrl: tegra: Add devicetree binding document for Tegra124
On 12/09/2013 10:48 AM, Mark Rutland wrote:
> On Mon, Dec 09, 2013 at 05:39:42PM +0000, Stephen Warren wrote:
>> On 12/09/2013 03:51 AM, Mark Rutland wrote:
>>> On Mon, Dec 09, 2013 at 10:32:19AM +0000, Laxman Dewangan wrote:
>>>> This device tree binding document describes the Tegra124 pincontrol
>>>> DT bindings. This document lists all valid properties, names, mux
>>>> options of Tegra124 pins.
>>>>
>>>> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
>>>> ---
>>>> Changes from V1:
>>>> - Referred the dt-binding header file on describing the nodes.
>>>>
>>>> .../bindings/pinctrl/nvidia,tegra124-pinmux.txt | 147 ++++++++++++++++++++
>>>> 1 files changed, 147 insertions(+), 0 deletions(-)
>>>> create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.txt
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.txt b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.txt
>>>> new file mode 100644
>>>> index 0000000..12ef772
>>>> --- /dev/null
>>>> +++ b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.txt
>>>> @@ -0,0 +1,147 @@
>>>> +NVIDIA Tegra124 pinmux controller
>>>> +
>>>> +The Tegra124 pinctrl binding is very similar to the Tegra20 and Tegra30
>>>> +pinctrl binding, as described in nvidia,tegra20-pinmux.txt and
>>>> +nvidia,tegra30-pinmux.txt. In fact, this document assumes that binding as
>>>> +a baseline, and only documents the differences between the two bindings.
>>>> +
>>>> +Required properties:
>>>> +- compatible: "nvidia,tegra124-pinmux"
>>>> +- reg: Should contain the register physical address and length for each of
>>>> + the pad control and mux registers. The first bank of address must be the
>>>> + driver strength pad control register address and second bank address must
>>>> + be pinmux register address.
>>>
>>> The wording here could be improved. The first sentence implies an entry
>>> for each individual register but I assume these are actually banks of
>>> registers (the sizes in the exanple imply this). The second sentence is
>>> more sepcific.
>>>
>>> How about something like:
>>>
>>> reg: Should contain a list of base address and size pairs for:
>>> * first entry - the driver strength and pad control registers
>>
>> If this patch gets revised, please s/driver/drive/ here.
>
> Whoops. I'm too used to typing the former.

Don't worry - I pointed it out because the same typo was in the original
patch, so I assumed that's where it came from.


\
 
 \ /
  Last update: 2013-12-09 19:41    [W:0.083 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site