From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:57059 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751927AbdHIPWD (ORCPT ); Wed, 9 Aug 2017 11:22:03 -0400 Date: Wed, 9 Aug 2017 08:21:53 -0700 From: Greg KH To: Eyal Reizer Cc: eyalr@ti.com, "# 4 . 9+" Subject: Re: [v6] wlcore: add missing nvs file name info for wilink8 Message-ID: <20170809152153.GC26708@kroah.com> References: <1502264840-10569-1-git-send-email-eyalr@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1502264840-10569-1-git-send-email-eyalr@ti.com> Sender: stable-owner@vger.kernel.org List-ID: On Wed, Aug 09, 2017 at 10:47:20AM +0300, Eyal Reizer wrote: > The following commits: > commit c815fdebef44 ("wlcore: spi: Populate config firmware data") > commit 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 overrides the mac > address that is stored inside the device. > Warn users about this bogus mac address and use the internal mac address > > Fixes: c815fdebef44 ("wlcore: spi: Populate config firmware data") > Fixes: d776fc86b82f ("wlcore: sdio: Populate config firmware data") > Cc: # 4.9+ > Signed-off-by: Eyal Reizer > --- > v2->v3: add a check for default deadbeef... mac address and warn about it > v3->v4: use a random TI mac address instead of the bogus one > v4->v5: add constant definition for TI oui address > v5->v6: after also verifying on wilink6/7 Use mac internal mac address > instead of a random one > --- > drivers/net/wireless/ti/wlcore/main.c | 15 +++++++++++++++ > drivers/net/wireless/ti/wlcore/sdio.c | 1 + > drivers/net/wireless/ti/wlcore/spi.c | 1 + > 3 files changed, 17 insertions(+) This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read: https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html for how to do this properly.