On Tue, Oct 10, 2023 at 03:46:26PM +0800, Jacky Bai wrote: > if the wdog is already enabled, and try to enabled it again, > we should ignore the error and continue, rather than return > error. > > Signed-off-by: Jacky Bai <ping.bai@xxxxxxx> > Reviewed-by: Peng Fan <peng.fan@xxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/watchdog/imx_sc_wdt.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c > index 8ac021748d16..e51fe1b78518 100644 > --- a/drivers/watchdog/imx_sc_wdt.c > +++ b/drivers/watchdog/imx_sc_wdt.c > @@ -34,6 +34,7 @@ > > #define SC_IRQ_WDOG 1 > #define SC_IRQ_GROUP_WDOG 1 > +#define SC_TIMER_ERR_BUSY 10 > > static bool nowayout = WATCHDOG_NOWAYOUT; > module_param(nowayout, bool, 0000); > @@ -61,7 +62,9 @@ static int imx_sc_wdt_start(struct watchdog_device *wdog) > > arm_smccc_smc(IMX_SIP_TIMER, IMX_SIP_TIMER_START_WDOG, > 0, 0, 0, 0, 0, 0, &res); > - if (res.a0) > + > + /* Ignore if already enabled(SC_TIMER_ERR_BUSY) */ > + if (res.a0 && res.a0 != SC_TIMER_ERR_BUSY) > return -EACCES; > > arm_smccc_smc(IMX_SIP_TIMER, IMX_SIP_TIMER_SET_WDOG_ACT, > -- > 2.34.1 >
