mbox series

[00/44] ASoC: no Platform support

Message ID 877e9ydz81.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
Headers show
Series ASoC: no Platform support | expand

Message

Kuninori Morimoto June 7, 2019, 2:21 a.m. UTC 
Hi Mark

Some drivers don't need to have special Platform component,
but was mandatory at legacy style ALSA SoC.
Thus, there is this kind of settings on some drivers.

	dai_link->platform_of_node = dai_link->cpu_of_node;

This means, Platform == CPU.
But it will be ignored, because it is already added when CPU bindings.
Thus, this settings is not needed.
This patchset supports no Platform.

Last patch removes dummy platform which is no longer needed.
I think we don't need it anymore, but, should we keep it... ?

Kuninori Morimoto (44):
  ASoC: soc-core: allow no Platform on dai_link
  ASoC: vc4: vc4_hdmi: don't select unnecessary Platform
  ASoC: atmel: atmel-classd: don't select unnecessary Platform
  ASoC: atmel: atmel-pdmic: don't select unnecessary Platform
  ASoC: atmel: atmel_wm8904: don't select unnecessary Platform
  ASoC: atmel: mikroe-proto: don't select unnecessary Platform
  ASoC: atmel: sam9g20_wm8731: don't select unnecessary Platform
  ASoC: atmel: sam9x5_wm8731: don't select unnecessary Platform
  ASoC: atmel: tse850-pcm5142: don't select unnecessary Platform
  ASoC: fsl: eukrea-tlv320: don't select unnecessary Platform
  ASoC: fsl: fsl-asoc-card: don't select unnecessary Platform
  ASoC: fsl: imx-es8328: don't select unnecessary Platform
  ASoC: fsl: imx-sgtl5000: don't select unnecessary Platform
  ASoC: fsl: imx-spdif: don't select unnecessary Platform
  ASoC: fsl: imx-audmix: don't select unnecessary Platform
  ASoC: kirkwood: armada-370-db: don't select unnecessary Platform
  ASoC: mxs: mxs-sgtl5000: don't select unnecessary Platform
  ASoC: qcom: apq8016_sbc: don't select unnecessary Platform
  ASoC: qcom: storm: don't select unnecessary Platform
  ASoC: rockchip: rk3288_hdmi_analog: don't select unnecessary Platform
  ASoC: rockchip: rockchip_max98090: don't select unnecessary Platform
  ASoC: rockchip: rockchip_rt5645: don't select unnecessary Platform
  ASoC: samsung: arndale_rt5631: don't select unnecessary Platform
  ASoC: samsung: smdk_wm8994: don't select unnecessary Platform
  ASoC: samsung: snow: don't select unnecessary Platform
  ASoC: samsung: tm2_wm5110: don't select unnecessary Platform
  ASoC: sirf: sirf-audio: don't select unnecessary Platform
  ASoC: sunxi: sun4i-codec: don't select unnecessary Platform
  ASoC: tegra: tegra_alc5632: don't select unnecessary Platform
  ASoC: tegra: tegra_max98090: don't select unnecessary Platform
  ASoC: tegra: tegra_rt5640: don't select unnecessary Platform
  ASoC: tegra: tegra_rt5677: don't select unnecessary Platform
  ASoC: tegra: tegra_sgtl5000: don't select unnecessary Platform
  ASoC: tegra: tegra_wm8753: don't select unnecessary Platform
  ASoC: tegra: tegra_wm8903: don't select unnecessary Platform
  ASoC: tegra: tegra_wm9712: don't select unnecessary Platform
  ASoC: tegra: trimslice: don't select unnecessary Platform
  ASoC: ti: davinci-evm: don't select unnecessary Platform
  ASoC: ti: omap-abe-twl6040: don't select unnecessary Platform
  ASoC: ti: omap-hdmi: don't select unnecessary Platform
  ASoC: ti: omap-twl4030: don't select unnecessary Platform
  ASoC: ti: rx51: don't select unnecessary Platform
  ASoC: ux500: mop500: don't select unnecessary Platform
  ASoC: soc-utils: remove dummy Platform

 drivers/gpu/drm/vc4/vc4_hdmi.c          |  4 ---
 include/sound/soc.h                     |  2 +-
 sound/soc/atmel/atmel-classd.c          |  5 +--
 sound/soc/atmel/atmel-pdmic.c           |  5 +--
 sound/soc/atmel/atmel_wm8904.c          |  4 +--
 sound/soc/atmel/mikroe-proto.c          |  7 ++--
 sound/soc/atmel/sam9g20_wm8731.c        |  5 +--
 sound/soc/atmel/sam9x5_wm8731.c         |  5 +--
 sound/soc/atmel/tse850-pcm5142.c        |  4 +--
 sound/soc/fsl/eukrea-tlv320.c           |  5 +--
 sound/soc/fsl/fsl-asoc-card.c           |  6 +---
 sound/soc/fsl/imx-audmix.c              | 14 +++-----
 sound/soc/fsl/imx-es8328.c              |  5 +--
 sound/soc/fsl/imx-sgtl5000.c            |  5 +--
 sound/soc/fsl/imx-spdif.c               |  5 +--
 sound/soc/kirkwood/armada-370-db.c      | 12 ++-----
 sound/soc/mxs/mxs-sgtl5000.c            |  8 ++---
 sound/soc/qcom/apq8016_sbc.c            | 10 ++----
 sound/soc/qcom/storm.c                  |  4 +--
 sound/soc/rockchip/rk3288_hdmi_analog.c |  5 +--
 sound/soc/rockchip/rockchip_max98090.c  |  5 +--
 sound/soc/rockchip/rockchip_rt5645.c    |  5 +--
 sound/soc/samsung/arndale_rt5631.c      |  6 +---
 sound/soc/samsung/smdk_wm8994.c         |  9 ++---
 sound/soc/samsung/snow.c                | 10 ++----
 sound/soc/samsung/tm2_wm5110.c          | 12 ++-----
 sound/soc/sirf/sirf-audio.c             |  5 +--
 sound/soc/soc-core.c                    | 64 +++++++++++++++++----------------
 sound/soc/soc-utils.c                   | 25 -------------
 sound/soc/sunxi/sun4i-codec.c           |  5 +--
 sound/soc/tegra/tegra_alc5632.c         |  7 +---
 sound/soc/tegra/tegra_max98090.c        |  5 +--
 sound/soc/tegra/tegra_rt5640.c          |  5 +--
 sound/soc/tegra/tegra_rt5677.c          |  6 +---
 sound/soc/tegra/tegra_sgtl5000.c        |  7 +---
 sound/soc/tegra/tegra_wm8753.c          |  5 +--
 sound/soc/tegra/tegra_wm8903.c          |  5 +--
 sound/soc/tegra/tegra_wm9712.c          |  5 +--
 sound/soc/tegra/trimslice.c             |  6 +---
 sound/soc/ti/davinci-evm.c              |  5 +--
 sound/soc/ti/omap-abe-twl6040.c         | 22 +++++-------
 sound/soc/ti/omap-hdmi.c                |  5 +--
 sound/soc/ti/omap-twl4030.c             | 12 ++-----
 sound/soc/ti/rx51.c                     |  5 +--
 sound/soc/ux500/mop500.c                |  8 ++---
 45 files changed, 98 insertions(+), 276 deletions(-)

Comments

Kuninori Morimoto June 7, 2019, 3:57 a.m. UTC | #1 
Hi Mark

> Some drivers don't need to have special Platform component,
> but was mandatory at legacy style ALSA SoC.
> Thus, there is this kind of settings on some drivers.
> 
> 	dai_link->platform_of_node = dai_link->cpu_of_node;
> 
> This means, Platform == CPU.
> But it will be ignored, because it is already added when CPU bindings.
> Thus, this settings is not needed.
> This patchset supports no Platform.
> 
> Last patch removes dummy platform which is no longer needed.
> I think we don't need it anymore, but, should we keep it... ?

I noticed few patches were missed.
I will post these


Thank you for your help !!
Best regards
---
Kuninori Morimoto