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@xxxxxxx> >> --- >> 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@xxxxxxx"); >> +MODULE_DESCRIPTION("AMD ACP6.3 PS SDW DMA Driver"); >> +MODULE_LICENSE("GPL v2"); >> +MODULE_ALIAS("platform:" DRV_NAME);