* [PATCH] cmd: add a panic command
@ 2020-06-26 11:36 Heiko Stuebner
2020-06-29 17:26 ` Simon Glass
0 siblings, 1 reply; 2+ messages in thread
From: Heiko Stuebner @ 2020-06-26 11:36 UTC (permalink / raw)
To: u-boot
From: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>
Even in boot scripts it may be needed to "panic" when all options
are exhausted and the device specification specifies hanging
instead of resetting the board.
So add a new panic command that just wraps around the core panic
call in uboot and can take an optional message.
Signed-off-by: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>
---
cmd/Makefile | 1 +
cmd/panic.c | 22 ++++++++++++++++++++++
2 files changed, 23 insertions(+)
create mode 100644 cmd/panic.c
diff --git a/cmd/Makefile b/cmd/Makefile
index ac843b4b16..027fa9083a 100644
--- a/cmd/Makefile
+++ b/cmd/Makefile
@@ -8,6 +8,7 @@ ifndef CONFIG_SPL_BUILD
obj-y += boot.o
obj-$(CONFIG_CMD_BOOTM) += bootm.o
obj-y += help.o
+obj-y += panic.o
obj-y += version.o
# command
diff --git a/cmd/panic.c b/cmd/panic.c
new file mode 100644
index 0000000000..696b4c73a3
--- /dev/null
+++ b/cmd/panic.c
@@ -0,0 +1,22 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (c) 2020 Theobroma Systems Design und Consulting GmbH
+ */
+
+#include <common.h>
+#include <command.h>
+
+static int do_panic(cmd_tbl_t *cmdtp, int flag, int argc,
+ char * const argv[])
+{
+ char *text = (argc < 2) ? "" : argv[1];
+
+ panic(text);
+ return CMD_RET_SUCCESS;
+}
+
+U_BOOT_CMD(
+ panic, 2, 1, do_panic,
+ "Panic with optional message",
+ "[message]"
+);
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] cmd: add a panic command
2020-06-26 11:36 [PATCH] cmd: add a panic command Heiko Stuebner
@ 2020-06-29 17:26 ` Simon Glass
0 siblings, 0 replies; 2+ messages in thread
From: Simon Glass @ 2020-06-29 17:26 UTC (permalink / raw)
To: u-boot
On Fri, 26 Jun 2020 at 05:36, Heiko Stuebner <heiko@sntech.de> wrote:
>
> From: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>
>
> Even in boot scripts it may be needed to "panic" when all options
> are exhausted and the device specification specifies hanging
> instead of resetting the board.
>
> So add a new panic command that just wraps around the core panic
> call in uboot and can take an optional message.
U-Boot
>
> Signed-off-by: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>
> ---
> cmd/Makefile | 1 +
> cmd/panic.c | 22 ++++++++++++++++++++++
> 2 files changed, 23 insertions(+)
> create mode 100644 cmd/panic.c
Reviewed-by: Simon Glass <sjg@chromium.org>
nit below
>
> diff --git a/cmd/Makefile b/cmd/Makefile
> index ac843b4b16..027fa9083a 100644
> --- a/cmd/Makefile
> +++ b/cmd/Makefile
> @@ -8,6 +8,7 @@ ifndef CONFIG_SPL_BUILD
> obj-y += boot.o
> obj-$(CONFIG_CMD_BOOTM) += bootm.o
> obj-y += help.o
> +obj-y += panic.o
> obj-y += version.o
>
> # command
> diff --git a/cmd/panic.c b/cmd/panic.c
> new file mode 100644
> index 0000000000..696b4c73a3
> --- /dev/null
> +++ b/cmd/panic.c
> @@ -0,0 +1,22 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright (c) 2020 Theobroma Systems Design und Consulting GmbH
> + */
> +
> +#include <common.h>
> +#include <command.h>
> +
> +static int do_panic(cmd_tbl_t *cmdtp, int flag, int argc,
> + char * const argv[])
> +{
> + char *text = (argc < 2) ? "" : argv[1];
> +
> + panic(text);
blank line here
> + return CMD_RET_SUCCESS;
> +}
> +
> +U_BOOT_CMD(
> + panic, 2, 1, do_panic,
> + "Panic with optional message",
> + "[message]"
> +);
> --
> 2.26.2
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-06-29 17:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-26 11:36 [PATCH] cmd: add a panic command Heiko Stuebner
2020-06-29 17:26 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.