From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rasmus Villemoes Date: Mon, 12 Oct 2020 09:06:00 +0200 Subject: [PATCH v2 2/3] allow positional arguments with "run" command In-Reply-To: References: <20200925111942.4629-1-rasmus.villemoes@prevas.dk> <20201007072052.28200-1-rasmus.villemoes@prevas.dk> <20201007072052.28200-3-rasmus.villemoes@prevas.dk> Message-ID: <7f0c6b5c-b105-c006-fccb-f7ca00d3bf50@prevas.dk> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 12/10/2020 05.34, Simon Glass wrote: > On Wed, 7 Oct 2020 at 01:21, Rasmus Villemoes > 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 > > 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