On Wed, Feb 21, 2024 at 09:03:53AM +0100, Francesco Dolcini wrote: > On Tue, Feb 20, 2024 at 09:52:13PM +0100, Greg Kroah-Hartman wrote: > > 6.1-stable review patch. If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Lukas Wunner <lukas@xxxxxxxxx> > > > > [ Upstream commit bba047f15851c8b053221f1b276eb7682d59f755 ] > > > > The Marvell SD8978 (aka NXP IW416) uses identical registers as SD8987, > > so reuse the existing mwifiex_reg_sd8987 definition. > > > > Note that mwifiex_reg_sd8977 and mwifiex_reg_sd8997 are likewise > > identical, save for the fw_dump_ctrl register: They define it as 0xf0 > > whereas mwifiex_reg_sd8987 defines it as 0xf9. I've verified that > > 0xf9 is the correct value on SD8978. NXP's out-of-tree driver uses > > 0xf9 for all of them, so there's a chance that 0xf0 is not correct > > in the mwifiex_reg_sd8977 and mwifiex_reg_sd8997 definitions. I cannot > > test that for lack of hardware, hence am leaving it as is. > > > > NXP has only released a firmware which runs Bluetooth over UART. > > Perhaps Bluetooth over SDIO is unsupported by this chipset. > > Consequently, only an "sdiouart" firmware image is referenced, not an > > alternative "sdsd" image. > > > > Signed-off-by: Lukas Wunner <lukas@xxxxxxxxx> > > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxx> > > Link: https://lore.kernel.org/r/536b4f17a72ca460ad1b07045757043fb0778988.1674827105.git.lukas@xxxxxxxxx > > Stable-dep-of: 1c5d463c0770 ("wifi: mwifiex: add extra delay for firmware ready") > > I would drop this and 1c5d463c0770. Why? Commit 1c5d463c0770 was explicitly tagged for stable inclusion, what changed? thanks, greg k-h
