Messages in this thread |  | | Subject | RE: [PATCH] Bluetooth: btmrvl_sdio: look for sd8688 firmware in alternate place | From | Lubomir Rintel <> | Date | Wed, 09 Jan 2013 07:31:28 +0100 |
| |
On Tue, 2013-01-08 at 18:43 -0800, Bing Zhao wrote: > > > linux-firmware ships the sd8688* firmware images that are shared with > > > libertas_sdio WiFi driver under libertas/. libertas_sdio looks in both places > > > and so should we. > > > > > > Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> > > > --- > > > drivers/bluetooth/btmrvl_sdio.c | 24 ++++++++++++++++++++++-- > > > drivers/bluetooth/btmrvl_sdio.h | 6 ++++-- > > > 2 files changed, 26 insertions(+), 4 deletions(-) > > > > NAK from me on this one. I do not want the driver to check two > > locations. That is what userspace can work around. > > > > If we want to unify the location between the WiFi driver and the > > Bluetooth driver, I am fine with that, but seriously, just pick one over > > the other. I do not care which one. > > The unified location is mrvl/ directory. > > We can probably move SD8688 firmware & helper binaries to mrvl/ and have both drivers grab the images there?
That would break existing setups, wouldn't it?
I was under impression (commit 3d32a58b) that we care about compatibility here. Do we?
-- Lubomir Rintel
|  |