Re: [PATCH] staging:brcm80211:brcmfmac:change firmware/config file name

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

 



On Tue, Oct 19, 2010 at 05:55:46PM -0700, Henry Ptasinski wrote:
> On Tue, Oct 19, 2010 at 05:47:43PM -0700, Greg KH wrote:
> > On Tue, Oct 19, 2010 at 04:45:26PM -0700, Nohee Ko wrote:
> > > >>
> > > Have you made the same changes in the linux-firmware source tree as
> > > well?  Without changing it there, this could get messy.
> > > <<
> > > 
> > > No not yet. But we are planning sooner or later.
> > 
> > Please do it sooner.
> > 
> > I'll wait for that to happen before committing this patch.
> 
> There's no specific changes needed in the linux-firmware repo to work with this
> driver change.  There is a bugfix version of the firmware that we're getting
> ready to send up, which is the motivation for this change, but the change
> will work properly with the firmware that's in the repo today.

How, you just renamed the file you look for in the driver, right?

> All this change does is replace the hard-coded full firmware filename used in
> the driver with the basename-apiversion construction as documented in
> 
> 	http://www.linuxwireless.org/en/developers/Documentation/firmware-versioning
> 
> plus add some info to the README to help users get the firmware properly
> installed.
> 
> Users will need to create the symlinks as described in the README to use these
> driver changes.

Yes, so you just broke their working setup, right?

So, please get linux-firmware fixed to add the proper symlinks, and then
I can apply the patch, and say "go upgrade your linux-firmware package",
which is almost acceptable.

Asking them to do a link-by-hand isn't, sorry.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux