[PATCH] cmd: Make use of U_BOOT_LONGHELP when missing
Tom Rini
trini at konsulko.com
Wed Jun 19 18:15:43 CEST 2024
On Wed, Jun 19, 2024 at 08:50:09AM -0600, Tom Rini wrote:
> On Wed, Jun 19, 2024 at 08:15:04AM +0200, Michal Simek wrote:
> > út 18. 6. 2024 v 19:45 odesílatel Tom Rini <trini at konsulko.com> napsal:
> > >
> > > After adding the U_BOOT_LONGHELP macro some new commands came in still
> > > that were not making use if it. Switch these cases over and in a few
> > > places add missing newlines as well.
> > >
> > > Signed-off-by: Tom Rini <trini at konsulko.com>
> > > ---
> > > arch/arm/mach-imx/cmd_dek.c | 4 ++--
> > > arch/arm/mach-imx/cmd_mfgprot.c | 4 ++--
> > > arch/arm/mach-imx/imx8/snvs_security_sc.c | 16 ++++++++--------
> > > arch/arm/mach-stm32mp/cmd_stm32key.c | 4 ++--
> > > board/freescale/common/cmd_esbc_validate.c | 4 ++--
> > > board/kontron/sl28/cmds.c | 4 ++--
> > > cmd/adc.c | 4 ++--
> > > cmd/arm/exception.c | 5 ++---
> > > cmd/arm/exception64.c | 5 ++---
> > > cmd/blob.c | 4 ++--
> > > cmd/riscv/exception.c | 5 ++---
> > > cmd/scmi.c | 5 ++---
> > > cmd/x86/exception.c | 5 ++---
> > > 13 files changed, 32 insertions(+), 37 deletions(-)
> [snip]
> > arch/arm/mach-imx/imx8/snvs_security_sc.c:781:static char
> > snvs_sec_status_help_text[] =
>
> I missed this one for some reason, yeah.
>
> > board/xilinx/versal-net/cmds.c:74:static char versalnet_help_text[] =
> >
> > These two should also be part of this patch.
>
> OK.
>
> > And this one is interesting.
> > cmd/cli.c:122:static char cli_help_text[] =
> >
> > it is guarded by SYS_LONGHELP but I think it should be just removed
> > and macro should be used instead.
> >
> > #if CONFIG_IS_ENABLED(SYS_LONGHELP)
> > static char cli_help_text[] =
> > "get - print current cli\n"
> > "set - set the current cli, possible value are: old, modern"
> > ;
> > #endif
> >
> > U_BOOT_CMD(cli, 3, 1, do_cli,
> > "cli",
> > #if CONFIG_IS_ENABLED(SYS_LONGHELP)
> > cli_help_text
> > #endif
> > );
>
> This one is actually kind of "funny". I know I changed it to that,
> rather than using the macro like everything else, when I first did the
> conversion as there was some sort of odd build failure introduced. I'll
> go and see if it still fails somewhere.
Welp, world build passed, and sandbox with and without LONGHELP set
passed, so, either I made a mistake back originally or something else
changed and it's fine now.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240619/c53dcb77/attachment.sig>
More information about the U-Boot
mailing list