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

[Alex Kiernan]

On Tue, Mar 27, 2018 at 10:28 AM, Quentin Schulz
<quentin.schulz at bootlin.com> wrote:
> Hi Alex,
>
> On Tue, Mar 27, 2018 at 08:43:26AM +0000, Alex Kiernan wrote:
>> When importing variables allow size to be elided using '-' and then
>> support a list of variables which restricts what will be picked during
>> the import.
>>
>> Signed-off-by: Alex Kiernan <alex.kiernan at gmail.com>
>
> I'm pretty sure it's the same goal as this patch[1] I suggested.

It is, maybe it was your message I was thinking of when I asked the
question the other day:

https://lists.denx.de/pipermail/u-boot/2018-March/323687.html

> Could you answer in the thread telling you need it as well so that we
> could get it merged or at least reviewed?
>

Assuming I've understood your patch correctly, I think I can replicate
your use case with this:

  env import ... ${whitelisted_vars}

I've two uses for this right now for this - with different white lists:

  # override defaults from uboot.env
  if fatload mmc ${mmcdev} ${loadaddr} uboot.env; then
          env import -c ${loadaddr} ${filesize} serial# ethaddr
  fi

  # source OSTree deployments
  if load ${devtype} ${bootpart} ${loadaddr} /boot/loader/uEnv.txt; then
          env import -t ${loadaddr} ${filesize} kernel_image
kernel_image2 bootargs bootargs2
  fi

-- 
Alex Kiernan