Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 15 Sep 2021 10:43:56 +0900
From:   Dominique MARTINET <dominique.martinet@atmark-techno.com>
To:     Jonas =?utf-8?Q?Dre=C3=9Fler?= <verdre@v0yd.nl>
Cc:     Sharvari Harisangam <sharvari.harisangam@nxp.com>,
        "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
        Amitkumar Karwar <amitkarwar@gmail.com>,
        Takashi Iwai <tiwai@suse.de>,
        Tsuchiya Yuto <kitakar@gmail.com>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Arnd Bergmann <arnd@arndb.de>,
        Lee Jones <lee.jones@linaro.org>,
        Kalle Valo <kvalo@codeaurora.org>,
        Xinming Hu <huxinming820@gmail.com>,
        Ganapathi Bhat <ganapathi017@gmail.com>
Subject: Re: [EXT] Re: mwifiex cmd timeout on one pci variant
Message-ID: <YUFP3InQ+NYLpqRB@atmark-techno.com>
References: <YTg/f5mHQ6jjHDt6@atmark-techno.com>
 <YThLznrMQ4EYUDEl@atmark-techno.com>
 <AM0PR04MB4529E0C28F43288E189D8F50FCD49@AM0PR04MB4529.eurprd04.prod.outlook.com>
 <YThQiMn7YHzPRwnJ@atmark-techno.com>
 <af5cff45-da9d-26b7-fd00-c4e91344cfc1@v0yd.nl>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <af5cff45-da9d-26b7-fd00-c4e91344cfc1@v0yd.nl>
Precedence: bulk

Hi Jonas,

Jonas Dreßler wrote on Tue, Sep 14, 2021 at 12:11:46PM +0200:
> regarding the firmware version, as you can see in the commit updating the
> firmware binaries (https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/mrvl/pcie8897_uapsta.bin?id=1a5773c0c89ee44cee51a285d5c7c1063cdb0891),
> indeed the version numbering differs between the different versions of the
> card (usb/usb, pcie/usb, pcie/uart(?)).

Right. The update frequency is also quite different, so I'm assuming the
pcie/uart version I'm using has a lot of vulnerabilities left open as
well...


> Anyway, if you manage to find newer firmware for any of those versions, I'd
> be happy if you could point me to that, apparently they just fixed a
> critical vulnerability in the Windows firmware again (see https://support.microsoft.com/en-us/surface/surface-pro-5th-gen-update-history-5203144a-90c1-63df-ce0b-7ec7ff32ff10),
> I wouldn't be surprised if our firmware is also affected by that.

That sounds like a safe bet..
I assume the firmwares are not compatible and we can't just load these?


> About the command timeout, I have no idea why the fix isn't working for you,
> but well, my analysis of the issue is also just a (not exactly educated)
> guess, so it might as well be a completely different problem and my fix is
> just a lucky hack.

Right, it really depends on why the firmware crashed, but we have no way
of investigating that at the moment.

> I'd kinda hope though that my proposed patches finally wake up some people
> at NXP and motivate them to take a look at that firmware repo again.

If it works well enough it could be a reason not to bother :D
Alternatively if they can't spend time on it maybe open the firmware
code (under NDA? my company probably already has one with NXP..), but
my problem will need more time to reach them through regular channels.

-- 
Dominique