[U-Boot] [PATCH] cmd: nvedit: Add filtering during env import

Alex Kiernan alex.kiernan at gmail.com
Tue Mar 27 16:06:43 UTC 2018


On Tue, Mar 27, 2018 at 2:30 PM, Quentin Schulz
<quentin.schulz at bootlin.com> wrote:

> What I'm more concerned about is that after we've merged your patch,
> it isn't possible to pass multiple parameters at the end of the command.
> There might be (I have no example as of today) some things we want to do
> in the future that can't be done an other way than passing a list of
> arguments at the end of the command and your patch prevent doing so in
> the future.
>

Fair comment... I was really just following what was already there for
env export (so much so I could cut'n'paste the documentation changes).

> We can do it now with a single option (-w or whatever letter we want to
> use) so I guess it's better to leave the option for cases that really
> require to have a list of parameters passed to the command.
>

I really don't like magic variables that get used invisibly, though I
guess you could pass a variable in.

-- 
Alex Kiernan


More information about the U-Boot mailing list