Messages in this thread | | | From | Lucas De Marchi <> | Date | Wed, 3 Oct 2012 18:58:32 -0300 | Subject | Re: udev breakages - was: Re: Need of an ".async_probe()" type of callback at driver's core - Was: Re: [PATCH] [media] drxk: change it to use request_firmware_nowait() |
| |
On Wed, Oct 3, 2012 at 5:39 PM, Linus Torvalds <torvalds@linux-foundation.org> wrote: > On Wed, Oct 3, 2012 at 12:50 PM, Greg KH <gregkh@linuxfoundation.org> wrote: >>> >>> Ok, like this? >> >> This looks good to me. Having udev do firmware loading and tieing it to >> the driver model may have not been such a good idea so many years ago. >> Doing it this way makes more sense. > > Ok, I wish this had been getting more testing in Linux-next or > something, but I suspect that what I'll do is to commit this patch > asap, and then commit another patch that turns off udev firmware > loading entirely for the synchronous firmware loading case.
This would break non-udev users with different paths. Namely Android, which uses /system/etc/firmware and /system/vendor/firmware (not sure about them though, I'm not keen on Android camp)
So maintaining the fallback or adding a configurable entry to set the firmware paths might be good.
Lucas De Marchi
| |