Re: [PATCH 3/7] ASoC: Intel: cht_bsw_max98090: add support for Baytrail

[Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>]

On 9/18/17 2:17 AM, Andy Shevchenko wrote:
> On Fri, 2017-09-08 at 00:13 -0500, Pierre-Louis Bossart wrote:
>> Distributions such as Fedora, Ubuntu and Gallium don't currently
>> have a means to support Baytrail Chromebooks and other platforms
>> with the same build [1][2] due to incompatible platform drivers.
>>
>> Add MCLK management to reuse this machine driver for Baytrail
>> platforms and solve this coexistence problem at last. UCM files are
>> provided at [3] and will eventually be submitted to the new repo.
>>
>> The legacy byt-max98090 machine driver is still maintained but can
>> only be used when the other Atom/DPCM driver is not compiled in, or
>> when users don't want to configure extra mixers required by the
>> Atom/sst driver.
>>
>> Tested on Lenovo 100s Baytrail Chromebook w/ Mr. Chromebox BOOT_STUB
>> firmware and Acer R11 Cherrytrail Chromebook
>>
>> [1] https://bugzilla.redhat.com/show_bug.cgi?id=1335196
>> [2] http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/
>> 111641.html
>> [3] https://github.com/plbossart/UCM/tree/master/byt-max98090
>>
>>
> 
>>   
>> +static inline struct snd_soc_dai *cht_get_codec_dai(struct
>> snd_soc_card *card)
>> +{
>> +	struct snd_soc_pcm_runtime *rtd;
>> +
>> +	list_for_each_entry(rtd, &card->rtd_list, list) {
>> +		if (!strncmp(rtd->codec_dai->name, CHT_CODEC_DAI,
>> +			     strlen(CHT_CODEC_DAI)))
> 
> Same comments as per another patch series wrt str_n_cmp() use.

Yes, if that's alright with you I'll clean this up in all Intel machine 
drivers in one patch since this is all copy-paste. I just need a clear 
indication on what the preferred means of string comparisons is...

> 
>> +			return rtd->codec_dai;
>> +	}
>> +	return NULL;
> 
>