Re: [PATCH 17/18] wl1271: Read MAC address from NVS file on HW startup

[Gery Kahn <geryk@ti.com>]

On 30.Nov.10 21:20, Arik Nemtsov wrote:
>> The idea is to eliminate the situation where anyone "should be careful to
>> not set the
>> MAC twice".
>> If there is MAC from NVS, make it system wide, w/o need to care about it. On
>> the other hand,
>> if MAC set by `ifconfig' same story, make it system wide.
> No one has to set the MAC address twice. This patch degrades
> gracefully - if there is no NVS file on disk, no harm done.
> This means a solution that only uses ifconfig or TESTMODE commands
> before adding an interface will also work.
>
> This patch only gives one the option to set the MAC address right when
> the HW is initialized, without resorting to scripts or usermode
> daemons. One has to simply put the NVS file on the filesystem and the
> MAC will be read from there. I found this easier from an engineering
> perspective.
>
> Note that even without this patch its possible to set the MAC address
> twice - use ifconfig, add a monitor interface and then use ifconfig
> again to change the MAC.
>
Fully agree. Just set vif where u set wl and the solution would be 
synced for your flow.
> Regards,
> Arik