[PATCH v2] mt7621: move uboot-envtools to DEFAULT_PACKAGES

[Bjørn Mork]

Hauke Mehrtens <hauke at hauke-m.de> writes:

> On 2/27/23 13:38, Bjørn Mork wrote:
>> Several devices depend on fw_printenv during sysupgrade.  Make sure
>> it always is present in all images, including initramfs images built
>> by the buildbots.
>> Fixes: 2449a632084b ("ramips: mt7621: Add support for ZyXEL NR7101")
>> Signed-off-by: Bjørn Mork <bjorn at mork.no>
>> ---
>> Changes since v1:
>>   - rebased onto current master
>>   target/linux/ramips/image/mt7621.mk  | 68
>> ++++++++++++----------------
>>   target/linux/ramips/mt7621/target.mk |  2 +-
>>   2 files changed, 29 insertions(+), 41 deletions(-)
>> 
> .....
>> diff --git a/target/linux/ramips/mt7621/target.mk b/target/linux/ramips/mt7621/target.mk
>> index cfb798e35852..153ff08421d6 100644
>> --- a/target/linux/ramips/mt7621/target.mk
>> +++ b/target/linux/ramips/mt7621/target.mk
>> @@ -10,7 +10,7 @@ KERNELNAME:=vmlinux vmlinuz
>>   # make Kernel/CopyImage use $LINUX_DIR/vmlinuz
>>   IMAGES_DIR:=../../..
>>   -DEFAULT_PACKAGES += wpad-basic-mbedtls
>> +DEFAULT_PACKAGES += wpad-basic-mbedtls uboot-envtools
>>     define Target/Description
>>   	Build firmware images for Ralink MT7621 based boards.
>
> This will add uboot-envtools to all devices. uboot-envtools is not
> included in all DEVICE_PACKAGES now, should we explicitly remove it
> from device definitions which do not had it before?
>
> The Device/adslr_g7 for example does not add uboot-envtools in its
> DEVICE_PACKAGES.

Good point.  I'll update the patch


Bjørn