Re: [PATCH] firmware: Allow release-specific firmware dir

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Marcel Holtmann
Date: Thursday, September 11, 2008 - 9:09 am

Hi Jeff,


let me repeat this, if a driver depends on newer firmware version, then
it should make sure that the firmware itself is versioned. Everything
else will break eventually.

The Intel wireless drivers started to version their firmware a long time
ago. That is the way to go. How will a user tell different firmwares
apart otherwise.


Checking /lib/firmware/$(uname -r) is not the issue. The issue is that
when installing a self-compiled kernel, the firmware is not present
in /lib/firmware.


If the firmware is not in /lib/firmware (especially external firmware)
it will not be found and the module will be useless if it depends on
that firmware. And the biggest part of the firmware is not distributed
with the kernel because of GPL restrictions. So whatever we decide for
the kernel right now helps nothing if the user needs external firmware
for their devices.

Just take the Intel wireless drivers as an example. If I boot my own
kernel, I have to make sure I either copied or linked them from the
distro directory to /lib/firmware or downloaded them. Whatever the
kernel firmware prefix is makes no difference here.


That is the whole point and Dave and I discussed this deeply. If the
blob changes in an incompatible way or if a driver needs a newer
firmware, the firmware filename should change.

Regards

Marcel


--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] firmware: Allow release-specific firmware dir, Jeff Mahoney, (Tue Sep 9, 7:15 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Andrew Morton, (Wed Sep 10, 3:35 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:01 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:15 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Andrew Morton, (Wed Sep 10, 4:24 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:36 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 5:39 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Faidon Liambotis, (Wed Sep 10, 11:35 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Jeff Mahoney, (Thu Sep 11, 12:15 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 12:44 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Faidon Liambotis, (Thu Sep 11, 1:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Thierry Vignaud, (Thu Sep 11, 4:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 7:50 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 8:31 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Linus Torvalds, (Thu Sep 11, 8:57 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 9:01 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:09 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:12 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:16 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:20 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:39 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 9:45 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Linus Torvalds, (Thu Sep 11, 10:49 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 1:38 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 2:15 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 3:25 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Joseph Fannin, (Fri Sep 12, 1:39 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Fri Sep 12, 7:32 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Kai Henningsen, (Fri Sep 12, 1:24 pm)