[PATCH v2 2/3] allow positional arguments with "run" command

Rasmus Villemoes rasmus.villemoes at prevas.dk
Mon Oct 12 09:06:00 CEST 2020


On 12/10/2020 05.34, Simon Glass wrote:
> On Wed, 7 Oct 2020 at 01:21, Rasmus Villemoes
> <rasmus.villemoes at prevas.dk> wrote:
>>
>>  cmd/Kconfig        | 10 ++++++++++
>>  cmd/nvedit.c       |  7 ++++++-
>>  common/cli.c       | 44 ++++++++++++++++++++++++++++++++++++++------
>>  common/cli_hush.c  | 32 +++++++++++++++++++++++++++++++-
>>  include/cli_hush.h |  9 +++++++++
>>  5 files changed, 94 insertions(+), 8 deletions(-)
>>
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> I'm not sure where the previous discussion went. But please think
> about how we can add some tests here.

Isn't that exactly what I do in 3/3? Or are you thinking of something else?

Rasmus


More information about the U-Boot mailing list