* Kalle Valo <kvalo@xxxxxxxxxxxxxx> [170703 04:30]: > "Reizer, Eyal" <eyalr@xxxxxx> writes: > > > When working with wl18xx the nvs file is used for defining an alternate > > mac address and override the default mac address that is stored inside > > the wl18xx chip. > > update the structure field with the same default nvs file name that has > > been used in the past, otherwise userspace backward compatibility is > > broken when upgrading from older kernels, as the alternate mac address > > would not be read from the nvs that is already present in the file system > > (lib/firmware/ti-connectivity/wl1271-nvs.bin) causing mac address change > > of the wlan interface. > > > > Signed-off-by: Eyal Reizer <eyalr@xxxxxx> > > Should we also cc stable? And a Fixes line would be nice. Argh this mess again. I think there are few things to consider here. What about booting the same rootfs on multiple devices for example with NFSroot? Not sure if this really is a regression as we've always had a bogus wl1271-nvs.bin in linux-firmware.git. Sure would be nice to fix it, but going back to using a generic wl1271-nvs.bin sure does not seem like a good long term fix to me :) Isn't the nvs file supposed to be device specific? If so, we should not provide it in linux-firmware.git at all because of the issues it causes.. And since it's provided, how are people supposed to know to remove it from their file system and replace it with something board specific? Maybe add a check to first try to find wl18xx-nvs.bin if it exists (and not add it to linux-firmware.git), and if not found, then fall back to wl1271-nvs.bin, but only if it's not the bogus generic file.. Produce a warning if the bogux linux-firmware.git wl1271-nvs.bin is being used.. Regards, Tony