From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: ARC-Seal: i=1; a=rsa-sha256; t=1517062501; cv=none; d=google.com; s=arc-20160816; b=EHQhynK2r3akZFVzsVmzBX7/VoNOKZqXZ7+26wYAngD1cGnohwQ8+Evw8WDj0fN4fx zllzX1IVEJUOOqkQxHZUpmxiX3x7WEce28CI+KEuRChieBXNccgif+iyVeoHNZ31668R mwNhTBu2tfynDF2kjXPSpGG8IgdFduSXOmZl118bqVa7ub1YPI9XO5A1cWjRXmzYimHz VHDrvttPBzi3y06G3t7jR0zc5MV3Wsd+IwoB/X7w0ePo/FANCrpS5znmUMXiSzdw/XK5 uf8C4EV9sqgwxsZ6rpG/zdU3YEIyO0+SUG8EGm3O2SsEmEDZFXJ9XyyL7DZ1Zo3IZcja vpLA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:dkim-signature :arc-authentication-results; bh=g4f2aidGHwNktxKCAISsLVGwkXxLpvJKJBCMqT0njHU=; b=Z/iLoXUJuK5nUci+XSdzIalX+8ETy+0xdte8FeOmAOkOrbgmkgzefRSWSEcR7rkC9I zp3IQUns8BLAyFToCBw6tudptpwkWJTtO8x9DUti9VaormzqUaPLRxYZFBbWbeJqAVGu AtPDMvkVSsdNJdDqU0q6iBJEYwQ2p0qgAZGx4+aXbWB+96P6UeNOvTnpxabCwdgMEDZT 0l5KINNNHH3lM81IG+CsKjlQsddSXYstvLoMIv7KxTUM1aEe5kw81+8Bzohrr7yIGAI3 2SwmyVPZsTo4NMKs1FeKkaD7aIGedOMJ+3vHVEJ+MB3nSJ5sLIlgMa9Ro4iCHs7ZkXd9 bboQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=u5xIro/B; spf=pass (google.com: domain of pali.rohar@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=pali.rohar@gmail.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=gmail.com Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=u5xIro/B; spf=pass (google.com: domain of pali.rohar@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=pali.rohar@gmail.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=gmail.com X-Google-Smtp-Source: AH8x225KXqcKfS/lwp8dow/TnMwR6l8+Ia8njhi5K9tcsspr37I/ZGRJHG5N2asLtEVVmZ5tBHCUDQ== Date: Sat, 27 Jan 2018 15:14:59 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Kalle Valo Cc: "Luis R. Rodriguez" , Greg Kroah-Hartman , David Gnedt , Daniel Wagner , Tony Lindgren , Sebastian Reichel , Pavel Machek , Ivaylo Dimitrov , Aaro Koskinen , Grazvydas Ignotas , linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH v2 0/6] wl1251: Fix MAC address for Nokia N900 Message-ID: <20180127141459.lym2mhd5md22co4q@pali> References: <1482598381-16513-1-git-send-email-pali.rohar@gmail.com> <1510270708-14377-1-git-send-email-pali.rohar@gmail.com> <20180102192345.dnywuv4sapltrfrl@pali> <20180105014510.GB22430@wotan.suse.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="wojkpjjfykxww634" Content-Disposition: inline In-Reply-To: <20180105014510.GB22430@wotan.suse.de> User-Agent: NeoMutt/20170113 (1.7.2) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1554617146502445782?= X-GMAIL-MSGID: =?utf-8?q?1590755330494395086?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: --wojkpjjfykxww634 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Friday 05 January 2018 02:45:10 Luis R. Rodriguez wrote: > On Tue, Jan 02, 2018 at 08:23:45PM +0100, Pali Roh=C3=A1r wrote: > > On Friday 10 November 2017 00:38:22 Pali Roh=C3=A1r wrote: > > > This patch series fix processing MAC address for wl1251 chip found in= Nokia N900. > > >=20 > > > Changes since v1: > > > * Added Acked-by for Pavel Machek > > > * Fixed grammar > > > * Magic numbers for NVS offsets are replaced by defines > > > * Check for validity of mac address NVS data is moved into function > > > * Changed order of patches as Pavel requested > > >=20 > > > Pali Roh=C3=A1r (6): > > > wl1251: Update wl->nvs_len after wl->nvs is valid > > > wl1251: Generate random MAC address only if driver does not have > > > valid > > > wl1251: Parse and use MAC address from supplied NVS data > > > wl1251: Set generated MAC address back to NVS data > > > firmware: Add request_firmware_prefer_user() function > > > wl1251: Use request_firmware_prefer_user() for loading NVS > > > calibration data > > >=20 > > > drivers/base/firmware_class.c | 45 +++++++++++++- > > > drivers/net/wireless/ti/wl1251/Kconfig | 1 + > > > drivers/net/wireless/ti/wl1251/main.c | 104 ++++++++++++++++++++++= ++++------ > > > include/linux/firmware.h | 9 +++ > > > 4 files changed, 138 insertions(+), 21 deletions(-) > >=20 > > Hi! Are there any comments for first 4 patches? If not, could they be > > accepted and merged? >=20 > Since the first 4 patches do not touch the firmware API they seem fine to= me so > long as the maintainer accepts them. Maybe resend and clarify you have dr= opped > the other ones and amend with the new tags. According to get_maintainer.pl, Kalle Valo is maintainer. Kalle Valo, if you do not have any other comments, can you accept first 4 patches? Or do you really need to resent first 4 patches again? --=20 Pali Roh=C3=A1r pali.rohar@gmail.com --wojkpjjfykxww634 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABECAB0WIQS4VrIQdKium2krgIWL8Mk9A+RDUgUCWmyJXgAKCRCL8Mk9A+RD UtxPAJ9u8R682FeXsOnGuZ5sdbAt0UdGhQCgkVmHq8z/ojmBAvh1XxEeTnxF6B4= =zwVx -----END PGP SIGNATURE----- --wojkpjjfykxww634--