linux-kernel - Re: [alsa-devel] [PATCH v8 2/6] ASoC: amd: Refactoring of DAI from DMA driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 20 Nov 2019 12:51:43 +0530
From:   vishnu <vravulap@....com>
To:     Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
        Ravulapati Vishnu vardhan rao 
        <Vishnuvardhanrao.Ravulapati@....com>
Cc:     "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." 
        <alsa-devel@...a-project.org>,
        Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
        "Gustavo A. R. Silva" <gustavo@...eddedor.com>,
        open list <linux-kernel@...r.kernel.org>,
        Takashi Iwai <tiwai@...e.com>,
        YueHaibing <yuehaibing@...wei.com>,
        Liam Girdwood <lgirdwood@...il.com>, Akshu.Agrawal@....com,
        Mark Brown <broonie@...nel.org>, djkurtz@...gle.com,
        Vijendar Mukunda <Vijendar.Mukunda@....com>,
        Alexander.Deucher@....com
Subject: Re: [alsa-devel] [PATCH v8 2/6] ASoC: amd: Refactoring of DAI from
 DMA driver



On 19/11/19 7:23 PM, Pierre-Louis Bossart wrote:
> 
> 
>> +static int acp3x_dai_probe(struct platform_device *pdev)
>> +{
>> +    struct resource *res;
>> +    struct i2s_dev_data *adata;
>> +    int status;
>> +
>> +    adata = devm_kzalloc(&pdev->dev, sizeof(struct i2s_dev_data),
>> +            GFP_KERNEL);
>> +    if (!adata)
>> +        return -ENOMEM;
>> +
>> +    res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>> +    if (!res) {
>> +        dev_err(&pdev->dev, "IORESOURCE_MEM FAILED\n");
>> +        return -ENOMEM;
>> +    }
>> +
>> +    adata->acp3x_base = devm_ioremap(&pdev->dev, res->start,
>> +            resource_size(res));
>> +    if (IS_ERR(adata->acp3x_base))
>> +        return PTR_ERR(adata->acp3x_base);
>> +
>> +    adata->i2s_irq = res->start;
>> +    dev_set_drvdata(&pdev->dev, adata);
>> +    status = devm_snd_soc_register_component(&pdev->dev,
>> +            &acp3x_dai_component, &acp3x_i2s_dai, 1);
>> +    if (status) {
>> +        dev_err(&pdev->dev, "Fail to register acp i2s dai\n");
>> +        return -ENODEV;
>> +    }
>> +    pm_runtime_set_autosuspend_delay(&pdev->dev, 5000);
>> +    pm_runtime_use_autosuspend(&pdev->dev);
>> +    pm_runtime_enable(&pdev->dev);
> 
> question: here you want to use pm_runtime for this platform device...
> 
>> +    return 0;
>> +}
>> +
>> +static int acp3x_dai_remove(struct platform_device *pdev)
>> +{
>> +    pm_runtime_disable(&pdev->dev);
>> +    return 0;
>> +}
>> +static struct platform_driver acp3x_dai_driver = {
>> +    .probe = acp3x_dai_probe,
>> +    .remove = acp3x_dai_remove,
>> +    .driver = {
>> +        .name = "acp3x_i2s_playcap",
> 
> ... but here there is no .pm structure and I don't see any 
> suspend/resume routines for this driver...
> 
>> +    },
>> +};
> 
>> @@ -774,13 +586,14 @@ static struct platform_driver acp3x_dma_driver = {
>>       .probe = acp3x_audio_probe,
>>       .remove = acp3x_audio_remove,
>>       .driver = {
>> -        .name = "acp3x_rv_i2s",
>> +        .name = "acp3x_rv_i2s_dma",
>>           .pm = &acp3x_pm_ops,
>>       },
> 
> ... but for this other platform_driver you do have a .pm structure and 
> suspend-resume implementations.
> 
> Wondering if this is a miss or a feature?
> 

As per our design, ACP IP specific changes like ACP power on/off will be 
handled in ACP pci driver(parent device for DMA device and I2S 
controller(nothing but CPU DAI))

Where as In DMA driver during runtime suspend/resume interrupts will be 
disabled and enabled.

But in DAI driver there is nothing to be done in suspend and resume just 
returning zero so we have not added PM suspend/resume here in DAI.

So is it expected to add the suspend resumes with returning zero.Or if 
pm runtime is not needed in CPU DAI shall we remove the existing PM 
related calls in DAI.

Please suggest us.


Thanks,
Vishnu