Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-07 Thread Shuah Khan
On Thu, Aug 7, 2014 at 1:57 AM, Takashi Iwai wrote: >> >> Hm? 3.16 doesn't contain my patch yet. It's merged for 3.17-rc1. >> > >> > Oh, you are right of course, I am on upstream kernel and I have your >> > patch. I don't mean your match is causing the issue though ;-). >> > >> >> I think this

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-07 Thread Takashi Iwai
At Wed, 6 Aug 2014 13:00:24 -0600, Shuah Khan wrote: > > On Wed, Aug 6, 2014 at 3:44 AM, Jean-Michel Hautbois > wrote: > > 2014-08-06 11:24 GMT+02:00 Takashi Iwai : > >> At Wed, 6 Aug 2014 11:10:27 +0200, > >> Jean-Michel Hautbois wrote: > >>> > >>> 2014-08-05 21:22 GMT+02:00 Shuah Khan : > >>>

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-07 Thread Takashi Iwai
At Wed, 6 Aug 2014 13:00:24 -0600, Shuah Khan wrote: On Wed, Aug 6, 2014 at 3:44 AM, Jean-Michel Hautbois jhautb...@gmail.com wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-07 Thread Shuah Khan
On Thu, Aug 7, 2014 at 1:57 AM, Takashi Iwai ti...@suse.de wrote: Hm? 3.16 doesn't contain my patch yet. It's merged for 3.17-rc1. Oh, you are right of course, I am on upstream kernel and I have your patch. I don't mean your match is causing the issue though ;-). I think this is what

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Shuah Khan
On Wed, Aug 6, 2014 at 3:44 AM, Jean-Michel Hautbois wrote: > 2014-08-06 11:24 GMT+02:00 Takashi Iwai : >> At Wed, 6 Aug 2014 11:10:27 +0200, >> Jean-Michel Hautbois wrote: >>> >>> 2014-08-05 21:22 GMT+02:00 Shuah Khan : >>> > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: >>> >>> Yes, it

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 13:24:37 +0200, Jean-Michel Hautbois wrote: > > 2014-08-06 12:52 GMT+02:00 Takashi Iwai : > > At Wed, 6 Aug 2014 12:50:28 +0200, > > Jean-Michel Hautbois wrote: > >> > >> 2014-08-06 12:21 GMT+02:00 Takashi Iwai : > >> > At Wed, 6 Aug 2014 11:44:14 +0200, > >> > Jean-Michel

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 12:52 GMT+02:00 Takashi Iwai : > At Wed, 6 Aug 2014 12:50:28 +0200, > Jean-Michel Hautbois wrote: >> >> 2014-08-06 12:21 GMT+02:00 Takashi Iwai : >> > At Wed, 6 Aug 2014 11:44:14 +0200, >> > Jean-Michel Hautbois wrote: >> >> >> >> 2014-08-06 11:24 GMT+02:00 Takashi Iwai : >> >> > At

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 12:50:28 +0200, Jean-Michel Hautbois wrote: > > 2014-08-06 12:21 GMT+02:00 Takashi Iwai : > > At Wed, 6 Aug 2014 11:44:14 +0200, > > Jean-Michel Hautbois wrote: > >> > >> 2014-08-06 11:24 GMT+02:00 Takashi Iwai : > >> > At Wed, 6 Aug 2014 11:10:27 +0200, > >> > Jean-Michel

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 12:21 GMT+02:00 Takashi Iwai : > At Wed, 6 Aug 2014 11:44:14 +0200, > Jean-Michel Hautbois wrote: >> >> 2014-08-06 11:24 GMT+02:00 Takashi Iwai : >> > At Wed, 6 Aug 2014 11:10:27 +0200, >> > Jean-Michel Hautbois wrote: >> >> >> >> 2014-08-05 21:22 GMT+02:00 Shuah Khan : >> >> > On Tue,

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 11:44:14 +0200, Jean-Michel Hautbois wrote: > > 2014-08-06 11:24 GMT+02:00 Takashi Iwai : > > At Wed, 6 Aug 2014 11:10:27 +0200, > > Jean-Michel Hautbois wrote: > >> > >> 2014-08-05 21:22 GMT+02:00 Shuah Khan : > >> > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: > >>

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 11:24 GMT+02:00 Takashi Iwai : > At Wed, 6 Aug 2014 11:10:27 +0200, > Jean-Michel Hautbois wrote: >> >> 2014-08-05 21:22 GMT+02:00 Shuah Khan : >> > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: >> >>> Yes, it worked, if the firmware was in the /lib/firmware directory. >> >> >>

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: > > 2014-08-05 21:22 GMT+02:00 Shuah Khan : > > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: > >>> Yes, it worked, if the firmware was in the /lib/firmware directory. > >> > > > > ok it works when the firmware is in

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-05 21:22 GMT+02:00 Shuah Khan : > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: >>> Yes, it worked, if the firmware was in the /lib/firmware directory. >> > > ok it works when the firmware is in /lib/firmware. It sounds to me the > reason load fails when the firmware is under

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-05 21:22 GMT+02:00 Shuah Khan shuahk...@gmail.com: On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai ti...@suse.de wrote: Yes, it worked, if the firmware was in the /lib/firmware directory. ok it works when the firmware is in /lib/firmware. It sounds to me the reason load fails when the

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah Khan shuahk...@gmail.com: On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai ti...@suse.de wrote: Yes, it worked, if the firmware was in the /lib/firmware directory. ok it works when the

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah Khan shuahk...@gmail.com: On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai ti...@suse.de wrote: Yes, it worked, if the firmware was in the

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 11:44:14 +0200, Jean-Michel Hautbois wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah Khan shuahk...@gmail.com: On Tue, Aug 5, 2014 at 10:01 AM, Takashi

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 12:21 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:44:14 +0200, Jean-Michel Hautbois wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah Khan

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 12:50:28 +0200, Jean-Michel Hautbois wrote: 2014-08-06 12:21 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:44:14 +0200, Jean-Michel Hautbois wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200,

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Jean-Michel Hautbois
2014-08-06 12:52 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 12:50:28 +0200, Jean-Michel Hautbois wrote: 2014-08-06 12:21 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:44:14 +0200, Jean-Michel Hautbois wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Takashi Iwai
At Wed, 6 Aug 2014 13:24:37 +0200, Jean-Michel Hautbois wrote: 2014-08-06 12:52 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 12:50:28 +0200, Jean-Michel Hautbois wrote: 2014-08-06 12:21 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:44:14 +0200,

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-06 Thread Shuah Khan
On Wed, Aug 6, 2014 at 3:44 AM, Jean-Michel Hautbois jhautb...@gmail.com wrote: 2014-08-06 11:24 GMT+02:00 Takashi Iwai ti...@suse.de: At Wed, 6 Aug 2014 11:10:27 +0200, Jean-Michel Hautbois wrote: 2014-08-05 21:22 GMT+02:00 Shuah Khan shuahk...@gmail.com: On Tue, Aug 5, 2014 at 10:01 AM,

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-05 Thread Shuah Khan
On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai wrote: >> Yes, it worked, if the firmware was in the /lib/firmware directory. > ok it works when the firmware is in /lib/firmware. It sounds to me the reason load fails when the firmware is under /sys/class/firmware is fw_load_from_user_helper()

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-05 Thread Takashi Iwai
At Tue, 5 Aug 2014 17:55:40 +0200, Jean-Michel Hautbois wrote: > > 2014-08-05 17:53 GMT+02:00 Takashi Iwai : > > At Tue, 5 Aug 2014 17:43:17 +0200, > > Jean-Michel Hautbois wrote: > >> > >> 2014-08-01 22:01 GMT+02:00 : > >> > Hi Takashi, > >> > > >> > Sorry for the late response, I tried with

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-05 Thread Jean-Michel Hautbois
2014-08-05 17:53 GMT+02:00 Takashi Iwai : > At Tue, 5 Aug 2014 17:43:17 +0200, > Jean-Michel Hautbois wrote: >> >> 2014-08-01 22:01 GMT+02:00 : >> > Hi Takashi, >> > >> > Sorry for the late response, I tried with latest stable kernel 3.15.8. >> > surprisingly the BIOS update works even without

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-05 Thread Takashi Iwai
At Tue, 5 Aug 2014 17:43:17 +0200, Jean-Michel Hautbois wrote: > > 2014-08-01 22:01 GMT+02:00 : > > Hi Takashi, > > > > Sorry for the late response, I tried with latest stable kernel 3.15.8. > > surprisingly the BIOS update works even without applying the patch in both > > the cases. > >

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-05 Thread Jean-Michel Hautbois
2014-08-01 22:01 GMT+02:00 : > Hi Takashi, > > Sorry for the late response, I tried with latest stable kernel 3.15.8. > surprisingly the BIOS update works even without applying the patch in both > the cases. Hi,I'm sorry for my late answer too, but I didn't try this patch recently... I have

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-05 Thread Jean-Michel Hautbois
2014-08-01 22:01 GMT+02:00 b_b_si...@dell.com: Hi Takashi, Sorry for the late response, I tried with latest stable kernel 3.15.8. surprisingly the BIOS update works even without applying the patch in both the cases. Hi,I'm sorry for my late answer too, but I didn't try this patch

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-05 Thread Takashi Iwai
At Tue, 5 Aug 2014 17:43:17 +0200, Jean-Michel Hautbois wrote: 2014-08-01 22:01 GMT+02:00 b_b_si...@dell.com: Hi Takashi, Sorry for the late response, I tried with latest stable kernel 3.15.8. surprisingly the BIOS update works even without applying the patch in both the cases.

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-05 Thread Jean-Michel Hautbois
2014-08-05 17:53 GMT+02:00 Takashi Iwai ti...@suse.de: At Tue, 5 Aug 2014 17:43:17 +0200, Jean-Michel Hautbois wrote: 2014-08-01 22:01 GMT+02:00 b_b_si...@dell.com: Hi Takashi, Sorry for the late response, I tried with latest stable kernel 3.15.8. surprisingly the BIOS update works

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-05 Thread Takashi Iwai
At Tue, 5 Aug 2014 17:55:40 +0200, Jean-Michel Hautbois wrote: 2014-08-05 17:53 GMT+02:00 Takashi Iwai ti...@suse.de: At Tue, 5 Aug 2014 17:43:17 +0200, Jean-Michel Hautbois wrote: 2014-08-01 22:01 GMT+02:00 b_b_si...@dell.com: Hi Takashi, Sorry for the late response, I tried

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-05 Thread Shuah Khan
On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai ti...@suse.de wrote: Yes, it worked, if the firmware was in the /lib/firmware directory. ok it works when the firmware is in /lib/firmware. It sounds to me the reason load fails when the firmware is under /sys/class/firmware is

RE: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-08-01 Thread B_B_Singh
Hi Takashi, Sorry for the late response, I tried with latest stable kernel 3.15.8. surprisingly the BIOS update works even without applying the patch in both the cases. Case I: BIOS update is always successful with or without applying the patch https://lkml.org/lkml/2014/6/4/327, I don't

RE: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-08-01 Thread B_B_Singh
Hi Takashi, Sorry for the late response, I tried with latest stable kernel 3.15.8. surprisingly the BIOS update works even without applying the patch in both the cases. Case I: BIOS update is always successful with or without applying the patch https://lkml.org/lkml/2014/6/4/327, I don't

RE: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-07-10 Thread B_B_Singh
Sorry Greg, Here is the patch which I was referring to https://lkml.org/lkml/2014/6/4/327 I was doing the BIOS update test using the DELL BIOS-DUP's with the above patch. Will provide you more technical analysis on why it is failing by tomorrow. Regards Balaji Singh -Original Message-

Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree

2014-07-10 Thread Greg KH
On Thu, Jul 10, 2014 at 08:10:01PM +0530, b_b_si...@dell.com wrote: > Resending the mail.. > > Hi All, > > As I communicated earlier the test never passed with older BIOS DUPs. That test? What patch? Please be explicit, I deal with thousands of patches and have no short term memory... >

Re: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-07-10 Thread Greg KH
On Thu, Jul 10, 2014 at 08:10:01PM +0530, b_b_si...@dell.com wrote: Resending the mail.. Hi All, As I communicated earlier the test never passed with older BIOS DUPs. That test? What patch? Please be explicit, I deal with thousands of patches and have no short term memory... Again

RE: patch firmware loader: allow disabling of udev as firmware loader added to driver-core tree

2014-07-10 Thread B_B_Singh
Sorry Greg, Here is the patch which I was referring to https://lkml.org/lkml/2014/6/4/327 I was doing the BIOS update test using the DELL BIOS-DUP's with the above patch. Will provide you more technical analysis on why it is failing by tomorrow. Regards Balaji Singh -Original Message-