Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 19 Jul 2023 13:39:12 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc:     Takashi Iwai <tiwai@suse.de>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Alsa-devel <alsa-devel@alsa-project.org>,
        sound-open-firmware@alsa-project.org, linux-kernel@vger.kernel.org,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>,
        Cezary Rojewski <cezary.rojewski@intel.com>,
        Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <liam.r.girdwood@linux.intel.com>,
        Peter Ujfalusi <peter.ujfalusi@linux.intel.com>,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Ranjani Sridharan <ranjani.sridharan@linux.intel.com>,
        Daniel Baluta <daniel.baluta@nxp.com>,
        Matthew Auld <matthew.auld@intel.com>
Subject: Re: [PATCH] ASoC: SOF: Intel: Remove deferred probe for SOF
Message-ID: <04ed7ed8-8a8d-427a-84e1-a326feee5547@sirena.org.uk>
References: <20230718084522.116952-1-maarten.lankhorst@linux.intel.com>
 <20230718084522.116952-7-maarten.lankhorst@linux.intel.com>
 <alpine.DEB.2.22.394.2307181922160.3532114@eliteleevi.tm.intel.com>
 <874jm0modf.wl-tiwai@suse.de>
 <66d22637-069f-5262-2249-6041ef9d4614@linux.intel.com>
 <875y6g5feo.wl-tiwai@suse.de>
 <2ed288d0-c8fe-1856-dbe9-74f4f7c075ba@linux.intel.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="kvCIDaYOYoh3Q9Er"
Content-Disposition: inline
In-Reply-To: <2ed288d0-c8fe-1856-dbe9-74f4f7c075ba@linux.intel.com>
Precedence: bulk


--kvCIDaYOYoh3Q9Er
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 19, 2023 at 02:13:59PM +0200, Maarten Lankhorst wrote:
>=20
> On 2023-07-19 13:06, Takashi Iwai wrote:
> > On Wed, 19 Jul 2023 11:48:06 +0200,
> > Maarten Lankhorst wrote:
> > >=20
> > >      The 60 seconds timeout is a thing "better than complete disablem=
ent",
> > >      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.
> >=20
> > Yes, probably it's the easiest option and sufficient.
> >=20
> >=20
> > thanks,
> >=20
> > Takashi
> Hey,
>=20
> Patch below, can be applied immediately iresspective of the other patches.
>=20
> ---->8----------
>=20
> 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.

--kvCIDaYOYoh3Q9Er
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmS32W8ACgkQJNaLcl1U
h9DFNQf+JoAoRAzXMzLzPZ15cYtJAJA0zNP7JlIcMDsWKJPtZn8SzrU/jeCKhrJE
SV6C5/umfGMgpi7FOVjO+XNecsstrTTcX7QtWtOdGPHRuZn8rhjeEsu5zMwmBfgH
Ei02irX/xZdkgo1b0x0ju4zUrKHvbUMKOdtfNlHgJrTHmT4PdqJt7RLp9NJXViRG
Ev8us7xmWgvnBVGt8BLRC+Z5l/W44mJNAAWe7hM2XxWH0k6n3GkOL/d0elrdu9ac
QfLHy86WaxTzfShKxVwnZyRJnYC0+qoJGjxtT6y4Kxhl9zoA/vVHXzom+GIHrnLj
Koior/rT1HgDMVHnu07rdUEg8fM0jA==
=zqCu
-----END PGP SIGNATURE-----

--kvCIDaYOYoh3Q9Er--