Re: [PATCH] iwlwifi: fix firmware names for 9xxx and a0 series hw

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-11-14 at 13:29 +0200, Thomas Backlund wrote:
> Den 14-11-2017 kl. 12:52, skrev Luca Coelho:
> > On Tue, 2017-11-14 at 12:37 +0200, Thomas Backlund wrote:
> > > iwlwifi 9xxx and a0 series hw contains an extra dash in firmware
> > > file
> > > name
> > > as seeen in modinfo output for kernel 4.14:
> > > 
> > > firmware:       iwlwifi-9260-th-b0-jf-b0--34.ucode
> > > firmware:       iwlwifi-9260-th-a0-jf-a0--34.ucode
> > > firmware:       iwlwifi-9000-pu-a0-jf-b0--34.ucode
> > > firmware:       iwlwifi-9000-pu-a0-jf-a0--34.ucode
> > > firmware:       iwlwifi-QuQnj-a0-hr-a0--34.ucode
> > > firmware:       iwlwifi-QuQnj-a0-jf-b0--34.ucode
> > > firmware:       iwlwifi-QuQnj-f0-hr-a0--34.ucode
> > > firmware:       iwlwifi-Qu-a0-jf-b0--34.ucode
> > > firmware:       iwlwifi-Qu-a0-hr-a0--34.ucode
> > > 
> > > Fix that by dropping the extra adding of '"-"'.
> > > 
> > > Signed-off-by: Thomas Backlund <tmb-odJJhXpcy38dnm+yROfE0A@public
> > > .gmane.org>
> > > cc: Luca Coelho <luciano.coelho-ral2JQCrhuEAvxtiuMwx3w@xxxxxxxxxx
> > > ne.org>
> > > cc: stable-u79uwXL29TY76Z2rM5mHXA@xxxxxxxxxxxxxxxx
> > > 
> > > ---
> > 
> > Looks good, thanks!
> > 
> > Though I'm not sure this is worth sending to stable... Why do you
> > think
> > it's needed?
> > 
> 
> As 4.14 is a -longterm kernel and afaik atleast 9xxx series hw is
> out 
> there I thought it would be good to have it fixed there...
> 
> but in the end it's your call for upstream stable or not
> 
> I'll add it to the Mageia distrib kernels anyway.

Okay, fair enough.  I'll add it to stable, it's a very simple and not
risky fix anyway.  One can claim that userspace may get confused. 
Actually I think some distros use those values to check which firmwares
need to be installed, but I don't recall for sure now.

Thanks!

--
Luca.



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]