Re: [PATCH 6/6] wl1251: Set generated MAC address back to NVS data

[Kalle Valo]

Pali RohÃr <pali.rohar@xxxxxxxxx> writes:

> On Friday 27 January 2017 09:56:09 Kalle Valo wrote:
>> Pali RohÃr <pali.rohar@xxxxxxxxx> writes:
>> 
>> > In case there is no valid MAC address kernel generates random one. This
>> > patch propagate this generated MAC address back to NVS data which will be
>> > uploaded to wl1251 chip. So HW would have same MAC address as linux kernel
>> > uses.
>> >
>> > Signed-off-by: Pali RohÃr <pali.rohar@xxxxxxxxx>
>> 
>> Why? What issue does this fix?
>
> Send permanent MAC address to wl1251 chip, same what is doing wl12xx
> driver.

Ok, so this doesn't change functionality in any way and you are adding
it only because wl12xx does the same? You should document that in the
the commit log.

If there's no change I don't really see the point of this. But if
there's harm, hopefully, I guess it's ok.

-- 
Kalle Valo