Hi Sean, On Fri, 9 Apr 2021 at 04:32, Sean Anderson <sean...@gmail.com> wrote: > > On 4/6/21 12:30 AM, Simon Glass wrote: > > This text should never change during execution, so it makes sense to > > use a const char * so that it can be declared as const in the code. > > Update struct cmd_tbl with a const char * pointer for 'help'. > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > --- > > > > include/command.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/include/command.h b/include/command.h > > index 747f8f80958..675c16a1249 100644 > > --- a/include/command.h > > +++ b/include/command.h > > @@ -45,7 +45,7 @@ struct cmd_tbl { > > char *const argv[]); > > char *usage; /* Usage message (short) */ > > Should usage also be const?
I think so, but would need to check it builds, etc. Regards, Simon