[Sound-open-firmware] [PATCH 3/5] DMIC: add empty get_loopback_mode function

Pan, Xiuli xiuli.pan at linux.intel.com
Wed Jun 20 08:00:20 CEST 2018



On 6/19/2018 19:03, Liam Girdwood wrote:
> On Tue, 2018-06-19 at 17:22 +0800, Xiuli Pan wrote:
>>   
>> +static inline int dmic_get_loopback_mode(struct dai *dai)
>> +{
>> +       return -EINVAL;
>> +}
>> +
>>   const struct dai_ops dmic_ops = {
>>          .trigger = dmic_trigger,
>>          .set_config = dmic_set_config,
>> @@ -1360,6 +1365,7 @@ const struct dai_ops dmic_ops = {
>>          .pm_context_restore = dmic_context_restore,
>>          .probe = dmic_probe,
>>          .set_loopback_mode = dmic_set_loopback_mode,
>> +       .get_loopback_mode = dmic_get_loopback_mode,
>
> Lest get rid of set/get loopback mode and use the standard component cmd APIs,
> I'm sure I've said this in an earlier version of this patch.
This is just helper function for COMP cmd APIs
We need to handle the DAI. So this wrapper is used for COMP access DAI.
The comp CMD APIs is in the 5/5 patch.

Or you suggest we just directly access dai data and change register 
without a wrapper?

Thanks
Xiuli

>
> Liam
>
>>   };
>>   
> _______________________________________________
> Sound-open-firmware mailing list
> Sound-open-firmware at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/sound-open-firmware



More information about the Sound-open-firmware mailing list