On Mon, 11 Feb 2019 at 16:19, Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> wrote:
>
> On 2/11/19 14:57, Krzysztof Kozlowski wrote:
> >> static struct snd_soc_dai_link odroid_card_dais[] = {
> >> + {
> >> + /* Primary FE <-> BE link */
> >> + .codec_name = "snd-soc-dummy",
> >> + .codec_dai_name = "snd-soc-dummy-dai",
> >> + .ops = &odroid_card_fe_ops,
> >> + .name = "Primary",
> >> + .stream_name = "Primary",
> >> + .platform_name = "3830000.i2s",
>
> > Why exposing address as platform_name? I think it is not used so how
> > about some friendlier name?
>
> This entry is for selecting "PCM" (DMA) component for the link.
> For proper matching we need to use names of devices for which
> the dmaengine based PCM component is registered (with a call to
> samsung_asoc_dma_platform_register()).
>
> We can't use platform_of_node because 2 PCM components ("3830000.i2s",
> "samsung-i2s-sec") are now associated with same DT node.
Thanks for explanation.
With the unneeded empty line fixup:
Acked-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Best regards,
Krzysztof
