Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Greg KH
On Fri, Jun 06, 2014 at 05:35:58PM +0200, Tom Gundersen wrote: > > On 6 Jun 2014 17:22, "Greg KH" wrote: > > > > On Fri, Jun 06, 2014 at 04:15:03PM +0200, Tom Gundersen wrote: > > > How about: > > > > > > "If you rely on a customized udev (or other userspace tool) to load > > > firmware from a no

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Takashi Iwai
At Fri, 6 Jun 2014 08:22:40 -0700, Greg KH wrote: > > This is all bikeshedding, right? Of course, what else? :) Takashi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Ilia Mirkin
On Fri, Jun 6, 2014 at 11:19 AM, Ming Lei wrote: > On Fri, Jun 6, 2014 at 10:15 PM, Tom Gundersen wrote: >> On Fri, Jun 6, 2014 at 4:03 PM, Takashi Iwai wrote: >>> At Fri, 6 Jun 2014 07:00:22 +0800, >>> Ming Lei wrote: On Thu, Jun 5, 2014 at 11:15 PM, Tom Gundersen wrote: > On Th

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Greg KH
On Fri, Jun 06, 2014 at 04:15:03PM +0200, Tom Gundersen wrote: > How about: > > "If you rely on a customized udev (or other userspace tool) to load > firmware from a non-standard path, say Y. Otherwise, say N. If your > udev version does not support firmware loading (which is currently the > upstr

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Ming Lei
On Fri, Jun 6, 2014 at 10:15 PM, Tom Gundersen wrote: > On Fri, Jun 6, 2014 at 4:03 PM, Takashi Iwai wrote: >> At Fri, 6 Jun 2014 07:00:22 +0800, >> Ming Lei wrote: >>> >>> On Thu, Jun 5, 2014 at 11:15 PM, Tom Gundersen wrote: >>> > On Thu, Jun 5, 2014 at 4:54 PM, Ming Lei wrote: >>> >>> Ubuntu

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Tom Gundersen
On Fri, Jun 6, 2014 at 4:03 PM, Takashi Iwai wrote: > At Fri, 6 Jun 2014 07:00:22 +0800, > Ming Lei wrote: >> >> On Thu, Jun 5, 2014 at 11:15 PM, Tom Gundersen wrote: >> > On Thu, Jun 5, 2014 at 4:54 PM, Ming Lei wrote: >> >>> Ubuntu currently enables the firmware loader in both the kernel and i

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-06 Thread Takashi Iwai
At Fri, 6 Jun 2014 07:00:22 +0800, Ming Lei wrote: > > On Thu, Jun 5, 2014 at 11:15 PM, Tom Gundersen wrote: > > On Thu, Jun 5, 2014 at 4:54 PM, Ming Lei wrote: > >>> Ubuntu currently enables the firmware loader in both the kernel and in > >>> udev, so would not yet have a problem here at the mo

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Thu, Jun 5, 2014 at 11:15 PM, Tom Gundersen wrote: > On Thu, Jun 5, 2014 at 4:54 PM, Ming Lei wrote: >>> Ubuntu currently enables the firmware loader in both the kernel and in >>> udev, so would not yet have a problem here at the moment. However, I >>> spoke with Martin Pitt and he told me tha

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Tom Gundersen
On Thu, Jun 5, 2014 at 4:54 PM, Ming Lei wrote: >> Ubuntu currently enables the firmware loader in both the kernel and in >> udev, so would not yet have a problem here at the moment. However, I >> spoke with Martin Pitt and he told me that both Debian and Ubuntu >> would like to switch this off in

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Takashi Iwai
At Thu, 5 Jun 2014 22:54:33 +0800, Ming Lei wrote: > > On Thu, Jun 5, 2014 at 10:32 PM, Tom Gundersen wrote: > > On Thu, Jun 5, 2014 at 4:24 PM, Ming Lei wrote: > >> On Thu, Jun 5, 2014 at 10:05 PM, Takashi Iwai wrote: > >>> At Thu, 5 Jun 2014 21:59:52 +0800, > >>> Ming Lei wrote: > >

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Thu, Jun 5, 2014 at 10:32 PM, Tom Gundersen wrote: > On Thu, Jun 5, 2014 at 4:24 PM, Ming Lei wrote: >> On Thu, Jun 5, 2014 at 10:05 PM, Takashi Iwai wrote: >>> At Thu, 5 Jun 2014 21:59:52 +0800, >>> Ming Lei wrote: On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: > At Thu,

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Tom Gundersen
On Thu, Jun 5, 2014 at 4:24 PM, Ming Lei wrote: > On Thu, Jun 5, 2014 at 10:05 PM, Takashi Iwai wrote: >> At Thu, 5 Jun 2014 21:59:52 +0800, >> Ming Lei wrote: >>> >>> On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: >>> > At Thu, 5 Jun 2014 21:31:56 +0800, >>> > Ming Lei wrote: >>> >> >>> >>

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Thu, Jun 5, 2014 at 10:05 PM, Takashi Iwai wrote: > At Thu, 5 Jun 2014 21:59:52 +0800, > Ming Lei wrote: >> >> On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: >> > At Thu, 5 Jun 2014 21:31:56 +0800, >> > Ming Lei wrote: >> >> >> >> On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: >> >

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Tom Gundersen
On Thu, Jun 5, 2014 at 3:59 PM, Ming Lei wrote: > On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: >> At Thu, 5 Jun 2014 21:31:56 +0800, >> Ming Lei wrote: >>> >>> On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: >>> > >>> > On 5 Jun 2014 14:18, "Ming Lei" wrote: >>> >> >>> >> On Wed, Ju

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Takashi Iwai
At Thu, 5 Jun 2014 21:59:52 +0800, Ming Lei wrote: > > On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: > > At Thu, 5 Jun 2014 21:31:56 +0800, > > Ming Lei wrote: > >> > >> On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: > >> > > >> > On 5 Jun 2014 14:18, "Ming Lei" wrote: > >> >> > >>

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Tom Gundersen
On Thu, Jun 5, 2014 at 3:31 PM, Ming Lei wrote: > On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: >> >> On 5 Jun 2014 14:18, "Ming Lei" wrote: >>> >>> On Wed, Jun 4, 2014 at 11:48 PM, Takashi Iwai wrote: >>> > [The patch was originally proposed by Tom Gundersen, and rewritten >>> > afterw

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Thu, Jun 5, 2014 at 9:47 PM, Takashi Iwai wrote: > At Thu, 5 Jun 2014 21:31:56 +0800, > Ming Lei wrote: >> >> On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: >> > >> > On 5 Jun 2014 14:18, "Ming Lei" wrote: >> >> >> >> On Wed, Jun 4, 2014 at 11:48 PM, Takashi Iwai wrote: >> >> > [The pa

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Takashi Iwai
At Thu, 5 Jun 2014 21:31:56 +0800, Ming Lei wrote: > > On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: > > > > On 5 Jun 2014 14:18, "Ming Lei" wrote: > >> > >> On Wed, Jun 4, 2014 at 11:48 PM, Takashi Iwai wrote: > >> > [The patch was originally proposed by Tom Gundersen, and rewritten > >

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Thu, Jun 5, 2014 at 8:25 PM, Tom Gundersen wrote: > > On 5 Jun 2014 14:18, "Ming Lei" wrote: >> >> On Wed, Jun 4, 2014 at 11:48 PM, Takashi Iwai wrote: >> > [The patch was originally proposed by Tom Gundersen, and rewritten >> > afterwards by me; most of changelogs below borrowed from Tom's

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-05 Thread Ming Lei
On Wed, Jun 4, 2014 at 11:48 PM, Takashi Iwai wrote: > [The patch was originally proposed by Tom Gundersen, and rewritten > afterwards by me; most of changelogs below borrowed from Tom's > original patch -- tiwai] > > Currently (at least) the dell-rbu driver selects FW_LOADER_USER_HELPER, > whic

Re: [PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-04 Thread Greg Kroah-Hartman
On Wed, Jun 04, 2014 at 05:48:15PM +0200, Takashi Iwai wrote: > [The patch was originally proposed by Tom Gundersen, and rewritten > afterwards by me; most of changelogs below borrowed from Tom's > original patch -- tiwai] > > Currently (at least) the dell-rbu driver selects FW_LOADER_USER_HELPE

[PATCH v2] firmware loader: allow disabling of udev as firmware loader

2014-06-04 Thread Takashi Iwai
[The patch was originally proposed by Tom Gundersen, and rewritten afterwards by me; most of changelogs below borrowed from Tom's original patch -- tiwai] Currently (at least) the dell-rbu driver selects FW_LOADER_USER_HELPER, which means that distros can't really stop loading firmware through u