diff mbox

[v3,1/4] ARM: docs: add documentation binding for meson watchdog

Message ID 1411158989-8075-2-git-send-email-carlo@caione.org (mailing list archive)
State New, archived
Headers show

Commit Message

Carlo Caione Sept. 19, 2014, 8:36 p.m. UTC
Signed-off-by: Carlo Caione <carlo@caione.org>
---
 Documentation/devicetree/bindings/watchdog/meson6-wdt.txt | 13 +++++++++++++
 1 file changed, 13 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/watchdog/meson6-wdt.txt

Comments

Guenter Roeck Sept. 20, 2014, 3:28 p.m. UTC | #1
On 09/19/2014 01:36 PM, Carlo Caione wrote:
> Signed-off-by: Carlo Caione <carlo@caione.org>
> ---
>   Documentation/devicetree/bindings/watchdog/meson6-wdt.txt | 13 +++++++++++++
>   1 file changed, 13 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>
> diff --git a/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
> new file mode 100644
> index 0000000..9200fc2
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
> @@ -0,0 +1,13 @@
> +Meson SoCs Watchdog timer
> +
> +Required properties:
> +
> +- compatible : should be "amlogic,meson6-wdt"
> +- reg : Specifies base physical address and size of the registers.
> +

The 'amlogic' vendor prefix is undefined.
It will have to be added to Documentation/devicetree/bindings/vendor-prefixes.txt.

Guenter
Carlo Caione Sept. 20, 2014, 4:08 p.m. UTC | #2
On Sat, Sep 20, 2014 at 5:28 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> On 09/19/2014 01:36 PM, Carlo Caione wrote:
>>
>> Signed-off-by: Carlo Caione <carlo@caione.org>
>> ---
>>   Documentation/devicetree/bindings/watchdog/meson6-wdt.txt | 13
>> +++++++++++++
>>   1 file changed, 13 insertions(+)
>>   create mode 100644
>> Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>>
>> diff --git a/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>> b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>> new file mode 100644
>> index 0000000..9200fc2
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>> @@ -0,0 +1,13 @@
>> +Meson SoCs Watchdog timer
>> +
>> +Required properties:
>> +
>> +- compatible : should be "amlogic,meson6-wdt"
>> +- reg : Specifies base physical address and size of the registers.
>> +
>
>
> The 'amlogic' vendor prefix is undefined.
> It will have to be added to
> Documentation/devicetree/bindings/vendor-prefixes.txt.

As stated in the cover, this patchset depends on
http://www.spinics.net/lists/arm-kernel/msg362521.html where the
vendor-prefixes.txt is updated.
Probably I should ping it.
Guenter Roeck Sept. 20, 2014, 4:10 p.m. UTC | #3
On 09/20/2014 09:08 AM, Carlo Caione wrote:
> On Sat, Sep 20, 2014 at 5:28 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>> On 09/19/2014 01:36 PM, Carlo Caione wrote:
>>>
>>> Signed-off-by: Carlo Caione <carlo@caione.org>
>>> ---
>>>    Documentation/devicetree/bindings/watchdog/meson6-wdt.txt | 13
>>> +++++++++++++
>>>    1 file changed, 13 insertions(+)
>>>    create mode 100644
>>> Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>>> b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>>> new file mode 100644
>>> index 0000000..9200fc2
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
>>> @@ -0,0 +1,13 @@
>>> +Meson SoCs Watchdog timer
>>> +
>>> +Required properties:
>>> +
>>> +- compatible : should be "amlogic,meson6-wdt"
>>> +- reg : Specifies base physical address and size of the registers.
>>> +
>>
>>
>> The 'amlogic' vendor prefix is undefined.
>> It will have to be added to
>> Documentation/devicetree/bindings/vendor-prefixes.txt.
>
> As stated in the cover, this patchset depends on
> http://www.spinics.net/lists/arm-kernel/msg362521.html where the
> vendor-prefixes.txt is updated.
> Probably I should ping it.
>
Ah, sorry, I missed that.

Thanks,
Guenter
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
new file mode 100644
index 0000000..9200fc2
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/meson6-wdt.txt
@@ -0,0 +1,13 @@ 
+Meson SoCs Watchdog timer
+
+Required properties:
+
+- compatible : should be "amlogic,meson6-wdt"
+- reg : Specifies base physical address and size of the registers.
+
+Example:
+
+wdt: watchdog@c1109900 {
+	compatible = "amlogic,meson6-wdt";
+	reg = <0xc1109900 0x8>;
+};