[PATCH 1/2] command.h: Add a U_BOOT_LONGHELP macro
Simon Glass
sjg at google.com
Sun Oct 15 16:39:44 CEST 2023
Hi Tom,
On Sat, 7 Oct 2023 at 13:13, Tom Rini <trini at konsulko.com> wrote:
>
> In order to be able to discard unused long help texts without further
> linker lists, add a macro for defining the long help messages which uses
> __maybe_unused. This allows us to discard them as unreferenced as part
> of the link.
>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> include/command.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/include/command.h b/include/command.h
> index 34ea989b39b4..a836ea795ab6 100644
> --- a/include/command.h
> +++ b/include/command.h
> @@ -322,6 +322,9 @@ int cmd_source_script(ulong addr, const char *fit_uname, const char *confname);
> # define _CMD_HELP(x)
> #endif
>
> +#define U_BOOT_LONGHELP(_cmdname, text) \
> + static __maybe_unused const char _cmdname##_help_text[] = text;
I just noticed this...do you really want the ; at the end? The only
purpose I can see is if you want the macro to compile to nothing.
Otherwise it is more normal to put it on the invocations, isn't it?
> +
> #define U_BOOT_SUBCMDS_DO_CMD(_cmdname) \
> static int do_##_cmdname(struct cmd_tbl *cmdtp, int flag, \
> int argc, char *const argv[], \
> --
> 2.34.1
>
Regards,
Simon
More information about the U-Boot
mailing list