* [PATCH_V3 0/2] watchdog: jz4740: Add DT support
@ 2015-02-03 10:25 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-03 10:25 UTC (permalink / raw)
To: wim-IQzOog9fTRqzQB+pC5nmwQ
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA,
paul-icTtO2rgO2OTuSrc4Mpeew
Hi,
Here are two simple patches that add DT support to the jz4740 watchdog driver.
Patches are based on 3.19-rc7. Quite disjoint and stay within jz4740
so should apply easily on other trees.
If you would like to have them rebased to a different tree, please tell.
V3 Changed
Added Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
Typo in commit message.
Rebased to rc7
V2 Changes
Renamed binding to jz4740 instead of jz47xx
Removed clk bindings. They can be added later when the clock tree is fixed.
Rather than add bindings now and change later.
Added MODULE_DEVICE_TABLE()
Thank-you
ZubairLK
Zubair Lutfullah Kakakhel (2):
dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
watchdog: jz4740: Add DT support
.../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
2 files changed, 22 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH_V3 1/2] dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
@ 2015-02-03 10:25 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-03 10:25 UTC (permalink / raw)
To: wim; +Cc: devicetree, linux-kernel, linux-watchdog, Zubair.Kakakhel, paul
Add binding for jz4740 watchdog timer. It is a simple watchdog timer.
Signed-off-by: Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
---
The jz4740 is platform only at the moment.
But DT support is being added
See http://patchwork.linux-mips.org/bundle/paulburton/ci20-v3.20/
V3: Added Reviewed-by: Guenter Roeck <linux@roeck-us.net>
V2 Changes
Removed clock binding because of pending work in clock tree. Will add
binding later. Rather than introduce a bad binding now and change later.
Renamed to jz4740 instead of jz47xx.
---
.../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
1 file changed, 12 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
diff --git a/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt b/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
new file mode 100644
index 0000000..e27763e
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
@@ -0,0 +1,12 @@
+Ingenic Watchdog Timer (WDT) Controller for JZ4740
+
+Required properties:
+compatible: "ingenic,jz4740-watchdog"
+reg: Register address and length for watchdog registers
+
+Example:
+
+watchdog: jz4740-watchdog@0x10002000 {
+ compatible = "ingenic,jz4740-watchdog";
+ reg = <0x10002000 0x100>;
+};
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH_V3 1/2] dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
@ 2015-02-03 10:25 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-03 10:25 UTC (permalink / raw)
To: wim-IQzOog9fTRqzQB+pC5nmwQ
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA,
paul-icTtO2rgO2OTuSrc4Mpeew
Add binding for jz4740 watchdog timer. It is a simple watchdog timer.
Signed-off-by: Zubair Lutfullah Kakakhel <Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
---
The jz4740 is platform only at the moment.
But DT support is being added
See http://patchwork.linux-mips.org/bundle/paulburton/ci20-v3.20/
V3: Added Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
V2 Changes
Removed clock binding because of pending work in clock tree. Will add
binding later. Rather than introduce a bad binding now and change later.
Renamed to jz4740 instead of jz47xx.
---
.../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
1 file changed, 12 insertions(+)
create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
diff --git a/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt b/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
new file mode 100644
index 0000000..e27763e
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
@@ -0,0 +1,12 @@
+Ingenic Watchdog Timer (WDT) Controller for JZ4740
+
+Required properties:
+compatible: "ingenic,jz4740-watchdog"
+reg: Register address and length for watchdog registers
+
+Example:
+
+watchdog: jz4740-watchdog@0x10002000 {
+ compatible = "ingenic,jz4740-watchdog";
+ reg = <0x10002000 0x100>;
+};
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH_V3 2/2] watchdog: jz4740: Add DT support
@ 2015-02-03 10:25 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-03 10:25 UTC (permalink / raw)
To: wim; +Cc: devicetree, linux-kernel, linux-watchdog, Zubair.Kakakhel, paul
Add DT support to the jz4740 driver. Simple of_match_ptr. No other
modification for probe needed
Signed-off-by: Zubair Lutfullah Kakakhel <Zubair.Kakakhel@imgtec.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
---
V3 Typo in Reviewed-by: Guenter Roeck <linux@roeck-us.net>
V2 changes
Add module device table. Even though we are moving to non-dt for jz4740.
Lets try and not break things while at it.
Renamed jz4740_of_match to jz4740_wdt_of_match
---
drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/watchdog/jz4740_wdt.c b/drivers/watchdog/jz4740_wdt.c
index 18e41af..4c2cc09 100644
--- a/drivers/watchdog/jz4740_wdt.c
+++ b/drivers/watchdog/jz4740_wdt.c
@@ -24,6 +24,7 @@
#include <linux/clk.h>
#include <linux/slab.h>
#include <linux/err.h>
+#include <linux/of.h>
#include <asm/mach-jz4740/timer.h>
@@ -142,6 +143,14 @@ static const struct watchdog_ops jz4740_wdt_ops = {
.set_timeout = jz4740_wdt_set_timeout,
};
+#ifdef CONFIG_OF
+static const struct of_device_id jz4740_wdt_of_matches[] = {
+ { .compatible = "ingenic,jz4740-watchdog", },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, jz4740_wdt_of_matches)
+#endif
+
static int jz4740_wdt_probe(struct platform_device *pdev)
{
struct jz4740_wdt_drvdata *drvdata;
@@ -211,6 +220,7 @@ static struct platform_driver jz4740_wdt_driver = {
.remove = jz4740_wdt_remove,
.driver = {
.name = "jz4740-wdt",
+ .of_match_table = of_match_ptr(jz4740_wdt_of_matches),
},
};
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH_V3 2/2] watchdog: jz4740: Add DT support
@ 2015-02-03 10:25 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-03 10:25 UTC (permalink / raw)
To: wim-IQzOog9fTRqzQB+pC5nmwQ
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA,
paul-icTtO2rgO2OTuSrc4Mpeew
Add DT support to the jz4740 driver. Simple of_match_ptr. No other
modification for probe needed
Signed-off-by: Zubair Lutfullah Kakakhel <Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
---
V3 Typo in Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
V2 changes
Add module device table. Even though we are moving to non-dt for jz4740.
Lets try and not break things while at it.
Renamed jz4740_of_match to jz4740_wdt_of_match
---
drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/watchdog/jz4740_wdt.c b/drivers/watchdog/jz4740_wdt.c
index 18e41af..4c2cc09 100644
--- a/drivers/watchdog/jz4740_wdt.c
+++ b/drivers/watchdog/jz4740_wdt.c
@@ -24,6 +24,7 @@
#include <linux/clk.h>
#include <linux/slab.h>
#include <linux/err.h>
+#include <linux/of.h>
#include <asm/mach-jz4740/timer.h>
@@ -142,6 +143,14 @@ static const struct watchdog_ops jz4740_wdt_ops = {
.set_timeout = jz4740_wdt_set_timeout,
};
+#ifdef CONFIG_OF
+static const struct of_device_id jz4740_wdt_of_matches[] = {
+ { .compatible = "ingenic,jz4740-watchdog", },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, jz4740_wdt_of_matches)
+#endif
+
static int jz4740_wdt_probe(struct platform_device *pdev)
{
struct jz4740_wdt_drvdata *drvdata;
@@ -211,6 +220,7 @@ static struct platform_driver jz4740_wdt_driver = {
.remove = jz4740_wdt_remove,
.driver = {
.name = "jz4740-wdt",
+ .of_match_table = of_match_ptr(jz4740_wdt_of_matches),
},
};
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support
@ 2015-02-03 19:04 ` Wim Van Sebroeck
0 siblings, 0 replies; 10+ messages in thread
From: Wim Van Sebroeck @ 2015-02-03 19:04 UTC (permalink / raw)
To: Zubair Lutfullah Kakakhel; +Cc: devicetree, linux-kernel, linux-watchdog, paul
Hi Zubair,
> Here are two simple patches that add DT support to the jz4740 watchdog driver.
>
> Patches are based on 3.19-rc7. Quite disjoint and stay within jz4740
> so should apply easily on other trees.
>
> If you would like to have them rebased to a different tree, please tell.
>
> V3 Changed
> Added Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> Typo in commit message.
> Rebased to rc7
>
> V2 Changes
> Renamed binding to jz4740 instead of jz47xx
> Removed clk bindings. They can be added later when the clock tree is fixed.
> Rather than add bindings now and change later.
>
> Added MODULE_DEVICE_TABLE()
>
> Thank-you
>
> ZubairLK
>
> Zubair Lutfullah Kakakhel (2):
> dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
> watchdog: jz4740: Add DT support
>
> .../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
> drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
> 2 files changed, 22 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
Both patches have been added to linux-watchdog-next.
Kind regards,
Wim.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support
@ 2015-02-03 19:04 ` Wim Van Sebroeck
0 siblings, 0 replies; 10+ messages in thread
From: Wim Van Sebroeck @ 2015-02-03 19:04 UTC (permalink / raw)
To: Zubair Lutfullah Kakakhel
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
paul-icTtO2rgO2OTuSrc4Mpeew
Hi Zubair,
> Here are two simple patches that add DT support to the jz4740 watchdog driver.
>
> Patches are based on 3.19-rc7. Quite disjoint and stay within jz4740
> so should apply easily on other trees.
>
> If you would like to have them rebased to a different tree, please tell.
>
> V3 Changed
> Added Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
> Typo in commit message.
> Rebased to rc7
>
> V2 Changes
> Renamed binding to jz4740 instead of jz47xx
> Removed clk bindings. They can be added later when the clock tree is fixed.
> Rather than add bindings now and change later.
>
> Added MODULE_DEVICE_TABLE()
>
> Thank-you
>
> ZubairLK
>
> Zubair Lutfullah Kakakhel (2):
> dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
> watchdog: jz4740: Add DT support
>
> .../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
> drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
> 2 files changed, 22 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
Both patches have been added to linux-watchdog-next.
Kind regards,
Wim.
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support
2015-02-03 19:04 ` Wim Van Sebroeck
@ 2015-02-05 8:54 ` Zubair Lutfullah Kakakhel
-1 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-05 8:54 UTC (permalink / raw)
To: Wim Van Sebroeck; +Cc: devicetree, linux-kernel, linux-watchdog, paul
On 03/02/15 19:04, Wim Van Sebroeck wrote:
> Hi Zubair,
...
>> Zubair Lutfullah Kakakhel (2):
>> dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
>> watchdog: jz4740: Add DT support
>>
>> .../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
>> drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
>> 2 files changed, 22 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
>
> Both patches have been added to linux-watchdog-next.
>
Thank-you very much.
Regards
ZubairLK
> Kind regards,
> Wim.
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support
@ 2015-02-05 8:54 ` Zubair Lutfullah Kakakhel
0 siblings, 0 replies; 10+ messages in thread
From: Zubair Lutfullah Kakakhel @ 2015-02-05 8:54 UTC (permalink / raw)
To: Wim Van Sebroeck; +Cc: devicetree, linux-kernel, linux-watchdog, paul
On 03/02/15 19:04, Wim Van Sebroeck wrote:
> Hi Zubair,
...
>> Zubair Lutfullah Kakakhel (2):
>> dt: watchdog: Add DT binding documentation for jz4740 watchdog timer
>> watchdog: jz4740: Add DT support
>>
>> .../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 12 ++++++++++++
>> drivers/watchdog/jz4740_wdt.c | 10 ++++++++++
>> 2 files changed, 22 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt
>
> Both patches have been added to linux-watchdog-next.
>
Thank-you very much.
Regards
ZubairLK
> Kind regards,
> Wim.
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread