From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from youngberry.canonical.com ([91.189.89.112]:42870 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753879AbcBPUVq (ORCPT ); Tue, 16 Feb 2016 15:21:46 -0500 Date: Tue, 16 Feb 2016 20:21:38 +0000 From: Luis Henriques To: Ben Hutchings Cc: Larry Finger , stable@vger.kernel.org, Kalle Valo , Greg Kroah-Hartman , LKML Subject: Re: [PATCH 4.4 112/117] rtlwifi: rtl8821ae: Fix errors in parameter initialization Message-ID: <20160216202138.GA8671@charon.olymp> References: <20160214222141.393531627@linuxfoundation.org> <20160214222144.781798389@linuxfoundation.org> <1455648364.1143.12.camel@decadent.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1455648364.1143.12.camel@decadent.org.uk> Sender: stable-owner@vger.kernel.org List-ID: On Tue, Feb 16, 2016 at 06:46:04PM +0000, Ben Hutchings wrote: > On Sun, 2016-02-14 at 14:22 -0800, Greg Kroah-Hartman wrote: > > > 4.4-stable review patch.��If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Larry Finger > > > > commit 78bae1de422a7f6f2b4b61f6a5c379e3d7f96f44 upstream. > > > > This driver failed to copy parameters sw_crypto and disable_watchdog into > > the locations actually used by the driver. In addition, msi_support was > > initialized three times and one of them used the wrong variable. The > > initialization of parameter int_clear was moved so that it is near that > > of the rest of the parameters. > > > > Signed-off-by: Larry Finger > > Signed-off-by: Kalle Valo > > Signed-off-by: Greg Kroah-Hartman > > > > --- > > �drivers/net/wireless/realtek/rtlwifi/rtl8821ae/sw.c |����9 +++++---- > > �1 file changed, 5 insertions(+), 4 deletions(-) > > > > --- a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/sw.c > > +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/sw.c > [...] > > @@ -168,12 +166,15 @@ int rtl8821ae_init_sw_vars(struct ieee80 > > � rtlpriv->psc.swctrl_lps = rtlpriv->cfg->mod_params->swctrl_lps; > > � rtlpriv->psc.fwctrl_lps = rtlpriv->cfg->mod_params->fwctrl_lps; > > � rtlpci->msi_support = rtlpriv->cfg->mod_params->msi_support; > > - rtlpci->msi_support = rtlpriv->cfg->mod_params->int_clear; > > + rtlpci->int_clear = rtlpriv->cfg->mod_params->int_clear; > > + rtlpriv->cfg->mod_params->sw_crypto = > > + rtlpriv->cfg->mod_params->sw_crypto; > > + rtlpriv->cfg->mod_params->disable_watchdog = > > + rtlpriv->cfg->mod_params->disable_watchdog; > [...] > > This is self-assignment. �How is that going to help? > Yeah, I did asked that question on the original patchset[1] but never actually got any reply. [1] https://marc.info/?l=linux-wireless&m=145437005409037&w=2 Cheers, -- Lu�s > Ben. > > -- > Ben Hutchings > Life is what happens to you while you're busy making other plans. > - John Lennon