mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] commands: wd: add new -x option to inhibit all watchdogs
@ 2022-01-19 14:08 Ahmad Fatoum
  2022-01-20  8:57 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2022-01-19 14:08 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum, Michael Olbrich, Yann Sionneau, Jules Maselbas

When booting after nv.autoboot=abort, barebox will drop to shell and
try to inhibit all watchdogs: If they can be disabled, they will be and
otherwise if autopoll support is compiled in, they will be periodically
fed. This is useful for development. For this to work though, barebox
needs an environment. Make watchdog_inhibit_all more accessible by
making it directly invocable with wd -x.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
Cc: Michael Olbrich <mol@pengutronix.de>
Cc: Yann Sionneau <ysionneau@kalray.eu>
Cc: Jules Maselbas <jmaselbas@kalray.eu>
---
 commands/wd.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/commands/wd.c b/commands/wd.c
index c186244b2b3a..11f888133aac 100644
--- a/commands/wd.c
+++ b/commands/wd.c
@@ -18,11 +18,13 @@ static int do_wd(int argc, char *argv[])
 	int opt;
 	int rc;
 
-	while ((opt = getopt(argc, argv, "d:")) > 0) {
+	while ((opt = getopt(argc, argv, "d:x")) > 0) {
 		switch (opt) {
 		case 'd':
 			wd = watchdog_get_by_name(optarg);
 			break;
+		case 'x':
+			return watchdog_inhibit_all();
 		default:
 			return COMMAND_ERROR_USAGE;
 		}
@@ -66,12 +68,13 @@ BAREBOX_CMD_HELP_TEXT("When TIME is 0, the watchdog gets disabled,")
 BAREBOX_CMD_HELP_TEXT("Without a parameter the watchdog will be re-triggered.")
 BAREBOX_CMD_HELP_TEXT("Options:")
 BAREBOX_CMD_HELP_OPT("-d DEVICE\t", "watchdog name (default is highest priority watchdog)")
+BAREBOX_CMD_HELP_OPT("-x\t", "inhibit all watchdogs (i.e. disable or autopoll if possible)")
 BAREBOX_CMD_HELP_END
 
 BAREBOX_CMD_START(wd)
 	.cmd = do_wd,
 	BAREBOX_CMD_DESC("enable/disable/trigger the watchdog")
-	BAREBOX_CMD_OPTS("[-d DEVICE] [TIME]")
+	BAREBOX_CMD_OPTS("[-d DEVICE] [-x] [TIME]")
 	BAREBOX_CMD_GROUP(CMD_GRP_HWMANIP)
 	BAREBOX_CMD_HELP(cmd_wd_help)
 	BAREBOX_CMD_COMPLETE(device_complete)
-- 
2.30.2


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] commands: wd: add new -x option to inhibit all watchdogs
  2022-01-19 14:08 [PATCH] commands: wd: add new -x option to inhibit all watchdogs Ahmad Fatoum
@ 2022-01-20  8:57 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2022-01-20  8:57 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox, Michael Olbrich, Yann Sionneau, Jules Maselbas

On Wed, Jan 19, 2022 at 03:08:16PM +0100, Ahmad Fatoum wrote:
> When booting after nv.autoboot=abort, barebox will drop to shell and
> try to inhibit all watchdogs: If they can be disabled, they will be and
> otherwise if autopoll support is compiled in, they will be periodically
> fed. This is useful for development. For this to work though, barebox
> needs an environment. Make watchdog_inhibit_all more accessible by
> making it directly invocable with wd -x.
> 
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
> Cc: Michael Olbrich <mol@pengutronix.de>
> Cc: Yann Sionneau <ysionneau@kalray.eu>
> Cc: Jules Maselbas <jmaselbas@kalray.eu>
> ---
>  commands/wd.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)

Applied, thanks

Sascha


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-01-20  8:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-19 14:08 [PATCH] commands: wd: add new -x option to inhibit all watchdogs Ahmad Fatoum
2022-01-20  8:57 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox