Re: [v5] wlcore: add missing nvs file name info for wilink8
* Reizer, Eyal[170809 00:26]: > Managed to test with a wilink6 module and in fact reading hardware mac > Address from fuse is working ok for wilink6/7 as well. > submitting v6 using this mac address instead of a random one when the > bogus (deadbeef...) mac address is read from default nvs file. Hey great, that should help clear quite a bit of the wl12xx confusion with distros! Thanks for doing that. Regards, Tony
RE: [v5] wlcore: add missing nvs file name info for wilink8
> > > Subject: Re: [v5] wlcore: add missing nvs file name info for wilink8 > > > > * Reizer, Eyal <ey...@ti.com> [170807 00:47]: > > > Hi Tony, > > > > > > > > * Reizer, Eyal <ey...@ti.com> [170807 00:32]: > > > > > The following commits: > > > > > c815fde wlcore: spi: Populate config firmware data > > > > > d776fc8 wlcore: sdio: Populate config firmware data > > > > > > > > > > Populated the nvs entry for wilink6 and wilink7 only while it is > > > > > still needed for wilink8 as well. > > > > > This broke user space backward compatibility when upgrading from > > older > > > > > kernels, as the alternate mac address would not be read from the nvs > > that > > > > is > > > > > present in the file system (lib/firmware/ti-connectivity/wl1271- > nvs.bin) > > > > > causing mac address change of the wlan interface. > > > > > > > > > > This patch fix this and update the structure field with the same > default > > > > > nvs file name that has been used before. > > > > > > > > > > In addition, some distros hold a default wl1271-nvs.bin in the file > > > > > system with a bogus mac address (deadbeef...) that for a wl18xx > device > > > > > also overrides the mac address that is stored inside the device. > > > > > Warn users about this bogus mac address and use a random mac > > instead > > > > > > > > Hmm looks pretty good to me except for one more thing I just noticed. > > > > > > > > Why don't you just use the hardware mac address instead of a random > > > > mac address on wl18xx device when you see a bogus nvs file? > > > > > > > > > > I agree that this would have been better but the problem is that > hardware > > > mac address is available for wilink8 onlyWilink6/7 don't have one stored. > > > The wlcore code responsible for handling mac address is common code > > > and there is method for detecting between them in this module. > > > > Care to clarify a bit.. Are you saying wilink8 will use the hardware > > mac address in case of bogus nvs file? > > > With present implementation it will not. It will use the random one for both > wilink6/7 as well as wilink8. > I need to get a hold of a wilink6/7 module and see if reverting to an internal > mac address is working. Will try to look around as it has been a while since > I used one. > Managed to test with a wilink6 module and in fact reading hardware mac Address from fuse is working ok for wilink6/7 as well. submitting v6 using this mac address instead of a random one when the bogus (deadbeef...) mac address is read from default nvs file. Best Regards, Eyal
RE: [v5] wlcore: add missing nvs file name info for wilink8
> Subject: Re: [v5] wlcore: add missing nvs file name info for wilink8 > > * Reizer, Eyal <ey...@ti.com> [170807 00:47]: > > Hi Tony, > > > > > > * Reizer, Eyal <ey...@ti.com> [170807 00:32]: > > > > The following commits: > > > > c815fde wlcore: spi: Populate config firmware data > > > > d776fc8 wlcore: sdio: Populate config firmware data > > > > > > > > Populated the nvs entry for wilink6 and wilink7 only while it is > > > > still needed for wilink8 as well. > > > > This broke user space backward compatibility when upgrading from > older > > > > kernels, as the alternate mac address would not be read from the nvs > that > > > is > > > > present in the file system (lib/firmware/ti-connectivity/wl1271-nvs.bin) > > > > causing mac address change of the wlan interface. > > > > > > > > This patch fix this and update the structure field with the same default > > > > nvs file name that has been used before. > > > > > > > > In addition, some distros hold a default wl1271-nvs.bin in the file > > > > system with a bogus mac address (deadbeef...) that for a wl18xx device > > > > also overrides the mac address that is stored inside the device. > > > > Warn users about this bogus mac address and use a random mac > instead > > > > > > Hmm looks pretty good to me except for one more thing I just noticed. > > > > > > Why don't you just use the hardware mac address instead of a random > > > mac address on wl18xx device when you see a bogus nvs file? > > > > > > > I agree that this would have been better but the problem is that hardware > > mac address is available for wilink8 onlyWilink6/7 don't have one stored. > > The wlcore code responsible for handling mac address is common code > > and there is method for detecting between them in this module. > > Care to clarify a bit.. Are you saying wilink8 will use the hardware > mac address in case of bogus nvs file? > With present implementation it will not. It will use the random one for both wilink6/7 as well as wilink8. I need to get a hold of a wilink6/7 module and see if reverting to an internal mac address is working. Will try to look around as it has been a while since I used one. Best Regards, Eyal
Re: [v5] wlcore: add missing nvs file name info for wilink8
* Reizer, Eyal[170807 00:47]: > Hi Tony, > > > > * Reizer, Eyal [170807 00:32]: > > > The following commits: > > > c815fde wlcore: spi: Populate config firmware data > > > d776fc8 wlcore: sdio: Populate config firmware data > > > > > > Populated the nvs entry for wilink6 and wilink7 only while it is > > > still needed for wilink8 as well. > > > This broke user space backward compatibility when upgrading from older > > > kernels, as the alternate mac address would not be read from the nvs that > > is > > > present in the file system (lib/firmware/ti-connectivity/wl1271-nvs.bin) > > > causing mac address change of the wlan interface. > > > > > > This patch fix this and update the structure field with the same default > > > nvs file name that has been used before. > > > > > > In addition, some distros hold a default wl1271-nvs.bin in the file > > > system with a bogus mac address (deadbeef...) that for a wl18xx device > > > also overrides the mac address that is stored inside the device. > > > Warn users about this bogus mac address and use a random mac instead > > > > Hmm looks pretty good to me except for one more thing I just noticed. > > > > Why don't you just use the hardware mac address instead of a random > > mac address on wl18xx device when you see a bogus nvs file? > > > > I agree that this would have been better but the problem is that hardware > mac address is available for wilink8 onlyWilink6/7 don't have one stored. > The wlcore code responsible for handling mac address is common code > and there is method for detecting between them in this module. Care to clarify a bit.. Are you saying wilink8 will use the hardware mac address in case of bogus nvs file? Regards, Tony
Re: [v5] wlcore: add missing nvs file name info for wilink8
"Reizer, Eyal"writes: > The following commits: > c815fde wlcore: spi: Populate config firmware data > d776fc8 wlcore: sdio: Populate config firmware data It's recommended to use 12 chars when abbreviating commit ids so the format should be this: c815fdebef44 wlcore: spi: Populate config firmware data d776fc86b82f wlcore: sdio: Populate config firmware data > Populated the nvs entry for wilink6 and wilink7 only while it is > still needed for wilink8 as well. > This broke user space backward compatibility when upgrading from older > kernels, as the alternate mac address would not be read from the nvs that is > present in the file system (lib/firmware/ti-connectivity/wl1271-nvs.bin) > causing mac address change of the wlan interface. > > This patch fix this and update the structure field with the same default > nvs file name that has been used before. > > In addition, some distros hold a default wl1271-nvs.bin in the file > system with a bogus mac address (deadbeef...) that for a wl18xx device > also overrides the mac address that is stored inside the device. > Warn users about this bogus mac address and use a random mac instead > > Cc: stable > Signed-off-by: Eyal Reizer It's always good to have a fixes line(s) and specify which stable versions should have the fix. As the commit were first in v4.9-rc1 that's the first version we want this to be included: Fixes: c815fdebef44 ("wlcore: spi: Populate config firmware data") Fixes: d776fc86b82f ("wlcore: sdio: Populate config firmware data") Cc: # 4.9+ But if you don't submit a new version I can fix the commit log during commit. -- Kalle Valo
RE: [v5] wlcore: add missing nvs file name info for wilink8
Hi Tony, > > * Reizer, Eyal[170807 00:32]: > > The following commits: > > c815fde wlcore: spi: Populate config firmware data > > d776fc8 wlcore: sdio: Populate config firmware data > > > > Populated the nvs entry for wilink6 and wilink7 only while it is > > still needed for wilink8 as well. > > This broke user space backward compatibility when upgrading from older > > kernels, as the alternate mac address would not be read from the nvs that > is > > present in the file system (lib/firmware/ti-connectivity/wl1271-nvs.bin) > > causing mac address change of the wlan interface. > > > > This patch fix this and update the structure field with the same default > > nvs file name that has been used before. > > > > In addition, some distros hold a default wl1271-nvs.bin in the file > > system with a bogus mac address (deadbeef...) that for a wl18xx device > > also overrides the mac address that is stored inside the device. > > Warn users about this bogus mac address and use a random mac instead > > Hmm looks pretty good to me except for one more thing I just noticed. > > Why don't you just use the hardware mac address instead of a random > mac address on wl18xx device when you see a bogus nvs file? > I agree that this would have been better but the problem is that hardware mac address is available for wilink8 onlyWilink6/7 don't have one stored. The wlcore code responsible for handling mac address is common code and there is method for detecting between them in this module. Best Regards, Eyal
Re: [v5] wlcore: add missing nvs file name info for wilink8
* Reizer, Eyal[170807 00:32]: > The following commits: > c815fde wlcore: spi: Populate config firmware data > d776fc8 wlcore: sdio: Populate config firmware data > > Populated the nvs entry for wilink6 and wilink7 only while it is > still needed for wilink8 as well. > This broke user space backward compatibility when upgrading from older > kernels, as the alternate mac address would not be read from the nvs that is > present in the file system (lib/firmware/ti-connectivity/wl1271-nvs.bin) > causing mac address change of the wlan interface. > > This patch fix this and update the structure field with the same default > nvs file name that has been used before. > > In addition, some distros hold a default wl1271-nvs.bin in the file > system with a bogus mac address (deadbeef...) that for a wl18xx device > also overrides the mac address that is stored inside the device. > Warn users about this bogus mac address and use a random mac instead Hmm looks pretty good to me except for one more thing I just noticed. Why don't you just use the hardware mac address instead of a random mac address on wl18xx device when you see a bogus nvs file? Regards, Tony