On Tue, 14 Nov 2023 14:31:25 +0100, Saarinen, Jani wrote: > > Hi, > > -----Original Message----- > > From: Maarten Lankhorst <[email protected]> > > Sent: Tuesday, November 14, 2023 3:30 PM > > To: Jani Nikula <[email protected]>; Ville Syrjälä > > <[email protected]> > > Cc: Takashi Iwai <[email protected]>; [email protected]; intel- > > [email protected]; [email protected]; Saarinen, Jani > > <[email protected]>; Kurmi, Suresh Kumar > > <[email protected]> > > Subject: Re: [Intel-xe] [PATCH 11/14] ALSA: hda/intel: Move > > snd_hdac_i915_init > > to before probe_work. > > > > Hey, > > > > Den 2023-11-14 kl. 13:35, skrev Jani Nikula: > > > On Tue, 14 Nov 2023, Ville Syrjälä <[email protected]> wrote: > > >> On Mon, Oct 02, 2023 at 09:38:44PM +0200, > > [email protected] wrote: > > >>> From: Maarten Lankhorst <[email protected]> > > >>> > > >>> Now that we can use -EPROBE_DEFER, it's no longer required to spin > > >>> off the snd_hdac_i915_init into a workqueue. > > >>> > > >>> Use the -EPROBE_DEFER mechanism instead, which must be returned in > > >>> the probe function. > > >> This completely broke i915 audio! > > >> > > >> I also can't see any trace of this stuff ever being posted to > > >> intel-gfx so it never went through the CI. > > >> > > >> Please fix or revert ASAP. > > > Cc: Jani, Suresh > > > > > > Ville, please file a bug at gitlab so we can track this, thanks. > > > > I've originally tested this on TGL and DG2, so can you be more specific on > > what > > broke? > Was this series tested on CI ever as Ville saying no? How this got merged?
Hm, I somehow believed that patches have been tested by intel people, as they came from intel. (Also thought that drm-tip or whatever tree also drags the recent changes from sound.git tree for relevant part.) In anyway, the whole patches were touching only sound/* files and no changes in DRM side at all, hence they weren't sent to intel-gfx, I suppose. Takashi
