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.