lkml.org 
[lkml]   [2021]   [Jul]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH V3 05/12] ASoC: amd: add ACP5x PCM platform driver
From
Date
On 7/19/21 11:39 PM, Pierre-Louis Bossart wrote:
>
>> +static int acp5x_audio_probe(struct platform_device *pdev)
>> +{
>> + struct resource *res;
>> + struct i2s_dev_data *adata;
>> + int status;
>> +
>> + if (!pdev->dev.platform_data) {
>> + dev_err(&pdev->dev, "platform_data not retrieved\n");
>> + return -ENODEV;
>> + }
>> + irqflags = *((unsigned int *)(pdev->dev.platform_data));
>> +
>> + res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>> + if (!res) {
>> + dev_err(&pdev->dev, "IORESOURCE_MEM FAILED\n");
>> + return -ENODEV;
>> + }
>> +
>> + adata = devm_kzalloc(&pdev->dev, sizeof(*adata), GFP_KERNEL);
>> + if (!adata)
>> + return -ENOMEM;
>> +
>> + adata->acp5x_base = devm_ioremap(&pdev->dev, res->start,
>> + resource_size(res));
>> + if (!adata->acp5x_base)
>> + return -ENOMEM;
>> + dev_set_drvdata(&pdev->dev, adata);
>> + status = devm_snd_soc_register_component(&pdev->dev,
>> + &acp5x_i2s_component,
>> + NULL, 0);
>> + if (status) {
>> + dev_err(&pdev->dev, "Fail to register acp i2s component\n");
>> + return -ENODEV;
>
> Unclear why you need to change the error code explicitly to -ENODEV?
>
> return status?

yes, we can return status rather than forcing error as -ENODEV.
Will fix it and post the new version.
>
>
>> + }
>> + return 0;
>> +}
>> +

\
 
 \ /
  Last update: 2021-07-19 21:35    [W:0.175 / U:0.264 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site