diff mbox

watchdog: renesas-wdt: Remove R-Car M2-W ES2.x from blacklist

Message ID 1524226849-13301-1-git-send-email-geert+renesas@glider.be (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Geert Uytterhoeven April 20, 2018, 12:20 p.m. UTC
System restart triggered by watchdog time-out works fine on a Koelsch
board with R-Car M2-W ES2.0.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Thanks to Magnus and Shimoda-san for providing board access!
---
 drivers/watchdog/renesas_wdt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Fabrizio Castro April 20, 2018, 12:45 p.m. UTC | #1
> Subject: [PATCH] watchdog: renesas-wdt: Remove R-Car M2-W ES2.x from blacklist
>
> System restart triggered by watchdog time-out works fine on a Koelsch
> board with R-Car M2-W ES2.0.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>

> ---
> Thanks to Magnus and Shimoda-san for providing board access!
> ---
>  drivers/watchdog/renesas_wdt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/watchdog/renesas_wdt.c b/drivers/watchdog/renesas_wdt.c
> index 514db5cc15951125..88d81feba4e60087 100644
> --- a/drivers/watchdog/renesas_wdt.c
> +++ b/drivers/watchdog/renesas_wdt.c
> @@ -146,7 +146,7 @@ static const struct soc_device_attribute rwdt_quirks_match[] = {
>  .data = (void *)1,/* needs single CPU */
>  }, {
>  .soc_id = "r8a7791",
> -.revision = "ES[12].*",
> +.revision = "ES1.*",
>  .data = (void *)1,/* needs single CPU */
>  }, {
>  .soc_id = "r8a7792",
> --
> 2.7.4




Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Guenter Roeck April 20, 2018, 1:12 p.m. UTC | #2
On 04/20/2018 05:20 AM, Geert Uytterhoeven wrote:
> System restart triggered by watchdog time-out works fine on a Koelsch
> board with R-Car M2-W ES2.0.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
> Thanks to Magnus and Shimoda-san for providing board access!
> ---
>   drivers/watchdog/renesas_wdt.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/watchdog/renesas_wdt.c b/drivers/watchdog/renesas_wdt.c
> index 514db5cc15951125..88d81feba4e60087 100644
> --- a/drivers/watchdog/renesas_wdt.c
> +++ b/drivers/watchdog/renesas_wdt.c
> @@ -146,7 +146,7 @@ static const struct soc_device_attribute rwdt_quirks_match[] = {
>   		.data = (void *)1,	/* needs single CPU */
>   	}, {
>   		.soc_id = "r8a7791",
> -		.revision = "ES[12].*",
> +		.revision = "ES1.*",
>   		.data = (void *)1,	/* needs single CPU */
>   	}, {
>   		.soc_id = "r8a7792",
>
Simon Horman April 23, 2018, 10:37 a.m. UTC | #3
On Fri, Apr 20, 2018 at 02:20:49PM +0200, Geert Uytterhoeven wrote:
> System restart triggered by watchdog time-out works fine on a Koelsch
> board with R-Car M2-W ES2.0.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
diff mbox

Patch

diff --git a/drivers/watchdog/renesas_wdt.c b/drivers/watchdog/renesas_wdt.c
index 514db5cc15951125..88d81feba4e60087 100644
--- a/drivers/watchdog/renesas_wdt.c
+++ b/drivers/watchdog/renesas_wdt.c
@@ -146,7 +146,7 @@  static const struct soc_device_attribute rwdt_quirks_match[] = {
 		.data = (void *)1,	/* needs single CPU */
 	}, {
 		.soc_id = "r8a7791",
-		.revision = "ES[12].*",
+		.revision = "ES1.*",
 		.data = (void *)1,	/* needs single CPU */
 	}, {
 		.soc_id = "r8a7792",