Re: [PATCH 07/19] ASoC: amd: ps: add soundwire dma driver for pink sardine platform

On 11/01/23 20:52, Pierre-Louis Bossart wrote:

On 1/11/23 03:02, Vijendar Mukunda wrote:

...

Soundwire DMA platform driver binds to the platform device created by ACP PCI device. Soundwire DMA driver registers ALSA DMA component with ASoC framework.

Signed-off-by: Vijendar Mukunda Vijendar.Mukunda@amd.com

sound/soc/amd/ps/acp63.h | 5 +++ sound/soc/amd/ps/ps-sdw-dma.c | 72 +++++++++++++++++++++++++++++++++++ 2 files changed, 77 insertions(+) create mode 100644 sound/soc/amd/ps/ps-sdw-dma.c

diff --git a/sound/soc/amd/ps/acp63.h b/sound/soc/amd/ps/acp63.h index 0bd9dc363461..b462320fdf2a 100644 --- a/sound/soc/amd/ps/acp63.h +++ b/sound/soc/amd/ps/acp63.h @@ -135,3 +135,8 @@ struct acp63_dev_data { bool is_sdw_dev; bool acp_sdw_power_off; };

+struct sdw_dma_dev_data {

• void __iomem *acp_base;
• struct mutex *acp_lock;

+}; diff --git a/sound/soc/amd/ps/ps-sdw-dma.c b/sound/soc/amd/ps/ps-sdw-dma.c new file mode 100644 index 000000000000..388a4b7df715 --- /dev/null +++ b/sound/soc/amd/ps/ps-sdw-dma.c @@ -0,0 +1,72 @@ +// SPDX-License-Identifier: GPL-2.0+ +/*

• • AMD ALSA SoC Pink Sardine Soundwire DMA Driver
• • Copyright 2023 Advanced Micro Devices, Inc.
• */

+#include <linux/err.h> +#include <linux/io.h> +#include <linux/module.h> +#include <linux/platform_device.h> +#include <sound/pcm_params.h> +#include <sound/soc.h> +#include <sound/soc-dai.h> +#include "acp63.h"

+#define DRV_NAME "amd_ps_sdw_dma"

+static const struct snd_soc_component_driver acp63_sdw_component = {

• .name = DRV_NAME,

+};

+static int acp63_sdw_platform_probe(struct platform_device *pdev) +{

• struct resource *res;
• struct sdw_dma_dev_data *sdw_data;
• int status;
• if (!pdev->dev.platform_data) {

• dev_err(&pdev->dev, "platform_data not retrieved\n");
  

• return -ENODEV;
  

• }
• res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
• if (!res) {

• dev_err(&pdev->dev, "IORESOURCE_MEM FAILED\n");
  

• return -ENODEV;
  

• }
• sdw_data = devm_kzalloc(&pdev->dev, sizeof(*sdw_data), GFP_KERNEL);
• if (!sdw_data)

• return -ENOMEM;
  

• sdw_data->acp_base = devm_ioremap(&pdev->dev, res->start, resource_size(res));
• if (!sdw_data->acp_base)

• return -ENOMEM;
  

• sdw_data->acp_lock = pdev->dev.platform_data;
• dev_set_drvdata(&pdev->dev, sdw_data);
• status = devm_snd_soc_register_component(&pdev->dev,

• 				 &acp63_sdw_component,
  

• 				 NULL, 0);
  

• if (status) {

• dev_err(&pdev->dev, "Fail to register acp pdm dai\n");
  

not sure what this means? Are you registering a PDM component or a DMA one?

It's my bad. We will correct the log statement.

...

• return -ENODEV;
  

• }
• return 0;

+}

+static struct platform_driver acp63_sdw_dma_driver = {

• .probe = acp63_sdw_platform_probe,
• .driver = {

• .name = "amd_ps_sdw_dma",
  

• },

+};

+module_platform_driver(acp63_sdw_dma_driver);

+MODULE_AUTHOR("Vijendar.Mukunda@amd.com"); +MODULE_DESCRIPTION("AMD ACP6.3 PS SDW DMA Driver"); +MODULE_LICENSE("GPL v2"); +MODULE_ALIAS("platform:" DRV_NAME);