diff mbox

Documentation/watchdog: Update node name in samsung-wdt example

Message ID 1382941457-2826-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat Oct. 28, 2013, 6:24 a.m. UTC
Update the name as per DT naming convention.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 .../devicetree/bindings/watchdog/samsung-wdt.txt   |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Rutland Oct. 28, 2013, 6:52 a.m. UTC | #1
On Mon, Oct 28, 2013 at 06:24:17AM +0000, Sachin Kamat wrote:
> Update the name as per DT naming convention.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>

Acked-by: Mark Rutland <mark.rutland@arm.com>

> ---
>  .../devicetree/bindings/watchdog/samsung-wdt.txt   |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> index 4c798e3..0642fb8 100644
> --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> @@ -17,7 +17,7 @@ Optional properties:
>  
>  Example:
>  
> -watchdog {
> +watchdog@101D0000 {
>  	compatible = "samsung,s3c2410-wdt";
>  	reg = <0x101D0000 0x100>, <0x10040408 0x4>, <0x1004040c 0x4>;
>  	interrupts = <0 42 0>;
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Guenter Roeck Oct. 28, 2013, 1:28 p.m. UTC | #2
On 10/27/2013 11:24 PM, Sachin Kamat wrote:
> Update the name as per DT naming convention.
>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Guenter Roeck Oct. 28, 2013, 1:37 p.m. UTC | #3
On 10/27/2013 11:24 PM, Sachin Kamat wrote:
> Update the name as per DT naming convention.
>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>   .../devicetree/bindings/watchdog/samsung-wdt.txt   |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> index 4c798e3..0642fb8 100644
> --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
> @@ -17,7 +17,7 @@ Optional properties:
>
>   Example:
>
> -watchdog {
> +watchdog@101D0000 {
>   	compatible = "samsung,s3c2410-wdt";
>   	reg = <0x101D0000 0x100>, <0x10040408 0x4>, <0x1004040c 0x4>;
>   	interrupts = <0 42 0>;
>

Sachin,

It appears that this patch depends on some other patch which is not upstream yet.
It is not in -next either. Can you point to that other patch ?

Thanks,
Guenter



--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sachin Kamat Oct. 30, 2013, 5:15 a.m. UTC | #4
On 28 October 2013 19:07, Guenter Roeck <linux@roeck-us.net> wrote:
> On 10/27/2013 11:24 PM, Sachin Kamat wrote:
>>
>> Update the name as per DT naming convention.
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
>> ---
>>   .../devicetree/bindings/watchdog/samsung-wdt.txt   |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
>> b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
>> index 4c798e3..0642fb8 100644
>> --- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
>> +++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
>> @@ -17,7 +17,7 @@ Optional properties:
>>
>>   Example:
>>
>> -watchdog {
>> +watchdog@101D0000 {
>>         compatible = "samsung,s3c2410-wdt";
>>         reg = <0x101D0000 0x100>, <0x10040408 0x4>, <0x1004040c 0x4>;
>>         interrupts = <0 42 0>;
>>
>
> Sachin,
>
> It appears that this patch depends on some other patch which is not upstream
> yet.

Yes you are right.

> It is not in -next either. Can you point to that other patch ?

This patch was based on a non-upstream patch. I have now posted the
right version.
Please ignore the current one. Sorry for the mess up.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
index 4c798e3..0642fb8 100644
--- a/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
+++ b/Documentation/devicetree/bindings/watchdog/samsung-wdt.txt
@@ -17,7 +17,7 @@  Optional properties:
 
 Example:
 
-watchdog {
+watchdog@101D0000 {
 	compatible = "samsung,s3c2410-wdt";
 	reg = <0x101D0000 0x100>, <0x10040408 0x4>, <0x1004040c 0x4>;
 	interrupts = <0 42 0>;