Re: [PATCH 4/6] wl1251: Generate random MAC address only if driver does not have valid
From: Pali RohÃr
Date: Sat Dec 24 2016 - 13:38:56 EST
On Saturday 24 December 2016 19:08:54 Pavel Machek wrote:
> On Sat 2016-12-24 17:52:59, Pali RohÃr wrote:
> > Before this patch driver generated random MAC address every time
> > when was doing initialization. And after that random MAC address
> > could be overwritten with fixed one if provided.
> Before this patch, driver generated random MAC address every time it
> was initialized. After that random MAC address could be overwritten
> with fixed one, if provided.
> > This patch changes order. First it tries to read fixed MAC address
> > and if it fails then driver generates random MAC address.
> I don't quite get where the advantage is supposed to be. Is it that
> "use_eeprom" is set, but reading fails?
Random bytes are read from kernel only if random MAC address is needed.
And in wl->mac_addr is always either invalid address or permanenent mac
address which will be used. Without patch in wl->mac_addr can be random
temporary address for some time...
> The only case where this helps is if wl1251_read_eeprom_mac()
> succeeds but reads invalid address.
> > Signed-off-by: Pali RohÃr <pali.rohar@xxxxxxxxx>
> Acked-by: Pavel Machek <pavel@xxxxxx>
Description: This is a digitally signed message part.