* [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies
@ 2023-02-14 10:39 Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
` (4 more replies)
0 siblings, 5 replies; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips
Hi all,
This series make an update in the MT7621 SoC's watchdog driver code. This
SoC already provides a system controller node to access reset status
register needed for the watchdog. Instead of using MIPS architecture
dependent operations in header 'asm/mach-ralink/ralink_regs.h' use
a phandle to the system controller node and use it through regmap APIs
from the code. Driver is also using some globals that are not needed at
all if a driver data structure is used along the code. Hence, add all
new needed stuff inside a new driver data structure. With this changes
driver can be properly compile tested.
Thanks in advance for reviewing this!
v1 of this series here [0].
v2 of this series here [1].
v3 of this series here [2].
v4 of this series here [3].
v5 of this series here [4].
v6 of this series here [5].
Changes in v7:
- Restore phandler 'mediatek,sysctl' in bindings with Krzysztof's 'Acked-by'.
- Use regmap syscon 'syscon_regmap_lookup_by_phandle()' and only if this
fails use 'syscon_regmap_lookup_by_compatible()' to avoid DTB's ABI breakage.
Changes in v6:
- Get regmap syscon using 'syscon_regmap_lookup_by_compatible()'
- Add COMPILE_TEST and dependencies to Kconfig.
- Collect Philippe Mathieu-Daudé 'Reviewed-by' tag for watchdog node
warning fix.
Changes in v5:
- Drop patches related with device tree ABI breakage and only
maintain the rest.
- Collect Arinc 'Reviewed-by' tag for watchdog node warning fix.
Changes in v4:
- Add a patch to fix a watchdog node warning with 'make dtbs_check'
because of a wrong node name.
- Collect Guenter 'Reviewed-by' tags for watchdog driver code.
- Add a missing 'COMPILE_TEST' to Kconfig which was lost when driver
code was split in two patches in v2.
Changes in v3:
- rename phandler from 'ralink,sysctl' into 'mediatek,sysctl'.
- Drop error message added in PATCH 3 that modifies functionality
and we only want to maintain current functionaloty by now.
Changes in v2:
- Remove no needed compatible 'syscon' from bindings.
- Rewrite new syscon phandle description in bindings.
- Remove 'syscon' from compatible in 'mt7621.dtsi'.
- Split PATCH 3 into two different patches:
- PATCH 3: removes static globals using a driver data structure.
- PATCH 4: remove ralink architecture dependent includes and code.
Best regards,
Sergio Paracuellos
[0]: https://lore.kernel.org/linux-watchdog/20230210065621.598120-1-sergio.paracuellos@gmail.com/T/#t
[1]: https://lore.kernel.org/linux-watchdog/20230210121735.639089-1-sergio.paracuellos@gmail.com/T/#t
[2]: https://lore.kernel.org/linux-watchdog/20230210173841.705783-1-sergio.paracuellos@gmail.com/T/#t
[3]: https://lore.kernel.org/linux-watchdog/20230211073357.755893-1-sergio.paracuellos@gmail.com/T/#t
[4]: https://lore.kernel.org/linux-watchdog/20230213120638.850612-1-sergio.paracuellos@gmail.com/T/#t
[5]: https://lore.kernel.org/linux-watchdog/b302c25e-5365-c335-c790-71b3135847d0@arinc9.com/T/#t
Sergio Paracuellos (5):
dt-bindings: watchdog: mt7621-wdt: add phandle to access system
controller registers
mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into
'watchdog'
mips: dts: ralink: mt7621: add phandle to system controller node for
watchdog
watchdog: mt7621-wdt: avoid static global declarations
watchdog: mt7621-wdt: avoid ralink architecture dependent code
.../watchdog/mediatek,mt7621-wdt.yaml | 7 +
arch/mips/boot/dts/ralink/mt7621.dtsi | 3 +-
drivers/watchdog/Kconfig | 4 +-
drivers/watchdog/mt7621_wdt.c | 122 ++++++++++++------
4 files changed, 93 insertions(+), 43 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
@ 2023-02-14 10:39 ` Sergio Paracuellos
2023-02-14 15:10 ` Guenter Roeck
2023-02-14 10:39 ` [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog' Sergio Paracuellos
` (3 subsequent siblings)
4 siblings, 1 reply; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips, Krzysztof Kozlowski
MT7621 SoC provides a system controller node for accessing to some registers.
Add a phandle in this node to avoid using MIPS related arch operations and
includes in watchdog driver code.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
.../devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
index b2b17fdf4e39..a668d0c2f14b 100644
--- a/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
@@ -19,6 +19,12 @@ properties:
reg:
maxItems: 1
+ mediatek,sysctl:
+ $ref: /schemas/types.yaml#/definitions/phandle
+ description:
+ phandle to system controller 'sysc' syscon node which
+ controls system registers
+
required:
- compatible
- reg
@@ -30,4 +36,5 @@ examples:
watchdog@100 {
compatible = "mediatek,mt7621-wdt";
reg = <0x100 0x100>;
+ mediatek,sysctl = <&sysc>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
@ 2023-02-14 10:39 ` Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
2023-02-28 22:15 ` Thomas Bogendoerfer
2023-02-14 10:39 ` [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
` (2 subsequent siblings)
4 siblings, 2 replies; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips, Philippe Mathieu-Daudé
Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
is performed the following warning appears:
wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
Fix this warning up properly renaming the node into 'watchdog'.
Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
arch/mips/boot/dts/ralink/mt7621.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
index 5ca40fd21662..ac818fd721ae 100644
--- a/arch/mips/boot/dts/ralink/mt7621.dtsi
+++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
@@ -70,7 +70,7 @@ sysc: syscon@0 {
"250m", "270m";
};
- wdt: wdt@100 {
+ wdt: watchdog@100 {
compatible = "mediatek,mt7621-wdt";
reg = <0x100 0x100>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog' Sergio Paracuellos
@ 2023-02-14 10:39 ` Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
2023-02-28 22:15 ` Thomas Bogendoerfer
2023-02-14 10:39 ` [PATCH v7 4/5] watchdog: mt7621-wdt: avoid static global declarations Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code Sergio Paracuellos
4 siblings, 2 replies; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips
To allow to access system controller registers from watchdog driver code
add a phandle in the watchdog 'wdt' node. This avoid using arch dependent
operations in driver code.
Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
arch/mips/boot/dts/ralink/mt7621.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
index ac818fd721ae..3d16beb77440 100644
--- a/arch/mips/boot/dts/ralink/mt7621.dtsi
+++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
@@ -73,6 +73,7 @@ sysc: syscon@0 {
wdt: watchdog@100 {
compatible = "mediatek,mt7621-wdt";
reg = <0x100 0x100>;
+ mediatek,sysctl = <&sysc>;
};
gpio: gpio@600 {
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v7 4/5] watchdog: mt7621-wdt: avoid static global declarations
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
` (2 preceding siblings ...)
2023-02-14 10:39 ` [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
@ 2023-02-14 10:39 ` Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code Sergio Paracuellos
4 siblings, 0 replies; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips
Instead of using static global definitions in driver code, refactor code
introducing a new watchdog driver data structure and use it along the
code.
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
drivers/watchdog/mt7621_wdt.c | 102 ++++++++++++++++++++++------------
1 file changed, 65 insertions(+), 37 deletions(-)
diff --git a/drivers/watchdog/mt7621_wdt.c b/drivers/watchdog/mt7621_wdt.c
index a8aa3522cfda..40fb2c9ba9d9 100644
--- a/drivers/watchdog/mt7621_wdt.c
+++ b/drivers/watchdog/mt7621_wdt.c
@@ -31,8 +31,11 @@
#define TMR1CTL_RESTART BIT(9)
#define TMR1CTL_PRESCALE_SHIFT 16
-static void __iomem *mt7621_wdt_base;
-static struct reset_control *mt7621_wdt_reset;
+struct mt7621_wdt_data {
+ void __iomem *base;
+ struct reset_control *rst;
+ struct watchdog_device wdt;
+};
static bool nowayout = WATCHDOG_NOWAYOUT;
module_param(nowayout, bool, 0);
@@ -40,27 +43,31 @@ MODULE_PARM_DESC(nowayout,
"Watchdog cannot be stopped once started (default="
__MODULE_STRING(WATCHDOG_NOWAYOUT) ")");
-static inline void rt_wdt_w32(unsigned reg, u32 val)
+static inline void rt_wdt_w32(void __iomem *base, unsigned reg, u32 val)
{
- iowrite32(val, mt7621_wdt_base + reg);
+ iowrite32(val, base + reg);
}
-static inline u32 rt_wdt_r32(unsigned reg)
+static inline u32 rt_wdt_r32(void __iomem *base, unsigned reg)
{
- return ioread32(mt7621_wdt_base + reg);
+ return ioread32(base + reg);
}
static int mt7621_wdt_ping(struct watchdog_device *w)
{
- rt_wdt_w32(TIMER_REG_TMRSTAT, TMR1CTL_RESTART);
+ struct mt7621_wdt_data *drvdata = watchdog_get_drvdata(w);
+
+ rt_wdt_w32(drvdata->base, TIMER_REG_TMRSTAT, TMR1CTL_RESTART);
return 0;
}
static int mt7621_wdt_set_timeout(struct watchdog_device *w, unsigned int t)
{
+ struct mt7621_wdt_data *drvdata = watchdog_get_drvdata(w);
+
w->timeout = t;
- rt_wdt_w32(TIMER_REG_TMR1LOAD, t * 1000);
+ rt_wdt_w32(drvdata->base, TIMER_REG_TMR1LOAD, t * 1000);
mt7621_wdt_ping(w);
return 0;
@@ -68,29 +75,31 @@ static int mt7621_wdt_set_timeout(struct watchdog_device *w, unsigned int t)
static int mt7621_wdt_start(struct watchdog_device *w)
{
+ struct mt7621_wdt_data *drvdata = watchdog_get_drvdata(w);
u32 t;
/* set the prescaler to 1ms == 1000us */
- rt_wdt_w32(TIMER_REG_TMR1CTL, 1000 << TMR1CTL_PRESCALE_SHIFT);
+ rt_wdt_w32(drvdata->base, TIMER_REG_TMR1CTL, 1000 << TMR1CTL_PRESCALE_SHIFT);
mt7621_wdt_set_timeout(w, w->timeout);
- t = rt_wdt_r32(TIMER_REG_TMR1CTL);
+ t = rt_wdt_r32(drvdata->base, TIMER_REG_TMR1CTL);
t |= TMR1CTL_ENABLE;
- rt_wdt_w32(TIMER_REG_TMR1CTL, t);
+ rt_wdt_w32(drvdata->base, TIMER_REG_TMR1CTL, t);
return 0;
}
static int mt7621_wdt_stop(struct watchdog_device *w)
{
+ struct mt7621_wdt_data *drvdata = watchdog_get_drvdata(w);
u32 t;
mt7621_wdt_ping(w);
- t = rt_wdt_r32(TIMER_REG_TMR1CTL);
+ t = rt_wdt_r32(drvdata->base, TIMER_REG_TMR1CTL);
t &= ~TMR1CTL_ENABLE;
- rt_wdt_w32(TIMER_REG_TMR1CTL, t);
+ rt_wdt_w32(drvdata->base, TIMER_REG_TMR1CTL, t);
return 0;
}
@@ -105,7 +114,9 @@ static int mt7621_wdt_bootcause(void)
static int mt7621_wdt_is_running(struct watchdog_device *w)
{
- return !!(rt_wdt_r32(TIMER_REG_TMR1CTL) & TMR1CTL_ENABLE);
+ struct mt7621_wdt_data *drvdata = watchdog_get_drvdata(w);
+
+ return !!(rt_wdt_r32(drvdata->base, TIMER_REG_TMR1CTL) & TMR1CTL_ENABLE);
}
static const struct watchdog_info mt7621_wdt_info = {
@@ -121,30 +132,39 @@ static const struct watchdog_ops mt7621_wdt_ops = {
.set_timeout = mt7621_wdt_set_timeout,
};
-static struct watchdog_device mt7621_wdt_dev = {
- .info = &mt7621_wdt_info,
- .ops = &mt7621_wdt_ops,
- .min_timeout = 1,
- .max_timeout = 0xfffful / 1000,
-};
-
static int mt7621_wdt_probe(struct platform_device *pdev)
{
struct device *dev = &pdev->dev;
- mt7621_wdt_base = devm_platform_ioremap_resource(pdev, 0);
- if (IS_ERR(mt7621_wdt_base))
- return PTR_ERR(mt7621_wdt_base);
+ struct watchdog_device *mt7621_wdt;
+ struct mt7621_wdt_data *drvdata;
+ int err;
+
+ drvdata = devm_kzalloc(dev, sizeof(*drvdata), GFP_KERNEL);
+ if (!drvdata)
+ return -ENOMEM;
- mt7621_wdt_reset = devm_reset_control_get_exclusive(dev, NULL);
- if (!IS_ERR(mt7621_wdt_reset))
- reset_control_deassert(mt7621_wdt_reset);
+ drvdata->base = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(drvdata->base))
+ return PTR_ERR(drvdata->base);
- mt7621_wdt_dev.bootstatus = mt7621_wdt_bootcause();
+ drvdata->rst = devm_reset_control_get_exclusive(dev, NULL);
+ if (!IS_ERR(drvdata->rst))
+ reset_control_deassert(drvdata->rst);
- watchdog_init_timeout(&mt7621_wdt_dev, mt7621_wdt_dev.max_timeout,
- dev);
- watchdog_set_nowayout(&mt7621_wdt_dev, nowayout);
- if (mt7621_wdt_is_running(&mt7621_wdt_dev)) {
+ mt7621_wdt = &drvdata->wdt;
+ mt7621_wdt->info = &mt7621_wdt_info;
+ mt7621_wdt->ops = &mt7621_wdt_ops;
+ mt7621_wdt->min_timeout = 1;
+ mt7621_wdt->max_timeout = 0xfffful / 1000;
+ mt7621_wdt->parent = dev;
+
+ mt7621_wdt->bootstatus = mt7621_wdt_bootcause();
+
+ watchdog_init_timeout(mt7621_wdt, mt7621_wdt->max_timeout, dev);
+ watchdog_set_nowayout(mt7621_wdt, nowayout);
+ watchdog_set_drvdata(mt7621_wdt, drvdata);
+
+ if (mt7621_wdt_is_running(mt7621_wdt)) {
/*
* Make sure to apply timeout from watchdog core, taking
* the prescaler of this driver here into account (the
@@ -154,17 +174,25 @@ static int mt7621_wdt_probe(struct platform_device *pdev)
* we first disable the watchdog, set the new prescaler
* and timeout, and then re-enable the watchdog.
*/
- mt7621_wdt_stop(&mt7621_wdt_dev);
- mt7621_wdt_start(&mt7621_wdt_dev);
- set_bit(WDOG_HW_RUNNING, &mt7621_wdt_dev.status);
+ mt7621_wdt_stop(mt7621_wdt);
+ mt7621_wdt_start(mt7621_wdt);
+ set_bit(WDOG_HW_RUNNING, &mt7621_wdt->status);
}
- return devm_watchdog_register_device(dev, &mt7621_wdt_dev);
+ err = devm_watchdog_register_device(dev, &drvdata->wdt);
+ if (err)
+ return err;
+
+ platform_set_drvdata(pdev, drvdata);
+
+ return 0;
}
static void mt7621_wdt_shutdown(struct platform_device *pdev)
{
- mt7621_wdt_stop(&mt7621_wdt_dev);
+ struct mt7621_wdt_data *drvdata = platform_get_drvdata(pdev);
+
+ mt7621_wdt_stop(&drvdata->wdt);
}
static const struct of_device_id mt7621_wdt_match[] = {
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
` (3 preceding siblings ...)
2023-02-14 10:39 ` [PATCH v7 4/5] watchdog: mt7621-wdt: avoid static global declarations Sergio Paracuellos
@ 2023-02-14 10:39 ` Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
4 siblings, 1 reply; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 10:39 UTC (permalink / raw)
To: linux-watchdog
Cc: wim, linux, robh+dt, krzysztof.kozlowski+dt, matthias.bgg,
arinc.unal, tsbogend, p.zabel, linux-kernel, devicetree,
linux-mips
MT7621 SoC has a system controller node. Watchdog need to access to reset
status register. Ralink architecture and related driver are old and from
the beggining they are using some architecture dependent operations for
accessing this shared registers through 'asm/mach-ralink/ralink_regs.h'
header file. However this is not ideal from a driver perspective which can
just access to the system controller registers in an arch independent way
using regmap syscon APIs. Update Kconfig accordingly to select new added
dependencies and allow driver to be compile tested.
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
drivers/watchdog/Kconfig | 4 +++-
drivers/watchdog/mt7621_wdt.c | 22 +++++++++++++++++-----
2 files changed, 20 insertions(+), 6 deletions(-)
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index b64bc49c7f30..cf752ad64392 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -1865,7 +1865,9 @@ config GXP_WATCHDOG
config MT7621_WDT
tristate "Mediatek SoC watchdog"
select WATCHDOG_CORE
- depends on SOC_MT7620 || SOC_MT7621
+ select REGMAP_MMIO
+ select MFD_SYSCON
+ depends on SOC_MT7620 || SOC_MT7621 || COMPILE_TEST
help
Hardware driver for the Mediatek/Ralink MT7621/8 SoC Watchdog Timer.
diff --git a/drivers/watchdog/mt7621_wdt.c b/drivers/watchdog/mt7621_wdt.c
index 40fb2c9ba9d9..f37d8298a480 100644
--- a/drivers/watchdog/mt7621_wdt.c
+++ b/drivers/watchdog/mt7621_wdt.c
@@ -15,8 +15,8 @@
#include <linux/moduleparam.h>
#include <linux/platform_device.h>
#include <linux/mod_devicetable.h>
-
-#include <asm/mach-ralink/ralink_regs.h>
+#include <linux/mfd/syscon.h>
+#include <linux/regmap.h>
#define SYSC_RSTSTAT 0x38
#define WDT_RST_CAUSE BIT(1)
@@ -34,6 +34,7 @@
struct mt7621_wdt_data {
void __iomem *base;
struct reset_control *rst;
+ struct regmap *sysc;
struct watchdog_device wdt;
};
@@ -104,9 +105,12 @@ static int mt7621_wdt_stop(struct watchdog_device *w)
return 0;
}
-static int mt7621_wdt_bootcause(void)
+static int mt7621_wdt_bootcause(struct mt7621_wdt_data *d)
{
- if (rt_sysc_r32(SYSC_RSTSTAT) & WDT_RST_CAUSE)
+ u32 val;
+
+ regmap_read(d->sysc, SYSC_RSTSTAT, &val);
+ if (val & WDT_RST_CAUSE)
return WDIOF_CARDRESET;
return 0;
@@ -134,6 +138,7 @@ static const struct watchdog_ops mt7621_wdt_ops = {
static int mt7621_wdt_probe(struct platform_device *pdev)
{
+ struct device_node *np = pdev->dev.of_node;
struct device *dev = &pdev->dev;
struct watchdog_device *mt7621_wdt;
struct mt7621_wdt_data *drvdata;
@@ -143,6 +148,13 @@ static int mt7621_wdt_probe(struct platform_device *pdev)
if (!drvdata)
return -ENOMEM;
+ drvdata->sysc = syscon_regmap_lookup_by_phandle(np, "mediatek,sysctl");
+ if (IS_ERR(drvdata->sysc)) {
+ drvdata->sysc = syscon_regmap_lookup_by_compatible("mediatek,mt7621-sysc");
+ if (IS_ERR(drvdata->sysc))
+ return PTR_ERR(drvdata->sysc);
+ }
+
drvdata->base = devm_platform_ioremap_resource(pdev, 0);
if (IS_ERR(drvdata->base))
return PTR_ERR(drvdata->base);
@@ -158,7 +170,7 @@ static int mt7621_wdt_probe(struct platform_device *pdev)
mt7621_wdt->max_timeout = 0xfffful / 1000;
mt7621_wdt->parent = dev;
- mt7621_wdt->bootstatus = mt7621_wdt_bootcause();
+ mt7621_wdt->bootstatus = mt7621_wdt_bootcause(drvdata);
watchdog_init_timeout(mt7621_wdt, mt7621_wdt->max_timeout, dev);
watchdog_set_nowayout(mt7621_wdt, nowayout);
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers
2023-02-14 10:39 ` [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
@ 2023-02-14 15:10 ` Guenter Roeck
0 siblings, 0 replies; 18+ messages in thread
From: Guenter Roeck @ 2023-02-14 15:10 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips, Krzysztof Kozlowski
On Tue, Feb 14, 2023 at 11:39:32AM +0100, Sergio Paracuellos wrote:
> MT7621 SoC provides a system controller node for accessing to some registers.
> Add a phandle in this node to avoid using MIPS related arch operations and
> includes in watchdog driver code.
>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> .../devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
> index b2b17fdf4e39..a668d0c2f14b 100644
> --- a/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
> @@ -19,6 +19,12 @@ properties:
> reg:
> maxItems: 1
>
> + mediatek,sysctl:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description:
> + phandle to system controller 'sysc' syscon node which
> + controls system registers
> +
> required:
> - compatible
> - reg
> @@ -30,4 +36,5 @@ examples:
> watchdog@100 {
> compatible = "mediatek,mt7621-wdt";
> reg = <0x100 0x100>;
> + mediatek,sysctl = <&sysc>;
> };
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-14 10:39 ` [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog' Sergio Paracuellos
@ 2023-02-14 15:11 ` Guenter Roeck
2023-02-14 15:12 ` Sergio Paracuellos
2023-02-28 22:15 ` Thomas Bogendoerfer
1 sibling, 1 reply; 18+ messages in thread
From: Guenter Roeck @ 2023-02-14 15:11 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> is performed the following warning appears:
>
> wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
>
> Fix this warning up properly renaming the node into 'watchdog'.
>
> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Acked-by: Guenter Roeck <linux@roeck-us.net>
Note that we can not apply this and the next patch of the series
through the watchdog tree since it crosses a maintainer boundary.
Guenter
> ---
> arch/mips/boot/dts/ralink/mt7621.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> index 5ca40fd21662..ac818fd721ae 100644
> --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> @@ -70,7 +70,7 @@ sysc: syscon@0 {
> "250m", "270m";
> };
>
> - wdt: wdt@100 {
> + wdt: watchdog@100 {
> compatible = "mediatek,mt7621-wdt";
> reg = <0x100 0x100>;
> };
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog
2023-02-14 10:39 ` [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
@ 2023-02-14 15:11 ` Guenter Roeck
2023-02-28 22:15 ` Thomas Bogendoerfer
1 sibling, 0 replies; 18+ messages in thread
From: Guenter Roeck @ 2023-02-14 15:11 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips
On Tue, Feb 14, 2023 at 11:39:34AM +0100, Sergio Paracuellos wrote:
> To allow to access system controller registers from watchdog driver code
> add a phandle in the watchdog 'wdt' node. This avoid using arch dependent
> operations in driver code.
>
> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Acked-by: Guenter Roeck <linux@roeck-us.net>
> ---
> arch/mips/boot/dts/ralink/mt7621.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> index ac818fd721ae..3d16beb77440 100644
> --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> @@ -73,6 +73,7 @@ sysc: syscon@0 {
> wdt: watchdog@100 {
> compatible = "mediatek,mt7621-wdt";
> reg = <0x100 0x100>;
> + mediatek,sysctl = <&sysc>;
> };
>
> gpio: gpio@600 {
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code
2023-02-14 10:39 ` [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code Sergio Paracuellos
@ 2023-02-14 15:11 ` Guenter Roeck
0 siblings, 0 replies; 18+ messages in thread
From: Guenter Roeck @ 2023-02-14 15:11 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips
On Tue, Feb 14, 2023 at 11:39:36AM +0100, Sergio Paracuellos wrote:
> MT7621 SoC has a system controller node. Watchdog need to access to reset
> status register. Ralink architecture and related driver are old and from
> the beggining they are using some architecture dependent operations for
> accessing this shared registers through 'asm/mach-ralink/ralink_regs.h'
> header file. However this is not ideal from a driver perspective which can
> just access to the system controller registers in an arch independent way
> using regmap syscon APIs. Update Kconfig accordingly to select new added
> dependencies and allow driver to be compile tested.
>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> drivers/watchdog/Kconfig | 4 +++-
> drivers/watchdog/mt7621_wdt.c | 22 +++++++++++++++++-----
> 2 files changed, 20 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index b64bc49c7f30..cf752ad64392 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -1865,7 +1865,9 @@ config GXP_WATCHDOG
> config MT7621_WDT
> tristate "Mediatek SoC watchdog"
> select WATCHDOG_CORE
> - depends on SOC_MT7620 || SOC_MT7621
> + select REGMAP_MMIO
> + select MFD_SYSCON
> + depends on SOC_MT7620 || SOC_MT7621 || COMPILE_TEST
> help
> Hardware driver for the Mediatek/Ralink MT7621/8 SoC Watchdog Timer.
>
> diff --git a/drivers/watchdog/mt7621_wdt.c b/drivers/watchdog/mt7621_wdt.c
> index 40fb2c9ba9d9..f37d8298a480 100644
> --- a/drivers/watchdog/mt7621_wdt.c
> +++ b/drivers/watchdog/mt7621_wdt.c
> @@ -15,8 +15,8 @@
> #include <linux/moduleparam.h>
> #include <linux/platform_device.h>
> #include <linux/mod_devicetable.h>
> -
> -#include <asm/mach-ralink/ralink_regs.h>
> +#include <linux/mfd/syscon.h>
> +#include <linux/regmap.h>
>
> #define SYSC_RSTSTAT 0x38
> #define WDT_RST_CAUSE BIT(1)
> @@ -34,6 +34,7 @@
> struct mt7621_wdt_data {
> void __iomem *base;
> struct reset_control *rst;
> + struct regmap *sysc;
> struct watchdog_device wdt;
> };
>
> @@ -104,9 +105,12 @@ static int mt7621_wdt_stop(struct watchdog_device *w)
> return 0;
> }
>
> -static int mt7621_wdt_bootcause(void)
> +static int mt7621_wdt_bootcause(struct mt7621_wdt_data *d)
> {
> - if (rt_sysc_r32(SYSC_RSTSTAT) & WDT_RST_CAUSE)
> + u32 val;
> +
> + regmap_read(d->sysc, SYSC_RSTSTAT, &val);
> + if (val & WDT_RST_CAUSE)
> return WDIOF_CARDRESET;
>
> return 0;
> @@ -134,6 +138,7 @@ static const struct watchdog_ops mt7621_wdt_ops = {
>
> static int mt7621_wdt_probe(struct platform_device *pdev)
> {
> + struct device_node *np = pdev->dev.of_node;
> struct device *dev = &pdev->dev;
> struct watchdog_device *mt7621_wdt;
> struct mt7621_wdt_data *drvdata;
> @@ -143,6 +148,13 @@ static int mt7621_wdt_probe(struct platform_device *pdev)
> if (!drvdata)
> return -ENOMEM;
>
> + drvdata->sysc = syscon_regmap_lookup_by_phandle(np, "mediatek,sysctl");
> + if (IS_ERR(drvdata->sysc)) {
> + drvdata->sysc = syscon_regmap_lookup_by_compatible("mediatek,mt7621-sysc");
> + if (IS_ERR(drvdata->sysc))
> + return PTR_ERR(drvdata->sysc);
> + }
> +
> drvdata->base = devm_platform_ioremap_resource(pdev, 0);
> if (IS_ERR(drvdata->base))
> return PTR_ERR(drvdata->base);
> @@ -158,7 +170,7 @@ static int mt7621_wdt_probe(struct platform_device *pdev)
> mt7621_wdt->max_timeout = 0xfffful / 1000;
> mt7621_wdt->parent = dev;
>
> - mt7621_wdt->bootstatus = mt7621_wdt_bootcause();
> + mt7621_wdt->bootstatus = mt7621_wdt_bootcause(drvdata);
>
> watchdog_init_timeout(mt7621_wdt, mt7621_wdt->max_timeout, dev);
> watchdog_set_nowayout(mt7621_wdt, nowayout);
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-14 15:11 ` Guenter Roeck
@ 2023-02-14 15:12 ` Sergio Paracuellos
2023-02-28 4:44 ` Sergio Paracuellos
0 siblings, 1 reply; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-14 15:12 UTC (permalink / raw)
To: Guenter Roeck
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
>
> On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> > Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> > is performed the following warning appears:
> >
> > wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
> >
> > Fix this warning up properly renaming the node into 'watchdog'.
> >
> > Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
>
> Acked-by: Guenter Roeck <linux@roeck-us.net>
>
> Note that we can not apply this and the next patch of the series
> through the watchdog tree since it crosses a maintainer boundary.
I was expecting Thomas to get these two arch/mips patches or get an
Acked-by from him in order for you to apply them.
Thanks,
Sergio Paracuellos
>
> Guenter
>
> > ---
> > arch/mips/boot/dts/ralink/mt7621.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> > index 5ca40fd21662..ac818fd721ae 100644
> > --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> > +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> > @@ -70,7 +70,7 @@ sysc: syscon@0 {
> > "250m", "270m";
> > };
> >
> > - wdt: wdt@100 {
> > + wdt: watchdog@100 {
> > compatible = "mediatek,mt7621-wdt";
> > reg = <0x100 0x100>;
> > };
> > --
> > 2.25.1
> >
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-14 15:12 ` Sergio Paracuellos
@ 2023-02-28 4:44 ` Sergio Paracuellos
2023-02-28 8:39 ` Thomas Bogendoerfer
0 siblings, 1 reply; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-28 4:44 UTC (permalink / raw)
To: Guenter Roeck
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, tsbogend, p.zabel, linux-kernel,
devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 14, 2023 at 4:12 PM Sergio Paracuellos
<sergio.paracuellos@gmail.com> wrote:
>
> On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
> >
> > On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> > > Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> > > is performed the following warning appears:
> > >
> > > wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
> > >
> > > Fix this warning up properly renaming the node into 'watchdog'.
> > >
> > > Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> > > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> >
> > Acked-by: Guenter Roeck <linux@roeck-us.net>
> >
> > Note that we can not apply this and the next patch of the series
> > through the watchdog tree since it crosses a maintainer boundary.
>
> I was expecting Thomas to get these two arch/mips patches or get an
> Acked-by from him in order for you to apply them.
Hi Thomas,
I think you have missed this series since you have started to apply
newer stuff in mips-next. Are you ok with taking or Acking patches 2
and 3 of this series?
Thanks,
Sergio Paracuellos
>
> Thanks,
> Sergio Paracuellos
>
> >
> > Guenter
> >
> > > ---
> > > arch/mips/boot/dts/ralink/mt7621.dtsi | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> > > index 5ca40fd21662..ac818fd721ae 100644
> > > --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> > > +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> > > @@ -70,7 +70,7 @@ sysc: syscon@0 {
> > > "250m", "270m";
> > > };
> > >
> > > - wdt: wdt@100 {
> > > + wdt: watchdog@100 {
> > > compatible = "mediatek,mt7621-wdt";
> > > reg = <0x100 0x100>;
> > > };
> > > --
> > > 2.25.1
> > >
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-28 4:44 ` Sergio Paracuellos
@ 2023-02-28 8:39 ` Thomas Bogendoerfer
2023-02-28 8:51 ` Sergio Paracuellos
0 siblings, 1 reply; 18+ messages in thread
From: Thomas Bogendoerfer @ 2023-02-28 8:39 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: Guenter Roeck, linux-watchdog, wim, robh+dt,
krzysztof.kozlowski+dt, matthias.bgg, arinc.unal, p.zabel,
linux-kernel, devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 28, 2023 at 05:44:21AM +0100, Sergio Paracuellos wrote:
> On Tue, Feb 14, 2023 at 4:12 PM Sergio Paracuellos
> <sergio.paracuellos@gmail.com> wrote:
> >
> > On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
> > >
> > > On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> > > > Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> > > > is performed the following warning appears:
> > > >
> > > > wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
> > > >
> > > > Fix this warning up properly renaming the node into 'watchdog'.
> > > >
> > > > Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> > > > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > >
> > > Acked-by: Guenter Roeck <linux@roeck-us.net>
> > >
> > > Note that we can not apply this and the next patch of the series
> > > through the watchdog tree since it crosses a maintainer boundary.
> >
> > I was expecting Thomas to get these two arch/mips patches or get an
> > Acked-by from him in order for you to apply them.
>
> Hi Thomas,
>
> I think you have missed this series since you have started to apply
> newer stuff in mips-next. Are you ok with taking or Acking patches 2
> and 3 of this series?
yes, I sort of missed it. If it's enough to take patch 2/3 I'll do that.
If it's better to keep the series, I'm also ok with acking them.
What's the best way forward ?
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-28 8:39 ` Thomas Bogendoerfer
@ 2023-02-28 8:51 ` Sergio Paracuellos
2023-02-28 14:55 ` Guenter Roeck
0 siblings, 1 reply; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-28 8:51 UTC (permalink / raw)
To: Thomas Bogendoerfer
Cc: Guenter Roeck, linux-watchdog, wim, robh+dt,
krzysztof.kozlowski+dt, matthias.bgg, arinc.unal, p.zabel,
linux-kernel, devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 28, 2023 at 9:41 AM Thomas Bogendoerfer
<tsbogend@alpha.franken.de> wrote:
>
> On Tue, Feb 28, 2023 at 05:44:21AM +0100, Sergio Paracuellos wrote:
> > On Tue, Feb 14, 2023 at 4:12 PM Sergio Paracuellos
> > <sergio.paracuellos@gmail.com> wrote:
> > >
> > > On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
> > > >
> > > > On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> > > > > Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> > > > > is performed the following warning appears:
> > > > >
> > > > > wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
> > > > >
> > > > > Fix this warning up properly renaming the node into 'watchdog'.
> > > > >
> > > > > Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> > > > > Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > > > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > > >
> > > > Acked-by: Guenter Roeck <linux@roeck-us.net>
> > > >
> > > > Note that we can not apply this and the next patch of the series
> > > > through the watchdog tree since it crosses a maintainer boundary.
> > >
> > > I was expecting Thomas to get these two arch/mips patches or get an
> > > Acked-by from him in order for you to apply them.
> >
> > Hi Thomas,
> >
> > I think you have missed this series since you have started to apply
> > newer stuff in mips-next. Are you ok with taking or Acking patches 2
> > and 3 of this series?
>
> yes, I sort of missed it. If it's enough to take patch 2/3 I'll do that.
> If it's better to keep the series, I'm also ok with acking them.
> What's the best way forward ?
Both trees work for me. The rest of the patches of this series should
go through the watchdog tree. Guenter, what is better for you?
Thanks,
Sergio Paracuellos
>
> Thomas.
>
> --
> Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
> good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-28 8:51 ` Sergio Paracuellos
@ 2023-02-28 14:55 ` Guenter Roeck
2023-02-28 15:46 ` Sergio Paracuellos
0 siblings, 1 reply; 18+ messages in thread
From: Guenter Roeck @ 2023-02-28 14:55 UTC (permalink / raw)
To: Sergio Paracuellos, Thomas Bogendoerfer
Cc: linux-watchdog, wim, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, p.zabel, linux-kernel, devicetree,
linux-mips, Philippe Mathieu-Daudé
On 2/28/23 00:51, Sergio Paracuellos wrote:
> On Tue, Feb 28, 2023 at 9:41 AM Thomas Bogendoerfer
> <tsbogend@alpha.franken.de> wrote:
>>
>> On Tue, Feb 28, 2023 at 05:44:21AM +0100, Sergio Paracuellos wrote:
>>> On Tue, Feb 14, 2023 at 4:12 PM Sergio Paracuellos
>>> <sergio.paracuellos@gmail.com> wrote:
>>>>
>>>> On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
>>>>>
>>>>> On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
>>>>>> Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
>>>>>> is performed the following warning appears:
>>>>>>
>>>>>> wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
>>>>>>
>>>>>> Fix this warning up properly renaming the node into 'watchdog'.
>>>>>>
>>>>>> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
>>>>>> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>>>>>> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
>>>>>
>>>>> Acked-by: Guenter Roeck <linux@roeck-us.net>
>>>>>
>>>>> Note that we can not apply this and the next patch of the series
>>>>> through the watchdog tree since it crosses a maintainer boundary.
>>>>
>>>> I was expecting Thomas to get these two arch/mips patches or get an
>>>> Acked-by from him in order for you to apply them.
>>>
>>> Hi Thomas,
>>>
>>> I think you have missed this series since you have started to apply
>>> newer stuff in mips-next. Are you ok with taking or Acking patches 2
>>> and 3 of this series?
>>
>> yes, I sort of missed it. If it's enough to take patch 2/3 I'll do that.
>> If it's better to keep the series, I'm also ok with acking them.
>> What's the best way forward ?
>
> Both trees work for me. The rest of the patches of this series should
> go through the watchdog tree. Guenter, what is better for you?
>
Wim is handling pull requests. He has queued the other three patches in his
tree. I would suggest to apply the two remaining patches through the mips
tree.
Guenter
> Thanks,
> Sergio Paracuellos
>
>>
>> Thomas.
>>
>> --
>> Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
>> good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-28 14:55 ` Guenter Roeck
@ 2023-02-28 15:46 ` Sergio Paracuellos
0 siblings, 0 replies; 18+ messages in thread
From: Sergio Paracuellos @ 2023-02-28 15:46 UTC (permalink / raw)
To: Guenter Roeck
Cc: Thomas Bogendoerfer, linux-watchdog, wim, robh+dt,
krzysztof.kozlowski+dt, matthias.bgg, arinc.unal, p.zabel,
linux-kernel, devicetree, linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 28, 2023 at 3:55 PM Guenter Roeck <linux@roeck-us.net> wrote:
>
> On 2/28/23 00:51, Sergio Paracuellos wrote:
> > On Tue, Feb 28, 2023 at 9:41 AM Thomas Bogendoerfer
> > <tsbogend@alpha.franken.de> wrote:
> >>
> >> On Tue, Feb 28, 2023 at 05:44:21AM +0100, Sergio Paracuellos wrote:
> >>> On Tue, Feb 14, 2023 at 4:12 PM Sergio Paracuellos
> >>> <sergio.paracuellos@gmail.com> wrote:
> >>>>
> >>>> On Tue, Feb 14, 2023 at 4:11 PM Guenter Roeck <linux@roeck-us.net> wrote:
> >>>>>
> >>>>> On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> >>>>>> Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> >>>>>> is performed the following warning appears:
> >>>>>>
> >>>>>> wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
> >>>>>>
> >>>>>> Fix this warning up properly renaming the node into 'watchdog'.
> >>>>>>
> >>>>>> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> >>>>>> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> >>>>>> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> >>>>>
> >>>>> Acked-by: Guenter Roeck <linux@roeck-us.net>
> >>>>>
> >>>>> Note that we can not apply this and the next patch of the series
> >>>>> through the watchdog tree since it crosses a maintainer boundary.
> >>>>
> >>>> I was expecting Thomas to get these two arch/mips patches or get an
> >>>> Acked-by from him in order for you to apply them.
> >>>
> >>> Hi Thomas,
> >>>
> >>> I think you have missed this series since you have started to apply
> >>> newer stuff in mips-next. Are you ok with taking or Acking patches 2
> >>> and 3 of this series?
> >>
> >> yes, I sort of missed it. If it's enough to take patch 2/3 I'll do that.
> >> If it's better to keep the series, I'm also ok with acking them.
> >> What's the best way forward ?
> >
> > Both trees work for me. The rest of the patches of this series should
> > go through the watchdog tree. Guenter, what is better for you?
> >
>
> Wim is handling pull requests. He has queued the other three patches in his
> tree. I would suggest to apply the two remaining patches through the mips
> tree.
Thanks for letting me know, Guenter.
Thomas, please, take the patches 2 and 3 through the mips tree, then.
Thanks,
Sergio Paracuellos
>
> Guenter
>
> > Thanks,
> > Sergio Paracuellos
> >
> >>
> >> Thomas.
> >>
> >> --
> >> Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
> >> good idea. [ RFC1925, 2.3 ]
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog'
2023-02-14 10:39 ` [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog' Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
@ 2023-02-28 22:15 ` Thomas Bogendoerfer
1 sibling, 0 replies; 18+ messages in thread
From: Thomas Bogendoerfer @ 2023-02-28 22:15 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, linux, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, p.zabel, linux-kernel, devicetree,
linux-mips, Philippe Mathieu-Daudé
On Tue, Feb 14, 2023 at 11:39:33AM +0100, Sergio Paracuellos wrote:
> Watchdog nodes must use 'watchdog' for node name. When a 'make dtbs_check'
> is performed the following warning appears:
>
> wdt@100: $nodename:0: 'wdt@100' does not match '^watchdog(@.*|-[0-9a-f])?$'
>
> Fix this warning up properly renaming the node into 'watchdog'.
>
> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> arch/mips/boot/dts/ralink/mt7621.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> index 5ca40fd21662..ac818fd721ae 100644
> --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> @@ -70,7 +70,7 @@ sysc: syscon@0 {
> "250m", "270m";
> };
>
> - wdt: wdt@100 {
> + wdt: watchdog@100 {
> compatible = "mediatek,mt7621-wdt";
> reg = <0x100 0x100>;
> };
> --
> 2.25.1
applied to mips-next.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog
2023-02-14 10:39 ` [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
@ 2023-02-28 22:15 ` Thomas Bogendoerfer
1 sibling, 0 replies; 18+ messages in thread
From: Thomas Bogendoerfer @ 2023-02-28 22:15 UTC (permalink / raw)
To: Sergio Paracuellos
Cc: linux-watchdog, wim, linux, robh+dt, krzysztof.kozlowski+dt,
matthias.bgg, arinc.unal, p.zabel, linux-kernel, devicetree,
linux-mips
On Tue, Feb 14, 2023 at 11:39:34AM +0100, Sergio Paracuellos wrote:
> To allow to access system controller registers from watchdog driver code
> add a phandle in the watchdog 'wdt' node. This avoid using arch dependent
> operations in driver code.
>
> Reviewed-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> arch/mips/boot/dts/ralink/mt7621.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/mips/boot/dts/ralink/mt7621.dtsi b/arch/mips/boot/dts/ralink/mt7621.dtsi
> index ac818fd721ae..3d16beb77440 100644
> --- a/arch/mips/boot/dts/ralink/mt7621.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7621.dtsi
> @@ -73,6 +73,7 @@ sysc: syscon@0 {
> wdt: watchdog@100 {
> compatible = "mediatek,mt7621-wdt";
> reg = <0x100 0x100>;
> + mediatek,sysctl = <&sysc>;
> };
>
> gpio: gpio@600 {
> --
> 2.25.1
applied to mips-next.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2023-02-28 22:19 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-14 10:39 [PATCH v7 0/5] watchdog: mt7621-wdt: avoid globals and arch dependencies Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 1/5] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
2023-02-14 15:10 ` Guenter Roeck
2023-02-14 10:39 ` [PATCH v7 2/5] mips: dts: ralink: mt7621: rename watchdog node from 'wdt' into 'watchdog' Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
2023-02-14 15:12 ` Sergio Paracuellos
2023-02-28 4:44 ` Sergio Paracuellos
2023-02-28 8:39 ` Thomas Bogendoerfer
2023-02-28 8:51 ` Sergio Paracuellos
2023-02-28 14:55 ` Guenter Roeck
2023-02-28 15:46 ` Sergio Paracuellos
2023-02-28 22:15 ` Thomas Bogendoerfer
2023-02-14 10:39 ` [PATCH v7 3/5] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
2023-02-28 22:15 ` Thomas Bogendoerfer
2023-02-14 10:39 ` [PATCH v7 4/5] watchdog: mt7621-wdt: avoid static global declarations Sergio Paracuellos
2023-02-14 10:39 ` [PATCH v7 5/5] watchdog: mt7621-wdt: avoid ralink architecture dependent code Sergio Paracuellos
2023-02-14 15:11 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).