Return-path: Received: from paleale.coelho.fi ([176.9.41.70]:48502 "EHLO farmhouse.coelho.fi" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752775AbdIGFlA (ORCPT ); Thu, 7 Sep 2017 01:41:00 -0400 Message-ID: <1504762851.5400.93.camel@coelho.fi> (sfid-20170907_074117_097075_F9495D62) From: Luca Coelho To: "torvalds@linux-foundation.org" Cc: "linux-kernel@vger.kernel.org" , linuxwifi , "Berg, Johannes" , "akpm@linux-foundation.org" , "kvalo@codeaurora.org" , "netdev@vger.kernel.org" , "davem@davemloft.net" , "linux-wireless@vger.kernel.org" , "Grumbach, Emmanuel" Date: Thu, 07 Sep 2017 08:40:51 +0300 In-Reply-To: <1504760688.5400.91.camel@intel.com> References: <20170905.214143.826912481689443792.davem@davemloft.net> <1504757495.5400.89.camel@intel.com> <1504760688.5400.91.camel@intel.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Subject: Re: [GIT] Networking Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, 2017-09-07 at 05:04 +0000, Coelho, Luciano wrote: > On Wed, 2017-09-06 at 21:57 -0700, Linus Torvalds wrote: > > On Wed, Sep 6, 2017 at 9:11 PM, Coelho, Luciano > > wrote: > > > > > > This seems to be a problem with backwards-compatibility with FW version > > > 27. We are now in version 31[1] and upgrading will probably fix that. > > > > I can confirm that fw version 31 works. > > Great, so I know for sure that this is a backwards-compatibility issue > with the FW API. > > > > > But obviously the driver should not fail miserably like this with > > > version 27, because it claims to support it still. > > > > Not just "claims to support it", but if it's what is shipped with a > > fairly recent distro like an up-to-date version of F26, I would really > > hope that the driver can still work with it. > > I totally agree, we support a bunch of older versions for that exact > reason. We just don't really test all the supported versions very > often. We should probably change that. > > I'll make sure it still works with version 27. Okay, I found the offending patch: commit 7089ae634c50544b29b31faf1a751e8765c8de3b Author:     Johannes Berg AuthorDate: Wed Jun 28 16:19:49 2017 +0200 Commit:     Luca Coelho CommitDate: Wed Aug 9 09:15:32 2017 +0300     iwlwifi: mvm: use firmware LED command where applicable          On devices starting from 8000 series, the host can no longer toggle     the LED through the CSR_LED_REG register, but must do it via the     firmware instead. Add support for this. Note that this means that     the LED cannot be turned on while the firmware is off, so using an     arbitrary LED trigger may not work as expected.          Fixes: 503ab8c56ca0 ("iwlwifi: Add 8000 HW family support")     Signed-off-by: Johannes Berg     Signed-off-by: Luca Coelho Reverting it solves the problem. We introduced a new command to control the LED lights and assumed it was available in older FW versions as well, which turned out not to be the case. This patch is not very important (unless you really like blinking lights -- maybe I'll change my mind when the holidays approach :P). so it is fine if you just want to revert it for now. In any case, I'll send a patch fixing this problem soon. -- Cheers, Luca.