On Fri, Nov 20, 2020 at 05:10:30PM +0000, Rojewski, Cezary wrote:
> On 2020-11-20 5:48 PM, Mark Brown wrote:

> > People care about any code that's in the kernel, especially people doing
> > anything treewide.  The fewer configurations people need to build to get
> > code coverage the better.

> Sure, but in this particular case there really shouldn't be "another
> option". If catpt is the sole option, why add intel-dsp-config
> dependency? The alternative shouldn't even exist in the kernel and be
> instead removed just like /haswell/ and /baytrail/ were.

If all the alternatives actually get removed then there'd be no need for
it, while they're there it is reasonable to have it - it does make it
easier for people like distros to try converting, it means they can
deploy the recommended setup without needing to ship new binaries to
people who run into trouble.  Besides TBH while there's several DSP
implementations in the tree having the code there makes it obvious that
this case works the same way as all the others to anyone looking at the
code.