Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Daniel Vetter
On Tue, Jul 4, 2017 at 5:28 PM, Takashi Iwai wrote: > On Tue, 04 Jul 2017 17:14:39 +0200, > Daniel Vetter wrote: >> On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: >> >> Now I took a closer look, and this appears rather like a brown paper >> >> bag bug, not

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Daniel Vetter
On Tue, Jul 4, 2017 at 5:28 PM, Takashi Iwai wrote: > On Tue, 04 Jul 2017 17:14:39 +0200, > Daniel Vetter wrote: >> On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: >> >> Now I took a closer look, and this appears rather like a brown paper >> >> bag bug, not about the deferred probe or

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Takashi Iwai
On Tue, 04 Jul 2017 17:14:39 +0200, Daniel Vetter wrote: > > On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: > >> Now I took a closer look, and this appears rather like a brown paper > >> bag bug, not about the deferred probe or module dependency. > >> The fix patch is

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Takashi Iwai
On Tue, 04 Jul 2017 17:14:39 +0200, Daniel Vetter wrote: > > On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: > >> Now I took a closer look, and this appears rather like a brown paper > >> bag bug, not about the deferred probe or module dependency. > >> The fix patch is below. Could you

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Daniel Vetter
On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: >> Now I took a closer look, and this appears rather like a brown paper >> bag bug, not about the deferred probe or module dependency. >> The fix patch is below. Could you check whether it works? > > Yay, this works! > >

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-07-04 Thread Daniel Vetter
On Thu, Jun 29, 2017 at 12:25 PM, Daniel Vetter wrote: >> Now I took a closer look, and this appears rather like a brown paper >> bag bug, not about the deferred probe or module dependency. >> The fix patch is below. Could you check whether it works? > > Yay, this works! > > Tested-by: Daniel

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-29 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 12:16:17PM +0200, Takashi Iwai wrote: > On Mon, 26 Jun 2017 19:54:49 +0200, > Daniel Vetter wrote: > > > > On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > > > On Mon, 26 Jun 2017 18:16:30 +0200, > > > Daniel Vetter wrote: > > >> > > >> On Wed, Jun

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-29 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 12:16:17PM +0200, Takashi Iwai wrote: > On Mon, 26 Jun 2017 19:54:49 +0200, > Daniel Vetter wrote: > > > > On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > > > On Mon, 26 Jun 2017 18:16:30 +0200, > > > Daniel Vetter wrote: > > >> > > >> On Wed, Jun 21, 2017 at

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-28 Thread Takashi Iwai
On Mon, 26 Jun 2017 19:54:49 +0200, Daniel Vetter wrote: > > On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > > On Mon, 26 Jun 2017 18:16:30 +0200, > > Daniel Vetter wrote: > >> > >> On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > >> > On Wed, 21 Jun 2017

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-28 Thread Takashi Iwai
On Mon, 26 Jun 2017 19:54:49 +0200, Daniel Vetter wrote: > > On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > > On Mon, 26 Jun 2017 18:16:30 +0200, > > Daniel Vetter wrote: > >> > >> On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > >> > On Wed, 21 Jun 2017 17:23:57 +0200, >

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > On Mon, 26 Jun 2017 18:16:30 +0200, > Daniel Vetter wrote: >> >> On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: >> > On Wed, 21 Jun 2017 17:23:57 +0200, >> > Chris Wilson wrote: >> > > >> > > Quoting Daniel

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Mon, Jun 26, 2017 at 7:47 PM, Takashi Iwai wrote: > On Mon, 26 Jun 2017 18:16:30 +0200, > Daniel Vetter wrote: >> >> On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: >> > On Wed, 21 Jun 2017 17:23:57 +0200, >> > Chris Wilson wrote: >> > > >> > > Quoting Daniel Vetter (2017-06-21

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Takashi Iwai
On Mon, 26 Jun 2017 18:16:30 +0200, Daniel Vetter wrote: > > On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > > On Wed, 21 Jun 2017 17:23:57 +0200, > > Chris Wilson wrote: > > > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > > So back when the i915 power well support

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Takashi Iwai
On Mon, 26 Jun 2017 18:16:30 +0200, Daniel Vetter wrote: > > On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > > On Wed, 21 Jun 2017 17:23:57 +0200, > > Chris Wilson wrote: > > > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > > So back when the i915 power well support

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > On Wed, 21 Jun 2017 17:23:57 +0200, > Chris Wilson wrote: > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > So back when the i915 power well support landed in > > > > > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > > >

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-26 Thread Daniel Vetter
On Wed, Jun 21, 2017 at 05:30:10PM +0200, Takashi Iwai wrote: > On Wed, 21 Jun 2017 17:23:57 +0200, > Chris Wilson wrote: > > > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > > So back when the i915 power well support landed in > > > > > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > > >

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Takashi Iwai
On Wed, 21 Jun 2017 17:23:57 +0200, Chris Wilson wrote: > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > So back when the i915 power well support landed in > > > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > > Author: Wang Xingchao > > Date: Thu May 30

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Takashi Iwai
On Wed, 21 Jun 2017 17:23:57 +0200, Chris Wilson wrote: > > Quoting Daniel Vetter (2017-06-21 16:08:54) > > So back when the i915 power well support landed in > > > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > > Author: Wang Xingchao > > Date: Thu May 30 22:07:10 2013 +0800 > > > >

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Chris Wilson
Quoting Daniel Vetter (2017-06-21 16:08:54) > So back when the i915 power well support landed in > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > Author: Wang Xingchao > Date: Thu May 30 22:07:10 2013 +0800 > > ALSA: hda - Add power-welll support for

Re: [PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Chris Wilson
Quoting Daniel Vetter (2017-06-21 16:08:54) > So back when the i915 power well support landed in > > commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 > Author: Wang Xingchao > Date: Thu May 30 22:07:10 2013 +0800 > > ALSA: hda - Add power-welll support for haswell HDA > > the logic to

[PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Daniel Vetter
So back when the i915 power well support landed in commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 Author: Wang Xingchao Date: Thu May 30 22:07:10 2013 +0800 ALSA: hda - Add power-welll support for haswell HDA the logic to handle the cross-module depencies

[PATCH] BUG-REPORT: snd-hda: hacked-together EPROBE_DEFER support

2017-06-21 Thread Daniel Vetter
So back when the i915 power well support landed in commit 99a2008d0b32d72dfc2a54e7be1eb698dd2e3bd6 Author: Wang Xingchao Date: Thu May 30 22:07:10 2013 +0800 ALSA: hda - Add power-welll support for haswell HDA the logic to handle the cross-module depencies was hand-rolled using a async