All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Documentation/watchdog: Update node name in samsung-wdt example
@ 2013-10-28  6:24 Sachin Kamat
  2013-10-28  6:52 ` Mark Rutland
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Sachin Kamat @ 2013-10-28  6:24 UTC (permalink / raw)
  To: linux-doc
  Cc: devicetree, linux-watchdog, wim, rob, linux-samsung-soc,
	kgene.kim, sachin.kamat

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>;
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] Documentation/watchdog: Update node name in samsung-wdt example
  2013-10-28  6:24 [PATCH] Documentation/watchdog: Update node name in samsung-wdt example Sachin Kamat
@ 2013-10-28  6:52 ` Mark Rutland
  2013-10-28 13:28 ` Guenter Roeck
  2013-10-28 13:37 ` Guenter Roeck
  2 siblings, 0 replies; 5+ messages in thread
From: Mark Rutland @ 2013-10-28  6:52 UTC (permalink / raw)
  To: Sachin Kamat
  Cc: linux-doc, devicetree, linux-watchdog, wim, rob,
	linux-samsung-soc, kgene.kim

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
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Documentation/watchdog: Update node name in samsung-wdt example
  2013-10-28  6:24 [PATCH] Documentation/watchdog: Update node name in samsung-wdt example Sachin Kamat
  2013-10-28  6:52 ` Mark Rutland
@ 2013-10-28 13:28 ` Guenter Roeck
  2013-10-28 13:37 ` Guenter Roeck
  2 siblings, 0 replies; 5+ messages in thread
From: Guenter Roeck @ 2013-10-28 13:28 UTC (permalink / raw)
  To: Sachin Kamat, linux-doc
  Cc: devicetree, linux-watchdog, wim, rob, linux-samsung-soc, kgene.kim

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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Documentation/watchdog: Update node name in samsung-wdt example
  2013-10-28  6:24 [PATCH] Documentation/watchdog: Update node name in samsung-wdt example Sachin Kamat
  2013-10-28  6:52 ` Mark Rutland
  2013-10-28 13:28 ` Guenter Roeck
@ 2013-10-28 13:37 ` Guenter Roeck
  2013-10-30  5:15   ` Sachin Kamat
  2 siblings, 1 reply; 5+ messages in thread
From: Guenter Roeck @ 2013-10-28 13:37 UTC (permalink / raw)
  To: Sachin Kamat, linux-doc
  Cc: devicetree, linux-watchdog, wim, rob, linux-samsung-soc, kgene.kim

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




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Documentation/watchdog: Update node name in samsung-wdt example
  2013-10-28 13:37 ` Guenter Roeck
@ 2013-10-30  5:15   ` Sachin Kamat
  0 siblings, 0 replies; 5+ messages in thread
From: Sachin Kamat @ 2013-10-30  5:15 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: linux-doc, devicetree, linux-watchdog, Wim Van Sebroeck,
	Rob Landley, linux-samsung-soc, Kukjin Kim

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.

-- 
With warm regards,
Sachin

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-10-30  5:15 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-28  6:24 [PATCH] Documentation/watchdog: Update node name in samsung-wdt example Sachin Kamat
2013-10-28  6:52 ` Mark Rutland
2013-10-28 13:28 ` Guenter Roeck
2013-10-28 13:37 ` Guenter Roeck
2013-10-30  5:15   ` Sachin Kamat

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.