public inbox for linux-arm-msm@vger.kernel.org
 help / color / mirror / Atom feed
From: Herve Codina <herve.codina@bootlin.com>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: "Martin Povišer" <povik+lin@cutebit.org>,
	"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
	"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
	"Andrei Simion" <andrei.simion@microchip.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Arnaud Pouliquen" <arnaud.pouliquen@foss.st.com>,
	"Baolin Wang" <baolin.wang@linux.alibaba.com>,
	"Bard Liao" <yung-chuan.liao@linux.intel.com>,
	"Benson Leung" <bleung@chromium.org>,
	"Biju Das" <biju.das.jz@bp.renesas.com>,
	"Cezary Rojewski" <cezary.rojewski@intel.com>,
	"Cheng-Yi Chiang" <cychiang@chromium.org>,
	"Chunyan Zhang" <zhang.lyra@gmail.com>,
	"Claudiu Beznea" <claudiu.beznea@tuxon.dev>,
	"Cyril Chao" <Cyril.Chao@mediatek.com>,
	"Dan Carpenter" <dan.carpenter@linaro.org>,
	"Daniel Baluta" <daniel.baluta@nxp.com>,
	"Daniel Mack" <daniel@zonque.org>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Frank Li" <Frank.Li@nxp.com>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	"Guenter Roeck" <groeck@chromium.org>,
	"Haojian Zhuang" <haojian.zhuang@gmail.com>,
	"Jaroslav Kysela" <perex@perex.cz>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Kai Vehmanen" <kai.vehmanen@linux.intel.com>,
	"Kunihiko Hayashi" <hayashi.kunihiko@socionext.com>,
	"Lad Prabhakar" <prabhakar.mahadev-lad.rj@bp.renesas.com>,
	"Lars-Peter Clausen" <lars@metafoo.de>,
	"Liam Girdwood" <lgirdwood@gmail.com>,
	"Mark Brown" <broonie@kernel.org>,
	"Masami Hiramatsu" <mhiramat@kernel.org>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"Max Filippov" <jcmvbkbc@gmail.com>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Michal Simek" <michal.simek@amd.com>,
	"Nicolas Ferre" <nicolas.ferre@microchip.com>,
	"Nicolin Chen" <nicoleotsuka@gmail.com>,
	"Oder Chiou" <oder_chiou@realtek.com>,
	"Olivier Moysan" <olivier.moysan@foss.st.com>,
	"Orson Zhai" <orsonzhai@gmail.com>,
	"Peter Ujfalusi" <peter.ujfalusi@linux.intel.com>,
	"Pierre-Louis Bossart" <pierre-louis.bossart@linux.dev>,
	"Ranjani Sridharan" <ranjani.sridharan@linux.intel.com>,
	"Ray Jui" <rjui@broadcom.com>,
	"Robert Jarzmik" <robert.jarzmik@free.fr>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Scott Branden" <sbranden@broadcom.com>,
	Sheetal <sheetal@nvidia.com>,
	"Shengjiu Wang" <shengjiu.wang@gmail.com>,
	"Srinivas Kandagatla" <srini@kernel.org>,
	"Sylwester Nawrocki" <s.nawrocki@samsung.com>,
	"Takashi Iwai" <tiwai@suse.com>,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Tim Bird" <tim.bird@sony.com>,
	"Tzung-Bi Shih" <tzungbi@kernel.org>,
	"Venkata Prasad Potturu" <venkataprasad.potturu@amd.com>,
	"Vijendar Mukunda" <Vijendar.Mukunda@amd.com>,
	"Vincenzo Frascino" <vincenzo.frascino@arm.com>,
	"Xiubo Li" <Xiubo.Lee@gmail.com>,
	chrome-platform@lists.linux.dev, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org,
	linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	linux-sound@vger.kernel.org
Subject: Re: [PATCH v2 01/29] ASoC: soc-component: re-add pcm_new()/pcm_free()
Date: Mon, 16 Mar 2026 16:56:37 +0100	[thread overview]
Message-ID: <20260316165637.7b09fa9b@bootlin.com> (raw)
In-Reply-To: <87a4w8lde4.wl-kuninori.morimoto.gx@renesas.com>

Hi Kuninori,

On Mon, 16 Mar 2026 02:24:43 +0000
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote:

> Because old pcm_new()/pcm_free() didn't care about parameter component,
> to avoid name collisions, we have added pcm_construct()/pcm_destruct() by
> commit c64bfc9066007 ("ASoC: soc-core: add new pcm_construct/pcm_destruct")
> 
> Because all driver switch to new pcm_construct()/pcm_destruct(), old
> pcm_new()/pcm_free() were remoted by commit e9067bb502787 ("ASoC:

s/remoted/removed

> soc-component: remove snd_pcm_ops from component driver")
> 
> But naming of pcm_construct()/pcm_destruct() are not goot. re-add
> pcm_new()/pcm_free(), and switch to use it, again.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>  include/sound/soc-component.h        |  4 ++++
>  sound/soc/generic/audio-graph-card.c |  1 +
>  sound/soc/soc-component.c            | 10 +++++++++-
>  3 files changed, 14 insertions(+), 1 deletion(-)

Nothing more than the typo,

Reviewed-by: Herve Codina <herve.codina@bootlin.com>

Best regards,
Hervé

  reply	other threads:[~2026-03-16 15:57 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-16  2:24 [PATCH v2 00/29] ASoC: soc-component: re-add pcm_new()/pcm_free() Kuninori Morimoto
2026-03-16  2:24 ` [PATCH v2 01/29] " Kuninori Morimoto
2026-03-16 15:56   ` Herve Codina [this message]
2026-03-16  2:24 ` [PATCH v2 02/29] ASoC: amd: name back to pcm_new()/pcm_free() Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 03/29] ASoC: apple: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 04/29] ASoC: atmel: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 05/29] ASoC: au1x: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 06/29] ASoC: bcm: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 07/29] ASoC: codecs: cros_ec_codec: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 08/29] ASoC: codecs: rt5xxx-spi: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 09/29] ASoC: dwc: " Kuninori Morimoto
2026-03-16  2:25 ` [PATCH v2 10/29] ASoC: fsl: " Kuninori Morimoto
2026-03-16 15:59   ` Herve Codina
2026-03-16  2:26 ` [PATCH v2 11/29] ASoC: generic: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 12/29] ASoC: google: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 13/29] ASoC: intel: " Kuninori Morimoto
2026-03-16 16:06   ` Cezary Rojewski
2026-03-16  2:26 ` [PATCH v2 14/29] ASoC: kirkwood: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 15/29] ASoC: loongson: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 16/29] ASoC: mediatek: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 17/29] ASoC: pxa: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 18/29] ASoC: qcom: " Kuninori Morimoto
2026-03-16  2:26 ` [PATCH v2 19/29] ASoC: renesas: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 20/29] ASoC: samsung: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 21/29] ASoC: soc-generic-dmaengine-pcm: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 22/29] ASoC: sof: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 23/29] ASoC: sprd: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 24/29] ASoC: stm: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 25/29] ASoC: tegra: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 26/29] ASoC: uniphier: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 27/29] ASoC: xilinx: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 28/29] ASoC: xtensa: " Kuninori Morimoto
2026-03-16  2:27 ` [PATCH v2 29/29] ASoC: soc-component: remove pcm_construct()/pcm_destruct() Kuninori Morimoto
2026-03-16 16:02   ` Herve Codina
2026-03-16 21:59 ` [PATCH v2 00/29] ASoC: soc-component: re-add pcm_new()/pcm_free() Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260316165637.7b09fa9b@bootlin.com \
    --to=herve.codina@bootlin.com \
    --cc=Cyril.Chao@mediatek.com \
    --cc=Frank.Li@nxp.com \
    --cc=Vijendar.Mukunda@amd.com \
    --cc=Xiubo.Lee@gmail.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=alexandre.torgue@foss.st.com \
    --cc=andrei.simion@microchip.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=arnaud.pouliquen@foss.st.com \
    --cc=baolin.wang@linux.alibaba.com \
    --cc=biju.das.jz@bp.renesas.com \
    --cc=bleung@chromium.org \
    --cc=broonie@kernel.org \
    --cc=cezary.rojewski@intel.com \
    --cc=chrome-platform@lists.linux.dev \
    --cc=claudiu.beznea@tuxon.dev \
    --cc=cychiang@chromium.org \
    --cc=dan.carpenter@linaro.org \
    --cc=daniel.baluta@nxp.com \
    --cc=daniel@zonque.org \
    --cc=festevam@gmail.com \
    --cc=geert+renesas@glider.be \
    --cc=groeck@chromium.org \
    --cc=haojian.zhuang@gmail.com \
    --cc=hayashi.kunihiko@socionext.com \
    --cc=imx@lists.linux.dev \
    --cc=jcmvbkbc@gmail.com \
    --cc=jonathanh@nvidia.com \
    --cc=kai.vehmanen@linux.intel.com \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=lars@metafoo.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=mhiramat@kernel.org \
    --cc=michal.simek@amd.com \
    --cc=nicolas.ferre@microchip.com \
    --cc=nicoleotsuka@gmail.com \
    --cc=oder_chiou@realtek.com \
    --cc=olivier.moysan@foss.st.com \
    --cc=orsonzhai@gmail.com \
    --cc=perex@perex.cz \
    --cc=peter.ujfalusi@linux.intel.com \
    --cc=pierre-louis.bossart@linux.dev \
    --cc=povik+lin@cutebit.org \
    --cc=prabhakar.mahadev-lad.rj@bp.renesas.com \
    --cc=ranjani.sridharan@linux.intel.com \
    --cc=rjui@broadcom.com \
    --cc=robert.jarzmik@free.fr \
    --cc=s.hauer@pengutronix.de \
    --cc=s.nawrocki@samsung.com \
    --cc=sbranden@broadcom.com \
    --cc=sheetal@nvidia.com \
    --cc=shengjiu.wang@gmail.com \
    --cc=srini@kernel.org \
    --cc=thierry.reding@gmail.com \
    --cc=tim.bird@sony.com \
    --cc=tiwai@suse.com \
    --cc=tzungbi@kernel.org \
    --cc=venkataprasad.potturu@amd.com \
    --cc=vincenzo.frascino@arm.com \
    --cc=yung-chuan.liao@linux.intel.com \
    --cc=zhang.lyra@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox