Re: [RESENT PATCH v7 1/7] ASoC: rockchip: Use codec of_node and dai_name for rt5514 dsp

From: Matthias Kaehlcke
Date: Thu Aug 24 2017 - 17:19:59 EST


El Thu, Aug 24, 2017 at 12:52:21PM +0800 Jeffy Chen ha dit:

> Currently we are using codec name for rt5514 dsp dai link, use codec
> of_node instead.
>
> Signed-off-by: Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> ---

Reviewed-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>
Tested-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>
(The codec is still 'detected' on kevin, with the name spi32765.0)

> Changes in v7:
> Rebase on the newest for-next
>
> Changes in v6: None
> Changes in v3: None
> Changes in v2: None
>
> sound/soc/rockchip/rk3399_gru_sound.c | 34 ++--------------------------------
> 1 file changed, 2 insertions(+), 32 deletions(-)
>
> diff --git a/sound/soc/rockchip/rk3399_gru_sound.c b/sound/soc/rockchip/rk3399_gru_sound.c
> index 566ccb39fb31..cd2fdba922f1 100644
> --- a/sound/soc/rockchip/rk3399_gru_sound.c
> +++ b/sound/soc/rockchip/rk3399_gru_sound.c
> @@ -247,8 +247,6 @@ enum {
> DAILINK_RT5514_DSP,
> };
>
> -#define DAILINK_ENTITIES (DAILINK_DA7219 + 1)
> -
> static struct snd_soc_dai_link rockchip_dailinks[] = {
> [DAILINK_MAX98357A] = {
> .name = "MAX98357A",
> @@ -282,8 +280,7 @@ static struct snd_soc_dai_link rockchip_dailinks[] = {
> [DAILINK_RT5514_DSP] = {
> .name = "RT5514 DSP",
> .stream_name = "Wake on Voice",
> - .codec_name = "snd-soc-dummy",
> - .codec_dai_name = "snd-soc-dummy-dai",
> + .codec_dai_name = "rt5514-dsp-cpu-dai",
> },
> };
>
> @@ -300,17 +297,10 @@ static struct snd_soc_card rockchip_sound_card = {
> .num_controls = ARRAY_SIZE(rockchip_controls),
> };
>
> -static int rockchip_sound_match_stub(struct device *dev, void *data)
> -{
> - return 1;
> -}
> -
> static int rockchip_sound_probe(struct platform_device *pdev)
> {
> struct snd_soc_card *card = &rockchip_sound_card;
> struct device_node *cpu_node;
> - struct device *dev;
> - struct device_driver *drv;
> int i, ret;
>
> cpu_node = of_parse_phandle(pdev->dev.of_node, "rockchip,cpu", 0);
> @@ -319,7 +309,7 @@ static int rockchip_sound_probe(struct platform_device *pdev)
> return -EINVAL;
> }
>
> - for (i = 0; i < DAILINK_ENTITIES; i++) {
> + for (i = 0; i < ARRAY_SIZE(rockchip_dailinks); i++) {
> rockchip_dailinks[i].platform_of_node = cpu_node;
> rockchip_dailinks[i].cpu_of_node = cpu_node;
>
> @@ -332,22 +322,6 @@ static int rockchip_sound_probe(struct platform_device *pdev)
> }
> }
>
> - /**
> - * To acquire the spi driver of the rt5514 and set the dai-links names
> - * for soc_bind_dai_link
> - */
> - drv = driver_find("rt5514", &spi_bus_type);
> - if (!drv) {
> - dev_err(&pdev->dev, "Can not find the rt5514 driver at the spi bus\n");
> - return -EINVAL;
> - }
> -
> - dev = driver_find_device(drv, NULL, NULL, rockchip_sound_match_stub);
> - if (!dev) {
> - dev_err(&pdev->dev, "Can not find the rt5514 device\n");
> - return -ENODEV;
> - }
> -
> /* Set DMIC wakeup delay */
> ret = device_property_read_u32(&pdev->dev, "dmic-wakeup-delay-ms",
> &dmic_wakeup_delay);
> @@ -357,10 +331,6 @@ static int rockchip_sound_probe(struct platform_device *pdev)
> "no optional property 'dmic-wakeup-delay-ms' found, default: no delay\n");
> }
>
> - rockchip_dailinks[DAILINK_RT5514_DSP].cpu_name = kstrdup_const(dev_name(dev), GFP_KERNEL);
> - rockchip_dailinks[DAILINK_RT5514_DSP].cpu_dai_name = kstrdup_const(dev_name(dev), GFP_KERNEL);
> - rockchip_dailinks[DAILINK_RT5514_DSP].platform_name = kstrdup_const(dev_name(dev), GFP_KERNEL);
> -
> card->dev = &pdev->dev;
>
> ret = devm_snd_soc_register_card(&pdev->dev, card);