Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1547094

Re: [PATCH 4/6] wl1251: Generate random MAC address only if driver does not have valid

From Pavel Machek <pavel@ucw.cz>
Newsgroups linux.kernel
Subject Re: [PATCH 4/6] wl1251: Generate random MAC address only if driver does not have valid
Date 2016-12-24 19:10 +0100
Message-ID <sRYVz-1Rj-11@gated-at.bofh.it> (permalink)
References <sRXPQ-Xz-5@gated-at.bofh.it> <sRXZv-1gP-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

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?

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@gmail.com>

Acked-by: Pavel Machek <pavel@ucw.cz>

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/6] wl1251: Fix MAC address for Nokia N900 Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:00 +0100
  [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for loading NVS calibration data Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:00 +0100
    Re: [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for  loading NVS calibration data Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-12-25 21:20 +0100
      Re: [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for loading NVS calibration data Pali Rohár <pali.rohar@gmail.com> - 2016-12-25 21:50 +0100
        Re: [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for  loading NVS calibration data Pavel Machek <pavel@ucw.cz> - 2016-12-26 16:50 +0100
          Re: [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for loading NVS calibration data Pali Rohár <pali.rohar@gmail.com> - 2016-12-26 17:10 +0100
      Re: [PATCH 2/6] wl1251: Use request_firmware_prefer_user() for  loading NVS calibration data Pavel Machek <pavel@ucw.cz> - 2016-12-26 17:40 +0100
  [PATCH 1/6] firmware: Add request_firmware_prefer_user() function Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:00 +0100
  [PATCH 3/6] wl1251: Update wl->nvs_len after wl->nvs is valid Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:00 +0100
    Re: [PATCH 3/6] wl1251: Update wl->nvs_len after wl->nvs is valid Pavel Machek <pavel@ucw.cz> - 2016-12-24 19:10 +0100
  [PATCH 4/6] wl1251: Generate random MAC address only if driver does not have valid Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:10 +0100
    Re: [PATCH 4/6] wl1251: Generate random MAC address only if driver  does not have valid Pavel Machek <pavel@ucw.cz> - 2016-12-24 19:10 +0100
      Re: [PATCH 4/6] wl1251: Generate random MAC address only if driver does not have valid Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 19:40 +0100
  [PATCH 5/6] wl1251: Parse and use MAC address from supplied NVS data Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:10 +0100
    Re: [PATCH 5/6] wl1251: Parse and use MAC address from supplied NVS  data Pavel Machek <pavel@ucw.cz> - 2016-12-24 19:20 +0100
      Re: [PATCH 5/6] wl1251: Parse and use MAC address from supplied NVS data Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 19:50 +0100
  [PATCH 6/6] wl1251: Set generated MAC address back to NVS data Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 18:10 +0100
    Re: [PATCH 6/6] wl1251: Set generated MAC address back to NVS data Pavel Machek <pavel@ucw.cz> - 2016-12-24 19:20 +0100
      Re: [PATCH 6/6] wl1251: Set generated MAC address back to NVS data Pali Rohár <pali.rohar@gmail.com> - 2016-12-24 19:50 +0100

csiph-web