<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body>
<pre>Hello Mark,
</pre>
<pre>
</pre>
<div class="moz-cite-prefix">On 10/2/20 5:41 PM, Mark Brown wrote:<br>
</div>
<blockquote type="cite" cite="mid:20201002154107.GC5527@sirena.org.uk">
<pre class="moz-quote-pre" wrap="">On Fri, Oct 02, 2020 at 05:29:04PM +0200, Olivier Moysan wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">When configured as module, CS42L51 codec driver uses two modules
snd-soc-cs42l51 and snd-soc-cs42l51-i2c.
Add soft dependency on snd-soc-cs42l51-i2c in snd-soc-cs42l51,
to allow smart module dependency solving.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
Doesn't the userspace tooling usually manage to figure this out from
symbol usage?
</pre>
</blockquote>
<br>
<pre>cs42l51.c does not use symbols from cs42l51-i2c.c, so cs42l51-i2c does not appear
as a software dependency of cs42l51, for tools such as depmod.
(cs42l51-i2c.c uses symbols from cs42l51.c, 
so there is a dependency in this way, but this does not help here)
</pre>
<pre>When enabling a sound card based on cs42l51, all required modules are loaded automatically,
excepted snd-soc-cs42l51-i2c module. This one has to be inserted explicitely.

With the use of softdep, cs42l51-i2c appears as a dependency of cs42l51 for depmod,
which allows the probe of snd-soc-cs42l51-i2c.
</pre>
<pre>If you think there is a better way to manage this dependency please let me know.

Regards
Olivier
</pre>
</body>
</html>