Re: [PATCH] ASoC: SOF: Intel: Remove deferred probe for SOF

Mark Brown
2023-07-19T12:39:23Z

[-- Attachment #1: Type: text/plain, Size: 1572 bytes --]

On Wed, Jul 19, 2023 at 02:13:59PM +0200, Maarten Lankhorst wrote:
>
> On 2023-07-19 13:06, Takashi Iwai wrote:
> > On Wed, 19 Jul 2023 11:48:06 +0200,
> > Maarten Lankhorst wrote:
> > >
> > > The 60 seconds timeout is a thing "better than complete disablement",
> > > so it's not ideal, either. Maybe we can add something like the
> > > following:
> > > - Check when the deferred probe takes too long, and warn it
> > > - Provide some runtime option to disable the component binding, so
> > > that user can work around it if needed
> > > A module option to snd_hdac_i915_init would probably be the least of all evils
> > > here.
> >
> > Yes, probably it's the easiest option and sufficient.
> >
> >
> > thanks,
> >
> > Takashi
> Hey,
>
> Patch below, can be applied immediately iresspective of the other patches.
>
> ---->8----------
>
> Selecting CONFIG_DRM selects CONFIG_VIDEO_NOMODESET, which exports
> video_firmware_drivers_only(). This can be used as a first approximation
> on whether i915 will be available. It's safe to use as this is only built

Please don't bury new patches in the middle of mails, it just makes it
hard to apply the patch since tooling doesn't understand what's going
on.

Please don't send new patches in reply to old patches or serieses, this
makes it harder for both people and tools to understand what is going
on - it can bury things in mailboxes and make it difficult to keep track
of what current patches are, both for the new patches and the old ones.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]