Re: [RFC][PATCH 0/4] ASoC: fsl: cleanup platform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 21, 2023 at 8:10 AM Kuninori Morimoto <
kuninori.morimoto.gx@xxxxxxxxxxx> wrote:

>
> Hi Mark
> Cc Shengjiu
>
> The driver which is using soc-generic-dmaengine-pcm is a little bit
> difficult to notice about it, because it prepares own Component,
> but shares same component->dev with CPU Component.
>
> Some fsl driver is using soc-generic-dmaengine-pcm as Platform Component.
>
> When binding CPU/Codec, it finds relevant Component, and checks target
> DAI. If it it was target DAI, ASoC binds its Component to Card.
> In Platform case, it finds relevant Component and bind it. It doesn't
> check DAI. This means, we can share CPU dlc with Platform in this case.
> It is more reasonable / understandable / intuitive.
>
> This patch-set share CPU dlc with Platform, and adds such comment.
>
> Because I can't test this patch-set, thus I added [RFC] on Subject.
> Please test these. This patch-set requests Tested-by.
>
> Link:
> https://lore.kernel.org/r/87o7ojjd06.wl-kuninori.morimoto.gx@xxxxxxxxxxx
> Link:
> https://lore.kernel.org/r/1681900158-17428-1-git-send-email-shengjiu.wang@xxxxxxx
>
>
Acked-by: Shengjiu Wang <shengjiu.wang@xxxxxxxxx>
Tested-by: Shengjiu Wang <shengjiu.wang@xxxxxxxxx>

Best regards
Wang shengjiu

Kuninori Morimoto (4):
>   ASoC: fsl: imx-es8328: cleanup platform which is using Generic DMA
>   ASoC: fsl: imx-spdif: cleanup platform which is using Generic DMA
>   ASoC: fsl: imx-audmix: cleanup platform which is using Generic DMA
>   ASoC: fsl: imx-audmix: remove dummy dai_link->platform
>
>  sound/soc/fsl/imx-audmix.c | 22 +++++++++++-----------
>  sound/soc/fsl/imx-es8328.c | 11 +++++++----
>  sound/soc/fsl/imx-spdif.c  | 11 +++++++----
>  3 files changed, 25 insertions(+), 19 deletions(-)
>
> --
> 2.25.1
>
>




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux