On 02/11/2020 11.25, zhangqilong wrote:
> Hi
>
>>
>> Hi,
>>
>> On 02/11/2020 11.07, Peter Ujfalusi wrote:
>>>
>>>
>>> On 02/11/2020 10.54, Zhang Qilong wrote:
>>>> Forgetting to call pm_runtime_disable if davinci_mcasp_get_dt_params
>>>> failed in function davinci_mcasp_probe, and we should go to error
>>>> handling branch.
>>>
>>> Acked-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
>>
>> Actually...
>>
>>>
>>>> Signed-off-by: Zhang Qilong <zhangqilong3@xxxxxxxxxx>
>>>> ---
>>>> Changelog:
>>>> V2
>>>> - changed the subject and fixed the description.
>>>> ---
>>>> sound/soc/ti/davinci-mcasp.c | 6 ++++--
>>>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/sound/soc/ti/davinci-mcasp.c
>>>> b/sound/soc/ti/davinci-mcasp.c index 4b46dd827f3f..ec90c23c9e20
>>>> 100644
>>>> --- a/sound/soc/ti/davinci-mcasp.c
>>>> +++ b/sound/soc/ti/davinci-mcasp.c
>>>> @@ -2362,8 +2362,10 @@ static int davinci_mcasp_probe(struct
>> platform_device *pdev)
>>>> goto err;
>>>>
>>>> ret = davinci_mcasp_get_dt_params(mcasp);
>>>> - if (ret)
>>>> - return -EINVAL;
>>>> + if (ret) {
>>>> + ret = -EINVAL;
>>>> + goto err;
>>
>> do not override the error code from davinci_mcasp_get_dt_params(), jut jump
>> to err.
>>
>
> I have ever thought this point. I can't make sure it's necessary, davinci_mcasp_get_dt_params always return 0 now.
> If it's necessary, I can submit a new patch to fix it. Look forward to your reply sincerely.
right.
davinci_mcasp_get_dt_params() should be void and we don't need the check
in probe for return value.
auxclk-fs-ratio is optional property.
>
> Thanks, best wish!
> Zhang Qilong
>
>>>> + }
>>>>
>>>> ret = devm_snd_soc_register_component(&pdev->dev,
>>>> &davinci_mcasp_component,
>>>>
>>>
>>> - Péter
>>>
>>> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
>>> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
>>>
>>
>> - Péter
>>
>> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
>> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
- Péter
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
