Messages in this thread | | | Date | Fri, 21 Jul 2023 13:06:21 +0300 (EEST) | From | Kai Vehmanen <> | Subject | Re: [PATCH v2 0/9] sound: Use -EPROBE_DEFER instead of i915 module loading. |
| |
Hi,
On Wed, 19 Jul 2023, Maarten Lankhorst wrote:
> Explicitly loading i915 becomes a problem when upstreaming the new intel driver > for Tiger Lake and higher graphics (xe). By loading i915, it doesn't wait for > driver load of xe, and will fail completely before it loads. > > -EPROBE_DEFER has to be returned before any device is created in probe(), > otherwise the removal of the device will cause EPROBE_DEFER to try again > in an infinite loop.
thanks, series looks good to me now. We'll need to adopt the new gpu_bind parameter in a number of CI systems (where we test without i915/xe), but this looks perfectly doable.
I'll give my
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
... for the hdac_i915.c changes. For AVS and SOF, I'd ask for some more review time to allow Cezary, Pierre et al to weigh in. I don't personally recall e.g. where we've used CONFIG_SOF_FORCE_PROBE_WORKQUEUE and do we have grounds to keep it even if workqueue is no longer set for HDA codec support.
Br, Kai
| |