* [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU @ 2019-05-12 10:10 Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node Anson Huang ` (2 more replies) 0 siblings, 3 replies; 14+ messages in thread From: Anson Huang @ 2019-05-12 10:10 UTC (permalink / raw) To: robh+dt, mark.rutland, wim, linux, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx i.MX system controller watchdog depends on SCU driver to support interrupt function, so it needs to be subnode of SCU node in DT, binding doc should be moved to fsl,scu.txt as well. Signed-off-by: Anson Huang <Anson.Huang@nxp.com> --- No change, just resend patch with correct encoding. --- .../devicetree/bindings/arm/freescale/fsl,scu.txt | 15 ++++++++++++++ .../bindings/watchdog/fsl-imx-sc-wdt.txt | 24 ---------------------- 2 files changed, 15 insertions(+), 24 deletions(-) delete mode 100644 Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt index 5d7dbab..1b56557 100644 --- a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt +++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt @@ -133,6 +133,16 @@ RTC bindings based on SCU Message Protocol Required properties: - compatible: should be "fsl,imx8qxp-sc-rtc"; +Watchdog bindings based on SCU Message Protocol +------------------------------------------------------------ + +Required properties: +- compatible: should be: + "fsl,imx8qxp-sc-wdt" + followed by "fsl,imx-sc-wdt"; +Optional properties: +- timeout-sec: contains the watchdog timeout in seconds. + Example (imx8qxp): ------------- aliases { @@ -185,6 +195,11 @@ firmware { rtc: rtc { compatible = "fsl,imx8qxp-sc-rtc"; }; + + watchdog { + compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; + timeout-sec = <60>; + }; }; }; diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt b/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt deleted file mode 100644 index 02b87e9..0000000 --- a/Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt +++ /dev/null @@ -1,24 +0,0 @@ -* Freescale i.MX System Controller Watchdog - -i.MX system controller watchdog is for i.MX SoCs with system controller inside, -the watchdog is managed by system controller, users can ONLY communicate with -system controller from secure mode for watchdog operations, so Linux i.MX system -controller watchdog driver will call ARM SMC API and trap into ARM-Trusted-Firmware -for watchdog operations, ARM-Trusted-Firmware is running at secure EL3 mode and -it will request system controller to execute the watchdog operation passed from -Linux kernel. - -Required properties: -- compatible: Should be : - "fsl,imx8qxp-sc-wdt" - followed by "fsl,imx-sc-wdt"; - -Optional properties: -- timeout-sec : Contains the watchdog timeout in seconds. - -Examples: - -watchdog { - compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; - timeout-sec = <60>; -}; -- 2.7.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node 2019-05-12 10:10 [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Anson Huang @ 2019-05-12 10:10 ` Anson Huang 2019-06-04 9:06 ` Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support Anson Huang 2019-05-13 17:12 ` [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Rob Herring 2 siblings, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-05-12 10:10 UTC (permalink / raw) To: robh+dt, mark.rutland, wim, linux, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx i.MX system controller watchdog has pretimeout function which depends on i.MX SCU driver, so it should be a subnode of SCU. Signed-off-by: Anson Huang <Anson.Huang@nxp.com> --- No change, just resend patch with correct encoding. --- arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi index 0683ee2..b17c22e 100644 --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi @@ -149,6 +149,11 @@ rtc: rtc { compatible = "fsl,imx8qxp-sc-rtc"; }; + + watchdog { + compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; + timeout-sec = <60>; + }; }; timer { @@ -517,9 +522,4 @@ power-domains = <&pd IMX_SC_R_GPIO_7>; }; }; - - watchdog { - compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; - timeout-sec = <60>; - }; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* RE: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node 2019-05-12 10:10 ` [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node Anson Huang @ 2019-06-04 9:06 ` Anson Huang 2019-06-05 6:00 ` Shawn Guo 0 siblings, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-06-04 9:06 UTC (permalink / raw) To: robh+dt, mark.rutland, wim, linux, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx Hi, Shawn The driver and binding doc are already reviewed and waiting for DT patch, will you help review this DT patch? Thanks, Anson > -----Original Message----- > From: Anson Huang > Sent: Sunday, May 12, 2019 6:11 PM > To: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; > linux@roeck-us.net; shawnguo@kernel.org; s.hauer@pengutronix.de; > kernel@pengutronix.de; festevam@gmail.com; Aisheng Dong > <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel Baluta > <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Cc: dl-linux-imx <linux-imx@nxp.com> > Subject: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node > into scu node > > i.MX system controller watchdog has pretimeout function which depends on > i.MX SCU driver, so it should be a subnode of SCU. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > No change, just resend patch with correct encoding. > --- > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > index 0683ee2..b17c22e 100644 > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > @@ -149,6 +149,11 @@ > rtc: rtc { > compatible = "fsl,imx8qxp-sc-rtc"; > }; > + > + watchdog { > + compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > + timeout-sec = <60>; > + }; > }; > > timer { > @@ -517,9 +522,4 @@ > power-domains = <&pd IMX_SC_R_GPIO_7>; > }; > }; > - > - watchdog { > - compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > - timeout-sec = <60>; > - }; > }; > -- > 2.7.4 ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node 2019-06-04 9:06 ` Anson Huang @ 2019-06-05 6:00 ` Shawn Guo 2019-06-05 6:09 ` Anson Huang 0 siblings, 1 reply; 14+ messages in thread From: Shawn Guo @ 2019-06-05 6:00 UTC (permalink / raw) To: Anson Huang Cc: robh+dt, mark.rutland, wim, linux, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx On Tue, Jun 04, 2019 at 09:06:28AM +0000, Anson Huang wrote: > Hi, Shawn > The driver and binding doc are already reviewed and waiting for DT patch, will you help review this DT patch? I'm fine with it. Should I just pick this patch up, or is there any dependency we need to handle? Shawn > > Thanks, > Anson > > > -----Original Message----- > > From: Anson Huang > > Sent: Sunday, May 12, 2019 6:11 PM > > To: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; > > linux@roeck-us.net; shawnguo@kernel.org; s.hauer@pengutronix.de; > > kernel@pengutronix.de; festevam@gmail.com; Aisheng Dong > > <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel Baluta > > <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > Cc: dl-linux-imx <linux-imx@nxp.com> > > Subject: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node > > into scu node > > > > i.MX system controller watchdog has pretimeout function which depends on > > i.MX SCU driver, so it should be a subnode of SCU. > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > --- > > No change, just resend patch with correct encoding. > > --- > > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > index 0683ee2..b17c22e 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > @@ -149,6 +149,11 @@ > > rtc: rtc { > > compatible = "fsl,imx8qxp-sc-rtc"; > > }; > > + > > + watchdog { > > + compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > > + timeout-sec = <60>; > > + }; > > }; > > > > timer { > > @@ -517,9 +522,4 @@ > > power-domains = <&pd IMX_SC_R_GPIO_7>; > > }; > > }; > > - > > - watchdog { > > - compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > > - timeout-sec = <60>; > > - }; > > }; > > -- > > 2.7.4 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node 2019-06-05 6:00 ` Shawn Guo @ 2019-06-05 6:09 ` Anson Huang 2019-06-05 6:12 ` Shawn Guo 0 siblings, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-06-05 6:09 UTC (permalink / raw) To: Shawn Guo Cc: robh+dt, mark.rutland, wim, linux, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx Hi, Shawn > -----Original Message----- > From: Shawn Guo <shawnguo@kernel.org> > Sent: Wednesday, June 5, 2019 2:00 PM > To: Anson Huang <anson.huang@nxp.com> > Cc: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; > linux@roeck-us.net; s.hauer@pengutronix.de; kernel@pengutronix.de; > festevam@gmail.com; Aisheng Dong <aisheng.dong@nxp.com>; > ulf.hansson@linaro.org; Daniel Baluta <daniel.baluta@nxp.com>; Peng Fan > <peng.fan@nxp.com>; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; linux-watchdog@vger.kernel.org; linux-arm- > kernel@lists.infradead.org; dl-linux-imx <linux-imx@nxp.com> > Subject: Re: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog > node into scu node > > On Tue, Jun 04, 2019 at 09:06:28AM +0000, Anson Huang wrote: > > Hi, Shawn > > The driver and binding doc are already reviewed and waiting for DT > patch, will you help review this DT patch? > > I'm fine with it. Should I just pick this patch up, or is there any dependency > we need to handle? No dependency, it just makes more sense to move the node into SCU node as it is NOW depending on SCU driver. Once you pick up this patch (and maybe the dt-binding patch as well?), I will notify the watchdog maintainer to pick up the watchdog patch in this series. Thanks, Anson > > Shawn > > > > > Thanks, > > Anson > > > > > -----Original Message----- > > > From: Anson Huang > > > Sent: Sunday, May 12, 2019 6:11 PM > > > To: robh+dt@kernel.org; mark.rutland@arm.com; > > > wim@linux-watchdog.org; linux@roeck-us.net; shawnguo@kernel.org; > > > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > > > Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel > > > Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > > > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > > > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > > Cc: dl-linux-imx <linux-imx@nxp.com> > > > Subject: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog > > > node into scu node > > > > > > i.MX system controller watchdog has pretimeout function which > > > depends on i.MX SCU driver, so it should be a subnode of SCU. > > > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > > --- > > > No change, just resend patch with correct encoding. > > > --- > > > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 10 +++++----- > > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > > index 0683ee2..b17c22e 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > > @@ -149,6 +149,11 @@ > > > rtc: rtc { > > > compatible = "fsl,imx8qxp-sc-rtc"; > > > }; > > > + > > > + watchdog { > > > + compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > > > + timeout-sec = <60>; > > > + }; > > > }; > > > > > > timer { > > > @@ -517,9 +522,4 @@ > > > power-domains = <&pd IMX_SC_R_GPIO_7>; > > > }; > > > }; > > > - > > > - watchdog { > > > - compatible = "fsl,imx8qxp-sc-wdt", "fsl,imx-sc-wdt"; > > > - timeout-sec = <60>; > > > - }; > > > }; > > > -- > > > 2.7.4 > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node 2019-06-05 6:09 ` Anson Huang @ 2019-06-05 6:12 ` Shawn Guo 0 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2019-06-05 6:12 UTC (permalink / raw) To: Anson Huang Cc: robh+dt, mark.rutland, wim, linux, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx On Wed, Jun 05, 2019 at 06:09:12AM +0000, Anson Huang wrote: > Hi, Shawn > > > -----Original Message----- > > From: Shawn Guo <shawnguo@kernel.org> > > Sent: Wednesday, June 5, 2019 2:00 PM > > To: Anson Huang <anson.huang@nxp.com> > > Cc: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; > > linux@roeck-us.net; s.hauer@pengutronix.de; kernel@pengutronix.de; > > festevam@gmail.com; Aisheng Dong <aisheng.dong@nxp.com>; > > ulf.hansson@linaro.org; Daniel Baluta <daniel.baluta@nxp.com>; Peng Fan > > <peng.fan@nxp.com>; devicetree@vger.kernel.org; linux- > > kernel@vger.kernel.org; linux-watchdog@vger.kernel.org; linux-arm- > > kernel@lists.infradead.org; dl-linux-imx <linux-imx@nxp.com> > > Subject: Re: [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog > > node into scu node > > > > On Tue, Jun 04, 2019 at 09:06:28AM +0000, Anson Huang wrote: > > > Hi, Shawn > > > The driver and binding doc are already reviewed and waiting for DT > > patch, will you help review this DT patch? > > > > I'm fine with it. Should I just pick this patch up, or is there any dependency > > we need to handle? > > No dependency, it just makes more sense to move the node into SCU node as it is NOW depending on SCU > driver. Once you pick up this patch Just applied. > (and maybe the dt-binding patch as well?), It makes more sense to have it go through watchdog tree. Shawn > I will notify the watchdog > maintainer to pick up the watchdog patch in this series. ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-05-12 10:10 [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node Anson Huang @ 2019-05-12 10:10 ` Anson Huang 2019-05-12 13:28 ` Guenter Roeck 2019-05-13 17:12 ` [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Rob Herring 2 siblings, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-05-12 10:10 UTC (permalink / raw) To: robh+dt, mark.rutland, wim, linux, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx i.MX system controller watchdog can support pretimeout IRQ via general SCU MU IRQ, it depends on IMX_SCU and driver MUST be probed after SCU IPC ready, then enable corresponding SCU IRQ group and register SCU IRQ notifier, when watchdog pretimeout IRQ fires, SCU MU IRQ will be handled and watchdog pretimeout notifier will be called to handle the event. Signed-off-by: Anson Huang <Anson.Huang@nxp.com> --- No change, just resend patch with correct encoding. --- drivers/watchdog/Kconfig | 1 + drivers/watchdog/imx_sc_wdt.c | 116 +++++++++++++++++++++++++++++++++++------- 2 files changed, 98 insertions(+), 19 deletions(-) diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 7ea6037..e08238c 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -716,6 +716,7 @@ config IMX2_WDT config IMX_SC_WDT tristate "IMX SC Watchdog" depends on HAVE_ARM_SMCCC + depends on IMX_SCU select WATCHDOG_CORE help This is the driver for the system controller watchdog diff --git a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c index 49848b6..6ecc03f 100644 --- a/drivers/watchdog/imx_sc_wdt.c +++ b/drivers/watchdog/imx_sc_wdt.c @@ -4,6 +4,7 @@ */ #include <linux/arm-smccc.h> +#include <linux/firmware/imx/sci.h> #include <linux/io.h> #include <linux/init.h> #include <linux/kernel.h> @@ -33,11 +34,19 @@ #define SC_TIMER_WDOG_ACTION_PARTITION 0 +#define SC_IRQ_WDOG 1 +#define SC_IRQ_GROUP_WDOG 1 + static bool nowayout = WATCHDOG_NOWAYOUT; module_param(nowayout, bool, 0000); MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); +struct imx_sc_wdt_device { + struct watchdog_device wdd; + struct notifier_block wdt_notifier; +}; + static int imx_sc_wdt_ping(struct watchdog_device *wdog) { struct arm_smccc_res res; @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct watchdog_device *wdog, return res.a0 ? -EACCES : 0; } +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, + unsigned int pretimeout) +{ + struct arm_smccc_res res; + + arm_smccc_smc(IMX_SIP_TIMER, IMX_SIP_TIMER_SET_PRETIME_WDOG, + pretimeout * 1000, 0, 0, 0, 0, 0, &res); + if (res.a0) + return -EACCES; + + wdog->pretimeout = pretimeout; + + return 0; +} + +static int imx_sc_wdt_notify(struct notifier_block *nb, + unsigned long event, void *group) +{ + struct imx_sc_wdt_device *imx_sc_wdd = + container_of(nb, + struct imx_sc_wdt_device, + wdt_notifier); + + if (event & SC_IRQ_WDOG && + *(u8 *)group == SC_IRQ_GROUP_WDOG) + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); + + return 0; +} + +static void imx_sc_wdt_action(void *data) +{ + struct notifier_block *wdt_notifier = data; + + imx_scu_irq_unregister_notifier(wdt_notifier); + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, + SC_IRQ_WDOG, + false); +} + static const struct watchdog_ops imx_sc_wdt_ops = { .owner = THIS_MODULE, .start = imx_sc_wdt_start, .stop = imx_sc_wdt_stop, .ping = imx_sc_wdt_ping, .set_timeout = imx_sc_wdt_set_timeout, + .set_pretimeout = imx_sc_wdt_set_pretimeout, }; -static const struct watchdog_info imx_sc_wdt_info = { +static struct watchdog_info imx_sc_wdt_info = { .identity = "i.MX SC watchdog timer", .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, + WDIOF_MAGICCLOSE, }; static int imx_sc_wdt_probe(struct platform_device *pdev) { + struct imx_sc_wdt_device *imx_sc_wdd; + struct watchdog_device *wdog; struct device *dev = &pdev->dev; - struct watchdog_device *imx_sc_wdd; int ret; imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), GFP_KERNEL); @@ -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct platform_device *pdev) platform_set_drvdata(pdev, imx_sc_wdd); - imx_sc_wdd->info = &imx_sc_wdt_info; - imx_sc_wdd->ops = &imx_sc_wdt_ops; - imx_sc_wdd->min_timeout = 1; - imx_sc_wdd->max_timeout = MAX_TIMEOUT; - imx_sc_wdd->parent = dev; - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; + wdog = &imx_sc_wdd->wdd; + wdog->info = &imx_sc_wdt_info; + wdog->ops = &imx_sc_wdt_ops; + wdog->min_timeout = 1; + wdog->max_timeout = MAX_TIMEOUT; + wdog->parent = dev; + wdog->timeout = DEFAULT_TIMEOUT; - watchdog_init_timeout(imx_sc_wdd, 0, dev); - watchdog_stop_on_reboot(imx_sc_wdd); - watchdog_stop_on_unregister(imx_sc_wdd); + watchdog_init_timeout(wdog, 0, dev); + watchdog_stop_on_reboot(wdog); + watchdog_stop_on_unregister(wdog); - ret = devm_watchdog_register_device(dev, imx_sc_wdd); + ret = devm_watchdog_register_device(dev, wdog); if (ret) { dev_err(dev, "Failed to register watchdog device\n"); return ret; } + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, + SC_IRQ_WDOG, + true); + if (ret) { + dev_warn(dev, "Enable irq failed, pretimeout NOT supported\n"); + return 0; + } + + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); + if (ret) { + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, + SC_IRQ_WDOG, + false); + dev_warn(dev, + "Register irq notifier failed, pretimeout NOT supported\n"); + return 0; + } + + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, + &imx_sc_wdd->wdt_notifier); + if (!ret) + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; + else + dev_warn(dev, "Add action failed, pretimeout NOT supported\n"); + return 0; } static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) { - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); - if (watchdog_active(imx_sc_wdd)) - imx_sc_wdt_stop(imx_sc_wdd); + if (watchdog_active(&imx_sc_wdd->wdd)) + imx_sc_wdt_stop(&imx_sc_wdd->wdd); return 0; } static int __maybe_unused imx_sc_wdt_resume(struct device *dev) { - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); - if (watchdog_active(imx_sc_wdd)) - imx_sc_wdt_start(imx_sc_wdd); + if (watchdog_active(&imx_sc_wdd->wdd)) + imx_sc_wdt_start(&imx_sc_wdd->wdd); return 0; } -- 2.7.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-05-12 10:10 ` [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support Anson Huang @ 2019-05-12 13:28 ` Guenter Roeck 2019-05-12 14:10 ` Anson Huang 2019-06-05 6:24 ` Anson Huang 0 siblings, 2 replies; 14+ messages in thread From: Guenter Roeck @ 2019-05-12 13:28 UTC (permalink / raw) To: Anson Huang, robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx On 5/12/19 3:10 AM, Anson Huang wrote: > i.MX system controller watchdog can support pretimeout IRQ > via general SCU MU IRQ, it depends on IMX_SCU and driver MUST > be probed after SCU IPC ready, then enable corresponding SCU > IRQ group and register SCU IRQ notifier, when watchdog pretimeout > IRQ fires, SCU MU IRQ will be handled and watchdog pretimeout > notifier will be called to handle the event. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> Revviewed-by: Guenter Roeck <linux@roeck-us.net> Other patches waiting for DT review. IMX API feedback below. Side note: This patch depends on 'firmware: imx: enable imx scu general irq function' which is not yet in mainline. > --- > No change, just resend patch with correct encoding. > --- > drivers/watchdog/Kconfig | 1 + > drivers/watchdog/imx_sc_wdt.c | 116 +++++++++++++++++++++++++++++++++++------- > 2 files changed, 98 insertions(+), 19 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 7ea6037..e08238c 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -716,6 +716,7 @@ config IMX2_WDT > config IMX_SC_WDT > tristate "IMX SC Watchdog" > depends on HAVE_ARM_SMCCC > + depends on IMX_SCU > select WATCHDOG_CORE > help > This is the driver for the system controller watchdog > diff --git a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c > index 49848b6..6ecc03f 100644 > --- a/drivers/watchdog/imx_sc_wdt.c > +++ b/drivers/watchdog/imx_sc_wdt.c > @@ -4,6 +4,7 @@ > */ > > #include <linux/arm-smccc.h> > +#include <linux/firmware/imx/sci.h> > #include <linux/io.h> > #include <linux/init.h> > #include <linux/kernel.h> > @@ -33,11 +34,19 @@ > > #define SC_TIMER_WDOG_ACTION_PARTITION 0 > > +#define SC_IRQ_WDOG 1 > +#define SC_IRQ_GROUP_WDOG 1 > + > static bool nowayout = WATCHDOG_NOWAYOUT; > module_param(nowayout, bool, 0000); > MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default=" > __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > > +struct imx_sc_wdt_device { > + struct watchdog_device wdd; > + struct notifier_block wdt_notifier; > +}; > + > static int imx_sc_wdt_ping(struct watchdog_device *wdog) > { > struct arm_smccc_res res; > @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct watchdog_device *wdog, > return res.a0 ? -EACCES : 0; > } > > +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, > + unsigned int pretimeout) > +{ > + struct arm_smccc_res res; > + > + arm_smccc_smc(IMX_SIP_TIMER, IMX_SIP_TIMER_SET_PRETIME_WDOG, > + pretimeout * 1000, 0, 0, 0, 0, 0, &res); > + if (res.a0) > + return -EACCES; > + > + wdog->pretimeout = pretimeout; > + > + return 0; > +} > + > +static int imx_sc_wdt_notify(struct notifier_block *nb, > + unsigned long event, void *group) > +{ > + struct imx_sc_wdt_device *imx_sc_wdd = > + container_of(nb, > + struct imx_sc_wdt_device, > + wdt_notifier); > + > + if (event & SC_IRQ_WDOG && > + *(u8 *)group == SC_IRQ_GROUP_WDOG) > + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); This should really not be necessary. Event mask and target group (if needed with a wildcard for the group) should be parameters of imx_scu_irq_register_notifier(), and be handled in the imx code. Also, passing 'group' as pointed seems excessive. Might as well pass it directly. Guenter > + > + return 0; > +} > + > +static void imx_sc_wdt_action(void *data) > +{ > + struct notifier_block *wdt_notifier = data; > + > + imx_scu_irq_unregister_notifier(wdt_notifier); > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > + SC_IRQ_WDOG, > + false); > +} > + > static const struct watchdog_ops imx_sc_wdt_ops = { > .owner = THIS_MODULE, > .start = imx_sc_wdt_start, > .stop = imx_sc_wdt_stop, > .ping = imx_sc_wdt_ping, > .set_timeout = imx_sc_wdt_set_timeout, > + .set_pretimeout = imx_sc_wdt_set_pretimeout, > }; > > -static const struct watchdog_info imx_sc_wdt_info = { > +static struct watchdog_info imx_sc_wdt_info = { > .identity = "i.MX SC watchdog timer", > .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | > - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, > + WDIOF_MAGICCLOSE, > }; > > static int imx_sc_wdt_probe(struct platform_device *pdev) > { > + struct imx_sc_wdt_device *imx_sc_wdd; > + struct watchdog_device *wdog; > struct device *dev = &pdev->dev; > - struct watchdog_device *imx_sc_wdd; > int ret; > > imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), GFP_KERNEL); > @@ -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, imx_sc_wdd); > > - imx_sc_wdd->info = &imx_sc_wdt_info; > - imx_sc_wdd->ops = &imx_sc_wdt_ops; > - imx_sc_wdd->min_timeout = 1; > - imx_sc_wdd->max_timeout = MAX_TIMEOUT; > - imx_sc_wdd->parent = dev; > - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; > + wdog = &imx_sc_wdd->wdd; > + wdog->info = &imx_sc_wdt_info; > + wdog->ops = &imx_sc_wdt_ops; > + wdog->min_timeout = 1; > + wdog->max_timeout = MAX_TIMEOUT; > + wdog->parent = dev; > + wdog->timeout = DEFAULT_TIMEOUT; > > - watchdog_init_timeout(imx_sc_wdd, 0, dev); > - watchdog_stop_on_reboot(imx_sc_wdd); > - watchdog_stop_on_unregister(imx_sc_wdd); > + watchdog_init_timeout(wdog, 0, dev); > + watchdog_stop_on_reboot(wdog); > + watchdog_stop_on_unregister(wdog); > > - ret = devm_watchdog_register_device(dev, imx_sc_wdd); > + ret = devm_watchdog_register_device(dev, wdog); > if (ret) { > dev_err(dev, "Failed to register watchdog device\n"); > return ret; > } > > + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > + SC_IRQ_WDOG, > + true); > + if (ret) { > + dev_warn(dev, "Enable irq failed, pretimeout NOT supported\n"); > + return 0; > + } > + > + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; > + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); > + if (ret) { > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > + SC_IRQ_WDOG, > + false); > + dev_warn(dev, > + "Register irq notifier failed, pretimeout NOT supported\n"); > + return 0; > + } > + > + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, > + &imx_sc_wdd->wdt_notifier); > + if (!ret) > + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; > + else > + dev_warn(dev, "Add action failed, pretimeout NOT supported\n"); > + > return 0; > } > > static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) > { > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > - if (watchdog_active(imx_sc_wdd)) > - imx_sc_wdt_stop(imx_sc_wdd); > + if (watchdog_active(&imx_sc_wdd->wdd)) > + imx_sc_wdt_stop(&imx_sc_wdd->wdd); > > return 0; > } > > static int __maybe_unused imx_sc_wdt_resume(struct device *dev) > { > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > - if (watchdog_active(imx_sc_wdd)) > - imx_sc_wdt_start(imx_sc_wdd); > + if (watchdog_active(&imx_sc_wdd->wdd)) > + imx_sc_wdt_start(&imx_sc_wdd->wdd); > > return 0; > } > ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-05-12 13:28 ` Guenter Roeck @ 2019-05-12 14:10 ` Anson Huang 2019-06-05 6:24 ` Anson Huang 1 sibling, 0 replies; 14+ messages in thread From: Anson Huang @ 2019-05-12 14:10 UTC (permalink / raw) To: Guenter Roeck, robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx > -----Original Message----- > From: Guenter Roeck [mailto:groeck7@gmail.com] On Behalf Of Guenter > Roeck > Sent: Sunday, May 12, 2019 9:28 PM > To: Anson Huang <anson.huang@nxp.com>; robh+dt@kernel.org; > mark.rutland@arm.com; wim@linux-watchdog.org; shawnguo@kernel.org; > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel > Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Cc: dl-linux-imx <linux-imx@nxp.com> > Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout > support > > On 5/12/19 3:10 AM, Anson Huang wrote: > > i.MX system controller watchdog can support pretimeout IRQ via general > > SCU MU IRQ, it depends on IMX_SCU and driver MUST be probed after SCU > > IPC ready, then enable corresponding SCU IRQ group and register SCU > > IRQ notifier, when watchdog pretimeout IRQ fires, SCU MU IRQ will be > > handled and watchdog pretimeout notifier will be called to handle the > > event. > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > Revviewed-by: Guenter Roeck <linux@roeck-us.net> > > Other patches waiting for DT review. IMX API feedback below. > > Side note: This patch depends on 'firmware: imx: enable imx scu general irq > function' which is not yet in mainline. Thanks Guenter. The ''firmware: imx: enable imx scu general irq function' patch is already in shawn's imx/drivers branch, should be in mainline after next merge window. Anson. > > > --- > > No change, just resend patch with correct encoding. > > --- > > drivers/watchdog/Kconfig | 1 + > > drivers/watchdog/imx_sc_wdt.c | 116 > +++++++++++++++++++++++++++++++++++------- > > 2 files changed, 98 insertions(+), 19 deletions(-) > > > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index > > 7ea6037..e08238c 100644 > > --- a/drivers/watchdog/Kconfig > > +++ b/drivers/watchdog/Kconfig > > @@ -716,6 +716,7 @@ config IMX2_WDT > > config IMX_SC_WDT > > tristate "IMX SC Watchdog" > > depends on HAVE_ARM_SMCCC > > + depends on IMX_SCU > > select WATCHDOG_CORE > > help > > This is the driver for the system controller watchdog diff --git > > a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c index > > 49848b6..6ecc03f 100644 > > --- a/drivers/watchdog/imx_sc_wdt.c > > +++ b/drivers/watchdog/imx_sc_wdt.c > > @@ -4,6 +4,7 @@ > > */ > > > > #include <linux/arm-smccc.h> > > +#include <linux/firmware/imx/sci.h> > > #include <linux/io.h> > > #include <linux/init.h> > > #include <linux/kernel.h> > > @@ -33,11 +34,19 @@ > > > > #define SC_TIMER_WDOG_ACTION_PARTITION 0 > > > > +#define SC_IRQ_WDOG 1 > > +#define SC_IRQ_GROUP_WDOG 1 > > + > > static bool nowayout = WATCHDOG_NOWAYOUT; > > module_param(nowayout, bool, 0000); > > MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once > started (default=" > > __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > > > > +struct imx_sc_wdt_device { > > + struct watchdog_device wdd; > > + struct notifier_block wdt_notifier; > > +}; > > + > > static int imx_sc_wdt_ping(struct watchdog_device *wdog) > > { > > struct arm_smccc_res res; > > @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct > watchdog_device *wdog, > > return res.a0 ? -EACCES : 0; > > } > > > > +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, > > + unsigned int pretimeout) > > +{ > > + struct arm_smccc_res res; > > + > > + arm_smccc_smc(IMX_SIP_TIMER, > IMX_SIP_TIMER_SET_PRETIME_WDOG, > > + pretimeout * 1000, 0, 0, 0, 0, 0, &res); > > + if (res.a0) > > + return -EACCES; > > + > > + wdog->pretimeout = pretimeout; > > + > > + return 0; > > +} > > + > > +static int imx_sc_wdt_notify(struct notifier_block *nb, > > + unsigned long event, void *group) { > > + struct imx_sc_wdt_device *imx_sc_wdd = > > + container_of(nb, > > + struct imx_sc_wdt_device, > > + wdt_notifier); > > + > > + if (event & SC_IRQ_WDOG && > > + *(u8 *)group == SC_IRQ_GROUP_WDOG) > > + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); > > This should really not be necessary. Event mask and target group (if needed > with a wildcard for the group) should be parameters of > imx_scu_irq_register_notifier(), and be handled in the imx code. > > Also, passing 'group' as pointed seems excessive. Might as well pass it > directly. > > Guenter > > > + > > + return 0; > > +} > > + > > +static void imx_sc_wdt_action(void *data) { > > + struct notifier_block *wdt_notifier = data; > > + > > + imx_scu_irq_unregister_notifier(wdt_notifier); > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + false); > > +} > > + > > static const struct watchdog_ops imx_sc_wdt_ops = { > > .owner = THIS_MODULE, > > .start = imx_sc_wdt_start, > > .stop = imx_sc_wdt_stop, > > .ping = imx_sc_wdt_ping, > > .set_timeout = imx_sc_wdt_set_timeout, > > + .set_pretimeout = imx_sc_wdt_set_pretimeout, > > }; > > > > -static const struct watchdog_info imx_sc_wdt_info = { > > +static struct watchdog_info imx_sc_wdt_info = { > > .identity = "i.MX SC watchdog timer", > > .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | > > - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, > > + WDIOF_MAGICCLOSE, > > }; > > > > static int imx_sc_wdt_probe(struct platform_device *pdev) > > { > > + struct imx_sc_wdt_device *imx_sc_wdd; > > + struct watchdog_device *wdog; > > struct device *dev = &pdev->dev; > > - struct watchdog_device *imx_sc_wdd; > > int ret; > > > > imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), GFP_KERNEL); > @@ > > -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct platform_device > > *pdev) > > > > platform_set_drvdata(pdev, imx_sc_wdd); > > > > - imx_sc_wdd->info = &imx_sc_wdt_info; > > - imx_sc_wdd->ops = &imx_sc_wdt_ops; > > - imx_sc_wdd->min_timeout = 1; > > - imx_sc_wdd->max_timeout = MAX_TIMEOUT; > > - imx_sc_wdd->parent = dev; > > - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; > > + wdog = &imx_sc_wdd->wdd; > > + wdog->info = &imx_sc_wdt_info; > > + wdog->ops = &imx_sc_wdt_ops; > > + wdog->min_timeout = 1; > > + wdog->max_timeout = MAX_TIMEOUT; > > + wdog->parent = dev; > > + wdog->timeout = DEFAULT_TIMEOUT; > > > > - watchdog_init_timeout(imx_sc_wdd, 0, dev); > > - watchdog_stop_on_reboot(imx_sc_wdd); > > - watchdog_stop_on_unregister(imx_sc_wdd); > > + watchdog_init_timeout(wdog, 0, dev); > > + watchdog_stop_on_reboot(wdog); > > + watchdog_stop_on_unregister(wdog); > > > > - ret = devm_watchdog_register_device(dev, imx_sc_wdd); > > + ret = devm_watchdog_register_device(dev, wdog); > > if (ret) { > > dev_err(dev, "Failed to register watchdog device\n"); > > return ret; > > } > > > > + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + true); > > + if (ret) { > > + dev_warn(dev, "Enable irq failed, pretimeout NOT > supported\n"); > > + return 0; > > + } > > + > > + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; > > + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); > > + if (ret) { > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + false); > > + dev_warn(dev, > > + "Register irq notifier failed, pretimeout NOT > supported\n"); > > + return 0; > > + } > > + > > + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, > > + &imx_sc_wdd->wdt_notifier); > > + if (!ret) > > + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; > > + else > > + dev_warn(dev, "Add action failed, pretimeout NOT > supported\n"); > > + > > return 0; > > } > > > > static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) > > { > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > - if (watchdog_active(imx_sc_wdd)) > > - imx_sc_wdt_stop(imx_sc_wdd); > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > + imx_sc_wdt_stop(&imx_sc_wdd->wdd); > > > > return 0; > > } > > > > static int __maybe_unused imx_sc_wdt_resume(struct device *dev) > > { > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > - if (watchdog_active(imx_sc_wdd)) > > - imx_sc_wdt_start(imx_sc_wdd); > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > + imx_sc_wdt_start(&imx_sc_wdd->wdd); > > > > return 0; > > } > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-05-12 13:28 ` Guenter Roeck 2019-05-12 14:10 ` Anson Huang @ 2019-06-05 6:24 ` Anson Huang 2019-06-07 17:40 ` Guenter Roeck 1 sibling, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-06-05 6:24 UTC (permalink / raw) To: Guenter Roeck, robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel Cc: dl-linux-imx Hi, Guenter > -----Original Message----- > From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck > Sent: Sunday, May 12, 2019 9:28 PM > To: Anson Huang <anson.huang@nxp.com>; robh+dt@kernel.org; > mark.rutland@arm.com; wim@linux-watchdog.org; shawnguo@kernel.org; > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel > Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Cc: dl-linux-imx <linux-imx@nxp.com> > Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout > support > > On 5/12/19 3:10 AM, Anson Huang wrote: > > i.MX system controller watchdog can support pretimeout IRQ via general > > SCU MU IRQ, it depends on IMX_SCU and driver MUST be probed after SCU > > IPC ready, then enable corresponding SCU IRQ group and register SCU > > IRQ notifier, when watchdog pretimeout IRQ fires, SCU MU IRQ will be > > handled and watchdog pretimeout notifier will be called to handle the > > event. > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > Revviewed-by: Guenter Roeck <linux@roeck-us.net> > > Other patches waiting for DT review. IMX API feedback below. Shaw just picked up the DT patch. Would you please pick up this driver and dt-binding patch to you git repo? Thanks, Anson > > Side note: This patch depends on 'firmware: imx: enable imx scu general irq > function' which is not yet in mainline. > > > --- > > No change, just resend patch with correct encoding. > > --- > > drivers/watchdog/Kconfig | 1 + > > drivers/watchdog/imx_sc_wdt.c | 116 > +++++++++++++++++++++++++++++++++++------- > > 2 files changed, 98 insertions(+), 19 deletions(-) > > > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index > > 7ea6037..e08238c 100644 > > --- a/drivers/watchdog/Kconfig > > +++ b/drivers/watchdog/Kconfig > > @@ -716,6 +716,7 @@ config IMX2_WDT > > config IMX_SC_WDT > > tristate "IMX SC Watchdog" > > depends on HAVE_ARM_SMCCC > > + depends on IMX_SCU > > select WATCHDOG_CORE > > help > > This is the driver for the system controller watchdog diff --git > > a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c index > > 49848b6..6ecc03f 100644 > > --- a/drivers/watchdog/imx_sc_wdt.c > > +++ b/drivers/watchdog/imx_sc_wdt.c > > @@ -4,6 +4,7 @@ > > */ > > > > #include <linux/arm-smccc.h> > > +#include <linux/firmware/imx/sci.h> > > #include <linux/io.h> > > #include <linux/init.h> > > #include <linux/kernel.h> > > @@ -33,11 +34,19 @@ > > > > #define SC_TIMER_WDOG_ACTION_PARTITION 0 > > > > +#define SC_IRQ_WDOG 1 > > +#define SC_IRQ_GROUP_WDOG 1 > > + > > static bool nowayout = WATCHDOG_NOWAYOUT; > > module_param(nowayout, bool, 0000); > > MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once > started (default=" > > __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > > > > +struct imx_sc_wdt_device { > > + struct watchdog_device wdd; > > + struct notifier_block wdt_notifier; > > +}; > > + > > static int imx_sc_wdt_ping(struct watchdog_device *wdog) > > { > > struct arm_smccc_res res; > > @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct > watchdog_device *wdog, > > return res.a0 ? -EACCES : 0; > > } > > > > +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, > > + unsigned int pretimeout) > > +{ > > + struct arm_smccc_res res; > > + > > + arm_smccc_smc(IMX_SIP_TIMER, > IMX_SIP_TIMER_SET_PRETIME_WDOG, > > + pretimeout * 1000, 0, 0, 0, 0, 0, &res); > > + if (res.a0) > > + return -EACCES; > > + > > + wdog->pretimeout = pretimeout; > > + > > + return 0; > > +} > > + > > +static int imx_sc_wdt_notify(struct notifier_block *nb, > > + unsigned long event, void *group) { > > + struct imx_sc_wdt_device *imx_sc_wdd = > > + container_of(nb, > > + struct imx_sc_wdt_device, > > + wdt_notifier); > > + > > + if (event & SC_IRQ_WDOG && > > + *(u8 *)group == SC_IRQ_GROUP_WDOG) > > + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); > > This should really not be necessary. Event mask and target group (if needed > with a wildcard for the group) should be parameters of > imx_scu_irq_register_notifier(), and be handled in the imx code. > > Also, passing 'group' as pointed seems excessive. Might as well pass it > directly. > > Guenter > > > + > > + return 0; > > +} > > + > > +static void imx_sc_wdt_action(void *data) { > > + struct notifier_block *wdt_notifier = data; > > + > > + imx_scu_irq_unregister_notifier(wdt_notifier); > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + false); > > +} > > + > > static const struct watchdog_ops imx_sc_wdt_ops = { > > .owner = THIS_MODULE, > > .start = imx_sc_wdt_start, > > .stop = imx_sc_wdt_stop, > > .ping = imx_sc_wdt_ping, > > .set_timeout = imx_sc_wdt_set_timeout, > > + .set_pretimeout = imx_sc_wdt_set_pretimeout, > > }; > > > > -static const struct watchdog_info imx_sc_wdt_info = { > > +static struct watchdog_info imx_sc_wdt_info = { > > .identity = "i.MX SC watchdog timer", > > .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | > > - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, > > + WDIOF_MAGICCLOSE, > > }; > > > > static int imx_sc_wdt_probe(struct platform_device *pdev) > > { > > + struct imx_sc_wdt_device *imx_sc_wdd; > > + struct watchdog_device *wdog; > > struct device *dev = &pdev->dev; > > - struct watchdog_device *imx_sc_wdd; > > int ret; > > > > imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), GFP_KERNEL); > @@ > > -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct platform_device > > *pdev) > > > > platform_set_drvdata(pdev, imx_sc_wdd); > > > > - imx_sc_wdd->info = &imx_sc_wdt_info; > > - imx_sc_wdd->ops = &imx_sc_wdt_ops; > > - imx_sc_wdd->min_timeout = 1; > > - imx_sc_wdd->max_timeout = MAX_TIMEOUT; > > - imx_sc_wdd->parent = dev; > > - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; > > + wdog = &imx_sc_wdd->wdd; > > + wdog->info = &imx_sc_wdt_info; > > + wdog->ops = &imx_sc_wdt_ops; > > + wdog->min_timeout = 1; > > + wdog->max_timeout = MAX_TIMEOUT; > > + wdog->parent = dev; > > + wdog->timeout = DEFAULT_TIMEOUT; > > > > - watchdog_init_timeout(imx_sc_wdd, 0, dev); > > - watchdog_stop_on_reboot(imx_sc_wdd); > > - watchdog_stop_on_unregister(imx_sc_wdd); > > + watchdog_init_timeout(wdog, 0, dev); > > + watchdog_stop_on_reboot(wdog); > > + watchdog_stop_on_unregister(wdog); > > > > - ret = devm_watchdog_register_device(dev, imx_sc_wdd); > > + ret = devm_watchdog_register_device(dev, wdog); > > if (ret) { > > dev_err(dev, "Failed to register watchdog device\n"); > > return ret; > > } > > > > + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + true); > > + if (ret) { > > + dev_warn(dev, "Enable irq failed, pretimeout NOT > supported\n"); > > + return 0; > > + } > > + > > + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; > > + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); > > + if (ret) { > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > + SC_IRQ_WDOG, > > + false); > > + dev_warn(dev, > > + "Register irq notifier failed, pretimeout NOT > supported\n"); > > + return 0; > > + } > > + > > + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, > > + &imx_sc_wdd->wdt_notifier); > > + if (!ret) > > + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; > > + else > > + dev_warn(dev, "Add action failed, pretimeout NOT > supported\n"); > > + > > return 0; > > } > > > > static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) > > { > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > - if (watchdog_active(imx_sc_wdd)) > > - imx_sc_wdt_stop(imx_sc_wdd); > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > + imx_sc_wdt_stop(&imx_sc_wdd->wdd); > > > > return 0; > > } > > > > static int __maybe_unused imx_sc_wdt_resume(struct device *dev) > > { > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > - if (watchdog_active(imx_sc_wdd)) > > - imx_sc_wdt_start(imx_sc_wdd); > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > + imx_sc_wdt_start(&imx_sc_wdd->wdd); > > > > return 0; > > } > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-06-05 6:24 ` Anson Huang @ 2019-06-07 17:40 ` Guenter Roeck 2019-06-08 9:32 ` Anson Huang 0 siblings, 1 reply; 14+ messages in thread From: Guenter Roeck @ 2019-06-07 17:40 UTC (permalink / raw) To: Anson Huang Cc: robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx On Wed, Jun 05, 2019 at 06:24:33AM +0000, Anson Huang wrote: > Hi, Guenter > > > -----Original Message----- > > From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck > > Sent: Sunday, May 12, 2019 9:28 PM > > To: Anson Huang <anson.huang@nxp.com>; robh+dt@kernel.org; > > mark.rutland@arm.com; wim@linux-watchdog.org; shawnguo@kernel.org; > > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > > Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel > > Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > Cc: dl-linux-imx <linux-imx@nxp.com> > > Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout > > support > > > > On 5/12/19 3:10 AM, Anson Huang wrote: > > > i.MX system controller watchdog can support pretimeout IRQ via general > > > SCU MU IRQ, it depends on IMX_SCU and driver MUST be probed after SCU > > > IPC ready, then enable corresponding SCU IRQ group and register SCU > > > IRQ notifier, when watchdog pretimeout IRQ fires, SCU MU IRQ will be > > > handled and watchdog pretimeout notifier will be called to handle the > > > event. > > > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > > > Revviewed-by: Guenter Roeck <linux@roeck-us.net> > > > > Other patches waiting for DT review. IMX API feedback below. > > Shaw just picked up the DT patch. Would you please pick up this driver and dt-binding patch > to you git repo? > I'll pick up patch 3/3, but I can not pick up patches into arch/arm64/boot/; that is the responsibility of arm64 maintainers. I can only do that if I get an explicit Ack and permission to do so from an arm64 maintainer, and I don't recall getting that. Guenter > Thanks, > Anson > > > > > Side note: This patch depends on 'firmware: imx: enable imx scu general irq > > function' which is not yet in mainline. > > > > > --- > > > No change, just resend patch with correct encoding. > > > --- > > > drivers/watchdog/Kconfig | 1 + > > > drivers/watchdog/imx_sc_wdt.c | 116 > > +++++++++++++++++++++++++++++++++++------- > > > 2 files changed, 98 insertions(+), 19 deletions(-) > > > > > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index > > > 7ea6037..e08238c 100644 > > > --- a/drivers/watchdog/Kconfig > > > +++ b/drivers/watchdog/Kconfig > > > @@ -716,6 +716,7 @@ config IMX2_WDT > > > config IMX_SC_WDT > > > tristate "IMX SC Watchdog" > > > depends on HAVE_ARM_SMCCC > > > + depends on IMX_SCU > > > select WATCHDOG_CORE > > > help > > > This is the driver for the system controller watchdog diff --git > > > a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c index > > > 49848b6..6ecc03f 100644 > > > --- a/drivers/watchdog/imx_sc_wdt.c > > > +++ b/drivers/watchdog/imx_sc_wdt.c > > > @@ -4,6 +4,7 @@ > > > */ > > > > > > #include <linux/arm-smccc.h> > > > +#include <linux/firmware/imx/sci.h> > > > #include <linux/io.h> > > > #include <linux/init.h> > > > #include <linux/kernel.h> > > > @@ -33,11 +34,19 @@ > > > > > > #define SC_TIMER_WDOG_ACTION_PARTITION 0 > > > > > > +#define SC_IRQ_WDOG 1 > > > +#define SC_IRQ_GROUP_WDOG 1 > > > + > > > static bool nowayout = WATCHDOG_NOWAYOUT; > > > module_param(nowayout, bool, 0000); > > > MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once > > started (default=" > > > __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > > > > > > +struct imx_sc_wdt_device { > > > + struct watchdog_device wdd; > > > + struct notifier_block wdt_notifier; > > > +}; > > > + > > > static int imx_sc_wdt_ping(struct watchdog_device *wdog) > > > { > > > struct arm_smccc_res res; > > > @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct > > watchdog_device *wdog, > > > return res.a0 ? -EACCES : 0; > > > } > > > > > > +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, > > > + unsigned int pretimeout) > > > +{ > > > + struct arm_smccc_res res; > > > + > > > + arm_smccc_smc(IMX_SIP_TIMER, > > IMX_SIP_TIMER_SET_PRETIME_WDOG, > > > + pretimeout * 1000, 0, 0, 0, 0, 0, &res); > > > + if (res.a0) > > > + return -EACCES; > > > + > > > + wdog->pretimeout = pretimeout; > > > + > > > + return 0; > > > +} > > > + > > > +static int imx_sc_wdt_notify(struct notifier_block *nb, > > > + unsigned long event, void *group) { > > > + struct imx_sc_wdt_device *imx_sc_wdd = > > > + container_of(nb, > > > + struct imx_sc_wdt_device, > > > + wdt_notifier); > > > + > > > + if (event & SC_IRQ_WDOG && > > > + *(u8 *)group == SC_IRQ_GROUP_WDOG) > > > + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); > > > > This should really not be necessary. Event mask and target group (if needed > > with a wildcard for the group) should be parameters of > > imx_scu_irq_register_notifier(), and be handled in the imx code. > > > > Also, passing 'group' as pointed seems excessive. Might as well pass it > > directly. > > > > Guenter > > > > > + > > > + return 0; > > > +} > > > + > > > +static void imx_sc_wdt_action(void *data) { > > > + struct notifier_block *wdt_notifier = data; > > > + > > > + imx_scu_irq_unregister_notifier(wdt_notifier); > > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > + SC_IRQ_WDOG, > > > + false); > > > +} > > > + > > > static const struct watchdog_ops imx_sc_wdt_ops = { > > > .owner = THIS_MODULE, > > > .start = imx_sc_wdt_start, > > > .stop = imx_sc_wdt_stop, > > > .ping = imx_sc_wdt_ping, > > > .set_timeout = imx_sc_wdt_set_timeout, > > > + .set_pretimeout = imx_sc_wdt_set_pretimeout, > > > }; > > > > > > -static const struct watchdog_info imx_sc_wdt_info = { > > > +static struct watchdog_info imx_sc_wdt_info = { > > > .identity = "i.MX SC watchdog timer", > > > .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | > > > - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, > > > + WDIOF_MAGICCLOSE, > > > }; > > > > > > static int imx_sc_wdt_probe(struct platform_device *pdev) > > > { > > > + struct imx_sc_wdt_device *imx_sc_wdd; > > > + struct watchdog_device *wdog; > > > struct device *dev = &pdev->dev; > > > - struct watchdog_device *imx_sc_wdd; > > > int ret; > > > > > > imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), GFP_KERNEL); > > @@ > > > -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct platform_device > > > *pdev) > > > > > > platform_set_drvdata(pdev, imx_sc_wdd); > > > > > > - imx_sc_wdd->info = &imx_sc_wdt_info; > > > - imx_sc_wdd->ops = &imx_sc_wdt_ops; > > > - imx_sc_wdd->min_timeout = 1; > > > - imx_sc_wdd->max_timeout = MAX_TIMEOUT; > > > - imx_sc_wdd->parent = dev; > > > - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; > > > + wdog = &imx_sc_wdd->wdd; > > > + wdog->info = &imx_sc_wdt_info; > > > + wdog->ops = &imx_sc_wdt_ops; > > > + wdog->min_timeout = 1; > > > + wdog->max_timeout = MAX_TIMEOUT; > > > + wdog->parent = dev; > > > + wdog->timeout = DEFAULT_TIMEOUT; > > > > > > - watchdog_init_timeout(imx_sc_wdd, 0, dev); > > > - watchdog_stop_on_reboot(imx_sc_wdd); > > > - watchdog_stop_on_unregister(imx_sc_wdd); > > > + watchdog_init_timeout(wdog, 0, dev); > > > + watchdog_stop_on_reboot(wdog); > > > + watchdog_stop_on_unregister(wdog); > > > > > > - ret = devm_watchdog_register_device(dev, imx_sc_wdd); > > > + ret = devm_watchdog_register_device(dev, wdog); > > > if (ret) { > > > dev_err(dev, "Failed to register watchdog device\n"); > > > return ret; > > > } > > > > > > + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > + SC_IRQ_WDOG, > > > + true); > > > + if (ret) { > > > + dev_warn(dev, "Enable irq failed, pretimeout NOT > > supported\n"); > > > + return 0; > > > + } > > > + > > > + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; > > > + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); > > > + if (ret) { > > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > + SC_IRQ_WDOG, > > > + false); > > > + dev_warn(dev, > > > + "Register irq notifier failed, pretimeout NOT > > supported\n"); > > > + return 0; > > > + } > > > + > > > + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, > > > + &imx_sc_wdd->wdt_notifier); > > > + if (!ret) > > > + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; > > > + else > > > + dev_warn(dev, "Add action failed, pretimeout NOT > > supported\n"); > > > + > > > return 0; > > > } > > > > > > static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) > > > { > > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > > > - if (watchdog_active(imx_sc_wdd)) > > > - imx_sc_wdt_stop(imx_sc_wdd); > > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > > + imx_sc_wdt_stop(&imx_sc_wdd->wdd); > > > > > > return 0; > > > } > > > > > > static int __maybe_unused imx_sc_wdt_resume(struct device *dev) > > > { > > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > > > - if (watchdog_active(imx_sc_wdd)) > > > - imx_sc_wdt_start(imx_sc_wdd); > > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > > + imx_sc_wdt_start(&imx_sc_wdd->wdd); > > > > > > return 0; > > > } > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* RE: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-06-07 17:40 ` Guenter Roeck @ 2019-06-08 9:32 ` Anson Huang 2019-06-08 13:22 ` Guenter Roeck 0 siblings, 1 reply; 14+ messages in thread From: Anson Huang @ 2019-06-08 9:32 UTC (permalink / raw) To: Guenter Roeck Cc: robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx Hi, Guenter > -----Original Message----- > From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck > Sent: Saturday, June 8, 2019 1:41 AM > To: Anson Huang <anson.huang@nxp.com> > Cc: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; > shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de; > festevam@gmail.com; Aisheng Dong <aisheng.dong@nxp.com>; > ulf.hansson@linaro.org; Daniel Baluta <daniel.baluta@nxp.com>; Peng Fan > <peng.fan@nxp.com>; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; linux-watchdog@vger.kernel.org; linux-arm- > kernel@lists.infradead.org; dl-linux-imx <linux-imx@nxp.com> > Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout > support > > On Wed, Jun 05, 2019 at 06:24:33AM +0000, Anson Huang wrote: > > Hi, Guenter > > > > > -----Original Message----- > > > From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck > > > Sent: Sunday, May 12, 2019 9:28 PM > > > To: Anson Huang <anson.huang@nxp.com>; robh+dt@kernel.org; > > > mark.rutland@arm.com; wim@linux-watchdog.org; > shawnguo@kernel.org; > > > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > > > Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel > > > Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; > > > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- > > > watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > > Cc: dl-linux-imx <linux-imx@nxp.com> > > > Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout > > > support > > > > > > On 5/12/19 3:10 AM, Anson Huang wrote: > > > > i.MX system controller watchdog can support pretimeout IRQ via > > > > general SCU MU IRQ, it depends on IMX_SCU and driver MUST be > > > > probed after SCU IPC ready, then enable corresponding SCU IRQ > > > > group and register SCU IRQ notifier, when watchdog pretimeout IRQ > > > > fires, SCU MU IRQ will be handled and watchdog pretimeout notifier > > > > will be called to handle the event. > > > > > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > > > > > Revviewed-by: Guenter Roeck <linux@roeck-us.net> > > > > > > Other patches waiting for DT review. IMX API feedback below. > > > > Shaw just picked up the DT patch. Would you please pick up this driver > > and dt-binding patch to you git repo? > > > I'll pick up patch 3/3, but I can not pick up patches into arch/arm64/boot/; > that is the responsibility of arm64 maintainers. I can only do that if I get an > explicit Ack and permission to do so from an arm64 maintainer, and I don't > recall getting that. Will you also pick up 1/3, the DT binding patch, as DT binding normally go with driver, if NOT, please advise who should pick up this patch. Thanks, Anson. > > Guenter > > > Thanks, > > Anson > > > > > > > > Side note: This patch depends on 'firmware: imx: enable imx scu > > > general irq function' which is not yet in mainline. > > > > > > > --- > > > > No change, just resend patch with correct encoding. > > > > --- > > > > drivers/watchdog/Kconfig | 1 + > > > > drivers/watchdog/imx_sc_wdt.c | 116 > > > +++++++++++++++++++++++++++++++++++------- > > > > 2 files changed, 98 insertions(+), 19 deletions(-) > > > > > > > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > > > > index 7ea6037..e08238c 100644 > > > > --- a/drivers/watchdog/Kconfig > > > > +++ b/drivers/watchdog/Kconfig > > > > @@ -716,6 +716,7 @@ config IMX2_WDT > > > > config IMX_SC_WDT > > > > tristate "IMX SC Watchdog" > > > > depends on HAVE_ARM_SMCCC > > > > + depends on IMX_SCU > > > > select WATCHDOG_CORE > > > > help > > > > This is the driver for the system controller watchdog diff > > > > --git a/drivers/watchdog/imx_sc_wdt.c > > > > b/drivers/watchdog/imx_sc_wdt.c index 49848b6..6ecc03f 100644 > > > > --- a/drivers/watchdog/imx_sc_wdt.c > > > > +++ b/drivers/watchdog/imx_sc_wdt.c > > > > @@ -4,6 +4,7 @@ > > > > */ > > > > > > > > #include <linux/arm-smccc.h> > > > > +#include <linux/firmware/imx/sci.h> > > > > #include <linux/io.h> > > > > #include <linux/init.h> > > > > #include <linux/kernel.h> > > > > @@ -33,11 +34,19 @@ > > > > > > > > #define SC_TIMER_WDOG_ACTION_PARTITION 0 > > > > > > > > +#define SC_IRQ_WDOG 1 > > > > +#define SC_IRQ_GROUP_WDOG 1 > > > > + > > > > static bool nowayout = WATCHDOG_NOWAYOUT; > > > > module_param(nowayout, bool, 0000); > > > > MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once > > > started (default=" > > > > __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > > > > > > > > +struct imx_sc_wdt_device { > > > > + struct watchdog_device wdd; > > > > + struct notifier_block wdt_notifier; }; > > > > + > > > > static int imx_sc_wdt_ping(struct watchdog_device *wdog) > > > > { > > > > struct arm_smccc_res res; > > > > @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct > > > watchdog_device *wdog, > > > > return res.a0 ? -EACCES : 0; > > > > } > > > > > > > > +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, > > > > + unsigned int pretimeout) { > > > > + struct arm_smccc_res res; > > > > + > > > > + arm_smccc_smc(IMX_SIP_TIMER, > > > IMX_SIP_TIMER_SET_PRETIME_WDOG, > > > > + pretimeout * 1000, 0, 0, 0, 0, 0, &res); > > > > + if (res.a0) > > > > + return -EACCES; > > > > + > > > > + wdog->pretimeout = pretimeout; > > > > + > > > > + return 0; > > > > +} > > > > + > > > > +static int imx_sc_wdt_notify(struct notifier_block *nb, > > > > + unsigned long event, void *group) { > > > > + struct imx_sc_wdt_device *imx_sc_wdd = > > > > + container_of(nb, > > > > + struct imx_sc_wdt_device, > > > > + wdt_notifier); > > > > + > > > > + if (event & SC_IRQ_WDOG && > > > > + *(u8 *)group == SC_IRQ_GROUP_WDOG) > > > > + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); > > > > > > This should really not be necessary. Event mask and target group (if > > > needed with a wildcard for the group) should be parameters of > > > imx_scu_irq_register_notifier(), and be handled in the imx code. > > > > > > Also, passing 'group' as pointed seems excessive. Might as well pass > > > it directly. > > > > > > Guenter > > > > > > > + > > > > + return 0; > > > > +} > > > > + > > > > +static void imx_sc_wdt_action(void *data) { > > > > + struct notifier_block *wdt_notifier = data; > > > > + > > > > + imx_scu_irq_unregister_notifier(wdt_notifier); > > > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > > + SC_IRQ_WDOG, > > > > + false); > > > > +} > > > > + > > > > static const struct watchdog_ops imx_sc_wdt_ops = { > > > > .owner = THIS_MODULE, > > > > .start = imx_sc_wdt_start, > > > > .stop = imx_sc_wdt_stop, > > > > .ping = imx_sc_wdt_ping, > > > > .set_timeout = imx_sc_wdt_set_timeout, > > > > + .set_pretimeout = imx_sc_wdt_set_pretimeout, > > > > }; > > > > > > > > -static const struct watchdog_info imx_sc_wdt_info = { > > > > +static struct watchdog_info imx_sc_wdt_info = { > > > > .identity = "i.MX SC watchdog timer", > > > > .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | > > > > - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, > > > > + WDIOF_MAGICCLOSE, > > > > }; > > > > > > > > static int imx_sc_wdt_probe(struct platform_device *pdev) > > > > { > > > > + struct imx_sc_wdt_device *imx_sc_wdd; > > > > + struct watchdog_device *wdog; > > > > struct device *dev = &pdev->dev; > > > > - struct watchdog_device *imx_sc_wdd; > > > > int ret; > > > > > > > > imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), > > > > GFP_KERNEL); > > > @@ > > > > -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct > > > > platform_device > > > > *pdev) > > > > > > > > platform_set_drvdata(pdev, imx_sc_wdd); > > > > > > > > - imx_sc_wdd->info = &imx_sc_wdt_info; > > > > - imx_sc_wdd->ops = &imx_sc_wdt_ops; > > > > - imx_sc_wdd->min_timeout = 1; > > > > - imx_sc_wdd->max_timeout = MAX_TIMEOUT; > > > > - imx_sc_wdd->parent = dev; > > > > - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; > > > > + wdog = &imx_sc_wdd->wdd; > > > > + wdog->info = &imx_sc_wdt_info; > > > > + wdog->ops = &imx_sc_wdt_ops; > > > > + wdog->min_timeout = 1; > > > > + wdog->max_timeout = MAX_TIMEOUT; > > > > + wdog->parent = dev; > > > > + wdog->timeout = DEFAULT_TIMEOUT; > > > > > > > > - watchdog_init_timeout(imx_sc_wdd, 0, dev); > > > > - watchdog_stop_on_reboot(imx_sc_wdd); > > > > - watchdog_stop_on_unregister(imx_sc_wdd); > > > > + watchdog_init_timeout(wdog, 0, dev); > > > > + watchdog_stop_on_reboot(wdog); > > > > + watchdog_stop_on_unregister(wdog); > > > > > > > > - ret = devm_watchdog_register_device(dev, imx_sc_wdd); > > > > + ret = devm_watchdog_register_device(dev, wdog); > > > > if (ret) { > > > > dev_err(dev, "Failed to register watchdog device\n"); > > > > return ret; > > > > } > > > > > > > > + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > > + SC_IRQ_WDOG, > > > > + true); > > > > + if (ret) { > > > > + dev_warn(dev, "Enable irq failed, pretimeout NOT > > > supported\n"); > > > > + return 0; > > > > + } > > > > + > > > > + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; > > > > + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); > > > > + if (ret) { > > > > + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, > > > > + SC_IRQ_WDOG, > > > > + false); > > > > + dev_warn(dev, > > > > + "Register irq notifier failed, pretimeout NOT > > > supported\n"); > > > > + return 0; > > > > + } > > > > + > > > > + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, > > > > + &imx_sc_wdd->wdt_notifier); > > > > + if (!ret) > > > > + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; > > > > + else > > > > + dev_warn(dev, "Add action failed, pretimeout NOT > > > supported\n"); > > > > + > > > > return 0; > > > > } > > > > > > > > static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) > > > > { > > > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > > > > > - if (watchdog_active(imx_sc_wdd)) > > > > - imx_sc_wdt_stop(imx_sc_wdd); > > > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > > > + imx_sc_wdt_stop(&imx_sc_wdd->wdd); > > > > > > > > return 0; > > > > } > > > > > > > > static int __maybe_unused imx_sc_wdt_resume(struct device *dev) > > > > { > > > > - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); > > > > > > > > - if (watchdog_active(imx_sc_wdd)) > > > > - imx_sc_wdt_start(imx_sc_wdd); > > > > + if (watchdog_active(&imx_sc_wdd->wdd)) > > > > + imx_sc_wdt_start(&imx_sc_wdd->wdd); > > > > > > > > return 0; > > > > } > > > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support 2019-06-08 9:32 ` Anson Huang @ 2019-06-08 13:22 ` Guenter Roeck 0 siblings, 0 replies; 14+ messages in thread From: Guenter Roeck @ 2019-06-08 13:22 UTC (permalink / raw) To: Anson Huang Cc: robh+dt, mark.rutland, wim, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx On 6/8/19 2:32 AM, Anson Huang wrote: > Hi, Guenter > >> -----Original Message----- >> From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck >> Sent: Saturday, June 8, 2019 1:41 AM >> To: Anson Huang <anson.huang@nxp.com> >> Cc: robh+dt@kernel.org; mark.rutland@arm.com; wim@linux-watchdog.org; >> shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de; >> festevam@gmail.com; Aisheng Dong <aisheng.dong@nxp.com>; >> ulf.hansson@linaro.org; Daniel Baluta <daniel.baluta@nxp.com>; Peng Fan >> <peng.fan@nxp.com>; devicetree@vger.kernel.org; linux- >> kernel@vger.kernel.org; linux-watchdog@vger.kernel.org; linux-arm- >> kernel@lists.infradead.org; dl-linux-imx <linux-imx@nxp.com> >> Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout >> support >> >> On Wed, Jun 05, 2019 at 06:24:33AM +0000, Anson Huang wrote: >>> Hi, Guenter >>> >>>> -----Original Message----- >>>> From: Guenter Roeck <groeck7@gmail.com> On Behalf Of Guenter Roeck >>>> Sent: Sunday, May 12, 2019 9:28 PM >>>> To: Anson Huang <anson.huang@nxp.com>; robh+dt@kernel.org; >>>> mark.rutland@arm.com; wim@linux-watchdog.org; >> shawnguo@kernel.org; >>>> s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; >>>> Aisheng Dong <aisheng.dong@nxp.com>; ulf.hansson@linaro.org; Daniel >>>> Baluta <daniel.baluta@nxp.com>; Peng Fan <peng.fan@nxp.com>; >>>> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux- >>>> watchdog@vger.kernel.org; linux-arm-kernel@lists.infradead.org >>>> Cc: dl-linux-imx <linux-imx@nxp.com> >>>> Subject: Re: [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout >>>> support >>>> >>>> On 5/12/19 3:10 AM, Anson Huang wrote: >>>>> i.MX system controller watchdog can support pretimeout IRQ via >>>>> general SCU MU IRQ, it depends on IMX_SCU and driver MUST be >>>>> probed after SCU IPC ready, then enable corresponding SCU IRQ >>>>> group and register SCU IRQ notifier, when watchdog pretimeout IRQ >>>>> fires, SCU MU IRQ will be handled and watchdog pretimeout notifier >>>>> will be called to handle the event. >>>>> >>>>> Signed-off-by: Anson Huang <Anson.Huang@nxp.com> >>>> >>>> Revviewed-by: Guenter Roeck <linux@roeck-us.net> >>>> >>>> Other patches waiting for DT review. IMX API feedback below. >>> >>> Shaw just picked up the DT patch. Would you please pick up this driver >>> and dt-binding patch to you git repo? >>> >> I'll pick up patch 3/3, but I can not pick up patches into arch/arm64/boot/; >> that is the responsibility of arm64 maintainers. I can only do that if I get an >> explicit Ack and permission to do so from an arm64 maintainer, and I don't >> recall getting that. > > Will you also pick up 1/3, the DT binding patch, as DT binding normally go with > driver, if NOT, please advise who should pick up this patch. > Makes sense. Will do. Guenter > Thanks, > Anson. > >> >> Guenter >> >>> Thanks, >>> Anson >>> >>>> >>>> Side note: This patch depends on 'firmware: imx: enable imx scu >>>> general irq function' which is not yet in mainline. >>>> >>>>> --- >>>>> No change, just resend patch with correct encoding. >>>>> --- >>>>> drivers/watchdog/Kconfig | 1 + >>>>> drivers/watchdog/imx_sc_wdt.c | 116 >>>> +++++++++++++++++++++++++++++++++++------- >>>>> 2 files changed, 98 insertions(+), 19 deletions(-) >>>>> >>>>> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >>>>> index 7ea6037..e08238c 100644 >>>>> --- a/drivers/watchdog/Kconfig >>>>> +++ b/drivers/watchdog/Kconfig >>>>> @@ -716,6 +716,7 @@ config IMX2_WDT >>>>> config IMX_SC_WDT >>>>> tristate "IMX SC Watchdog" >>>>> depends on HAVE_ARM_SMCCC >>>>> + depends on IMX_SCU >>>>> select WATCHDOG_CORE >>>>> help >>>>> This is the driver for the system controller watchdog diff >>>>> --git a/drivers/watchdog/imx_sc_wdt.c >>>>> b/drivers/watchdog/imx_sc_wdt.c index 49848b6..6ecc03f 100644 >>>>> --- a/drivers/watchdog/imx_sc_wdt.c >>>>> +++ b/drivers/watchdog/imx_sc_wdt.c >>>>> @@ -4,6 +4,7 @@ >>>>> */ >>>>> >>>>> #include <linux/arm-smccc.h> >>>>> +#include <linux/firmware/imx/sci.h> >>>>> #include <linux/io.h> >>>>> #include <linux/init.h> >>>>> #include <linux/kernel.h> >>>>> @@ -33,11 +34,19 @@ >>>>> >>>>> #define SC_TIMER_WDOG_ACTION_PARTITION 0 >>>>> >>>>> +#define SC_IRQ_WDOG 1 >>>>> +#define SC_IRQ_GROUP_WDOG 1 >>>>> + >>>>> static bool nowayout = WATCHDOG_NOWAYOUT; >>>>> module_param(nowayout, bool, 0000); >>>>> MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once >>>> started (default=" >>>>> __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); >>>>> >>>>> +struct imx_sc_wdt_device { >>>>> + struct watchdog_device wdd; >>>>> + struct notifier_block wdt_notifier; }; >>>>> + >>>>> static int imx_sc_wdt_ping(struct watchdog_device *wdog) >>>>> { >>>>> struct arm_smccc_res res; >>>>> @@ -85,24 +94,66 @@ static int imx_sc_wdt_set_timeout(struct >>>> watchdog_device *wdog, >>>>> return res.a0 ? -EACCES : 0; >>>>> } >>>>> >>>>> +static int imx_sc_wdt_set_pretimeout(struct watchdog_device *wdog, >>>>> + unsigned int pretimeout) { >>>>> + struct arm_smccc_res res; >>>>> + >>>>> + arm_smccc_smc(IMX_SIP_TIMER, >>>> IMX_SIP_TIMER_SET_PRETIME_WDOG, >>>>> + pretimeout * 1000, 0, 0, 0, 0, 0, &res); >>>>> + if (res.a0) >>>>> + return -EACCES; >>>>> + >>>>> + wdog->pretimeout = pretimeout; >>>>> + >>>>> + return 0; >>>>> +} >>>>> + >>>>> +static int imx_sc_wdt_notify(struct notifier_block *nb, >>>>> + unsigned long event, void *group) { >>>>> + struct imx_sc_wdt_device *imx_sc_wdd = >>>>> + container_of(nb, >>>>> + struct imx_sc_wdt_device, >>>>> + wdt_notifier); >>>>> + >>>>> + if (event & SC_IRQ_WDOG && >>>>> + *(u8 *)group == SC_IRQ_GROUP_WDOG) >>>>> + watchdog_notify_pretimeout(&imx_sc_wdd->wdd); >>>> >>>> This should really not be necessary. Event mask and target group (if >>>> needed with a wildcard for the group) should be parameters of >>>> imx_scu_irq_register_notifier(), and be handled in the imx code. >>>> >>>> Also, passing 'group' as pointed seems excessive. Might as well pass >>>> it directly. >>>> >>>> Guenter >>>> >>>>> + >>>>> + return 0; >>>>> +} >>>>> + >>>>> +static void imx_sc_wdt_action(void *data) { >>>>> + struct notifier_block *wdt_notifier = data; >>>>> + >>>>> + imx_scu_irq_unregister_notifier(wdt_notifier); >>>>> + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, >>>>> + SC_IRQ_WDOG, >>>>> + false); >>>>> +} >>>>> + >>>>> static const struct watchdog_ops imx_sc_wdt_ops = { >>>>> .owner = THIS_MODULE, >>>>> .start = imx_sc_wdt_start, >>>>> .stop = imx_sc_wdt_stop, >>>>> .ping = imx_sc_wdt_ping, >>>>> .set_timeout = imx_sc_wdt_set_timeout, >>>>> + .set_pretimeout = imx_sc_wdt_set_pretimeout, >>>>> }; >>>>> >>>>> -static const struct watchdog_info imx_sc_wdt_info = { >>>>> +static struct watchdog_info imx_sc_wdt_info = { >>>>> .identity = "i.MX SC watchdog timer", >>>>> .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | >>>>> - WDIOF_MAGICCLOSE | WDIOF_PRETIMEOUT, >>>>> + WDIOF_MAGICCLOSE, >>>>> }; >>>>> >>>>> static int imx_sc_wdt_probe(struct platform_device *pdev) >>>>> { >>>>> + struct imx_sc_wdt_device *imx_sc_wdd; >>>>> + struct watchdog_device *wdog; >>>>> struct device *dev = &pdev->dev; >>>>> - struct watchdog_device *imx_sc_wdd; >>>>> int ret; >>>>> >>>>> imx_sc_wdd = devm_kzalloc(dev, sizeof(*imx_sc_wdd), >>>>> GFP_KERNEL); >>>> @@ >>>>> -111,42 +162,69 @@ static int imx_sc_wdt_probe(struct >>>>> platform_device >>>>> *pdev) >>>>> >>>>> platform_set_drvdata(pdev, imx_sc_wdd); >>>>> >>>>> - imx_sc_wdd->info = &imx_sc_wdt_info; >>>>> - imx_sc_wdd->ops = &imx_sc_wdt_ops; >>>>> - imx_sc_wdd->min_timeout = 1; >>>>> - imx_sc_wdd->max_timeout = MAX_TIMEOUT; >>>>> - imx_sc_wdd->parent = dev; >>>>> - imx_sc_wdd->timeout = DEFAULT_TIMEOUT; >>>>> + wdog = &imx_sc_wdd->wdd; >>>>> + wdog->info = &imx_sc_wdt_info; >>>>> + wdog->ops = &imx_sc_wdt_ops; >>>>> + wdog->min_timeout = 1; >>>>> + wdog->max_timeout = MAX_TIMEOUT; >>>>> + wdog->parent = dev; >>>>> + wdog->timeout = DEFAULT_TIMEOUT; >>>>> >>>>> - watchdog_init_timeout(imx_sc_wdd, 0, dev); >>>>> - watchdog_stop_on_reboot(imx_sc_wdd); >>>>> - watchdog_stop_on_unregister(imx_sc_wdd); >>>>> + watchdog_init_timeout(wdog, 0, dev); >>>>> + watchdog_stop_on_reboot(wdog); >>>>> + watchdog_stop_on_unregister(wdog); >>>>> >>>>> - ret = devm_watchdog_register_device(dev, imx_sc_wdd); >>>>> + ret = devm_watchdog_register_device(dev, wdog); >>>>> if (ret) { >>>>> dev_err(dev, "Failed to register watchdog device\n"); >>>>> return ret; >>>>> } >>>>> >>>>> + ret = imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, >>>>> + SC_IRQ_WDOG, >>>>> + true); >>>>> + if (ret) { >>>>> + dev_warn(dev, "Enable irq failed, pretimeout NOT >>>> supported\n"); >>>>> + return 0; >>>>> + } >>>>> + >>>>> + imx_sc_wdd->wdt_notifier.notifier_call = imx_sc_wdt_notify; >>>>> + ret = imx_scu_irq_register_notifier(&imx_sc_wdd->wdt_notifier); >>>>> + if (ret) { >>>>> + imx_scu_irq_group_enable(SC_IRQ_GROUP_WDOG, >>>>> + SC_IRQ_WDOG, >>>>> + false); >>>>> + dev_warn(dev, >>>>> + "Register irq notifier failed, pretimeout NOT >>>> supported\n"); >>>>> + return 0; >>>>> + } >>>>> + >>>>> + ret = devm_add_action_or_reset(dev, imx_sc_wdt_action, >>>>> + &imx_sc_wdd->wdt_notifier); >>>>> + if (!ret) >>>>> + imx_sc_wdt_info.options |= WDIOF_PRETIMEOUT; >>>>> + else >>>>> + dev_warn(dev, "Add action failed, pretimeout NOT >>>> supported\n"); >>>>> + >>>>> return 0; >>>>> } >>>>> >>>>> static int __maybe_unused imx_sc_wdt_suspend(struct device *dev) >>>>> { >>>>> - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); >>>>> + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); >>>>> >>>>> - if (watchdog_active(imx_sc_wdd)) >>>>> - imx_sc_wdt_stop(imx_sc_wdd); >>>>> + if (watchdog_active(&imx_sc_wdd->wdd)) >>>>> + imx_sc_wdt_stop(&imx_sc_wdd->wdd); >>>>> >>>>> return 0; >>>>> } >>>>> >>>>> static int __maybe_unused imx_sc_wdt_resume(struct device *dev) >>>>> { >>>>> - struct watchdog_device *imx_sc_wdd = dev_get_drvdata(dev); >>>>> + struct imx_sc_wdt_device *imx_sc_wdd = dev_get_drvdata(dev); >>>>> >>>>> - if (watchdog_active(imx_sc_wdd)) >>>>> - imx_sc_wdt_start(imx_sc_wdd); >>>>> + if (watchdog_active(&imx_sc_wdd->wdd)) >>>>> + imx_sc_wdt_start(&imx_sc_wdd->wdd); >>>>> >>>>> return 0; >>>>> } >>>>> >>> ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU 2019-05-12 10:10 [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support Anson Huang @ 2019-05-13 17:12 ` Rob Herring 2 siblings, 0 replies; 14+ messages in thread From: Rob Herring @ 2019-05-13 17:12 UTC (permalink / raw) To: Anson Huang Cc: mark.rutland, wim, linux, shawnguo, s.hauer, kernel, festevam, Aisheng Dong, ulf.hansson, Daniel Baluta, Peng Fan, devicetree, linux-kernel, linux-watchdog, linux-arm-kernel, dl-linux-imx On Sun, May 12, 2019 at 10:10:30AM +0000, Anson Huang wrote: > i.MX system controller watchdog depends on SCU driver to support > interrupt function, so it needs to be subnode of SCU node in DT, > binding doc should be moved to fsl,scu.txt as well. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > No change, just resend patch with correct encoding. Quoted-printable isn't really correct either. It should be 7-bit or 8-bit if UTF-8. > --- > .../devicetree/bindings/arm/freescale/fsl,scu.txt | 15 ++++++++++++++ > .../bindings/watchdog/fsl-imx-sc-wdt.txt | 24 ---------------------- > 2 files changed, 15 insertions(+), 24 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/watchdog/fsl-imx-sc-wdt.txt Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-06-08 13:22 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-05-12 10:10 [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Anson Huang 2019-05-12 10:10 ` [PATCH RESEND V4 2/3] arm64: dts: imx8qxp: Move watchdog node into scu node Anson Huang 2019-06-04 9:06 ` Anson Huang 2019-06-05 6:00 ` Shawn Guo 2019-06-05 6:09 ` Anson Huang 2019-06-05 6:12 ` Shawn Guo 2019-05-12 10:10 ` [PATCH RESEND V4 3/3] watchdog: imx_sc: Add pretimeout support Anson Huang 2019-05-12 13:28 ` Guenter Roeck 2019-05-12 14:10 ` Anson Huang 2019-06-05 6:24 ` Anson Huang 2019-06-07 17:40 ` Guenter Roeck 2019-06-08 9:32 ` Anson Huang 2019-06-08 13:22 ` Guenter Roeck 2019-05-13 17:12 ` [PATCH RESEND V4 1/3] dt-bindings: watchdog: move i.MX system controller watchdog binding to SCU Rob Herring
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).