mbox series

[v2,0/4] ASoC: remove unnecessary dai_link->platform

Message ID 87pm8zjd0t.wl-kuninori.morimoto.gx@renesas.com
Headers show
Series ASoC: remove unnecessary dai_link->platform | expand

Message

Kuninori Morimoto March 23, 2023, 10:53 p.m. UTC
Hi Mark

Current ASoC will ignore already connected component when binding Card.
This will happen mainly "CPU Component" is handled as "Platform Component",
which was needed before.

	static int snd_soc_rtd_add_component(...)
	{
		...
		for_each_rtd_components(rtd, i, comp) {
			/* already connected */
			if (comp == component)
				return 0;
		}
		...
	}

Some drivers are still using CPU or Dummy Component as Platform Component,
but these are no meaning or ignored.
This patch-set remove these.

v1 -> v2

	- remove the patch for simple-card
	- remove extra simple-card patch
	- add Acked-by for atmel patch

Link: https://lore.kernel.org/r/878rfyjz4p.wl-kuninori.morimoto.gx@renesas.com

Kuninori Morimoto (4):
  ASoC: fsl: remove unnecessary dai_link->platform
  ASoC: atmel: remove unnecessary dai_link->platform
  ASoC: ti: remove unnecessary dai_link->platform
  ASoC: soc-topology.c: remove unnecessary dai_link->platform

 sound/soc/atmel/atmel-classd.c |  5 +----
 sound/soc/atmel/atmel-pdmic.c  |  5 +----
 sound/soc/fsl/imx-audmix.c     | 14 ++++----------
 sound/soc/fsl/imx-spdif.c      |  5 +----
 sound/soc/soc-topology.c       |  8 ++------
 sound/soc/ti/omap-hdmi.c       |  5 +----
 6 files changed, 10 insertions(+), 32 deletions(-)

Comments

Mark Brown March 24, 2023, 4:37 p.m. UTC | #1
On Thu, 23 Mar 2023 22:53:12 +0000, Kuninori Morimoto wrote:
> Current ASoC will ignore already connected component when binding Card.
> This will happen mainly "CPU Component" is handled as "Platform Component",
> which was needed before.
> 
> 	static int snd_soc_rtd_add_component(...)
> 	{
> 		...
> 		for_each_rtd_components(rtd, i, comp) {
> 			/* already connected */
> 			if (comp == component)
> 				return 0;
> 		}
> 		...
> 	}
> 
> [...]

Applied to

   broonie/sound.git for-next

Thanks!

[1/4] ASoC: fsl: remove unnecessary dai_link->platform
      commit: 33683cbf49b5412061cb1e4c876063fdef86def4
[2/4] ASoC: atmel: remove unnecessary dai_link->platform
      commit: 2f650f87c03cab72e751fc739f42a1e257bdc6b9
[3/4] ASoC: ti: remove unnecessary dai_link->platform
      commit: 3b0db249cf8fe0027e2a4161d27a8566d82fcd80
[4/4] ASoC: soc-topology.c: remove unnecessary dai_link->platform
      commit: e7098ba9b3785d626326040d300f95fec79aa765

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark