From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org>
X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on
	aws-us-west-2-korg-lkml-1.web.codeaurora.org
Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by smtp.lore.kernel.org (Postfix) with ESMTPS id 21720C74A5B
	for <linuxppc-dev@archiver.kernel.org>; Thu, 16 Mar 2023 23:59:54 +0000 (UTC)
Received: from boromir.ozlabs.org (localhost [IPv6:::1])
	by lists.ozlabs.org (Postfix) with ESMTP id 4Pd44m25GNz3f4L
	for <linuxppc-dev@archiver.kernel.org>; Fri, 17 Mar 2023 10:59:52 +1100 (AEDT)
Authentication-Results: lists.ozlabs.org;
	dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=H9aXq6XW;
	dkim=fail reason="signature verification failed" header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=vrFmJJMH;
	dkim-atps=neutral
Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=suse.de (client-ip=195.135.220.28; helo=smtp-out1.suse.de; envelope-from=tiwai@suse.de; receiver=<UNKNOWN>)
Authentication-Results: lists.ozlabs.org;
	dkim=pass (1024-bit key; unprotected) header.d=suse.de header.i=@suse.de header.a=rsa-sha256 header.s=susede2_rsa header.b=H9aXq6XW;
	dkim=pass header.d=suse.de header.i=@suse.de header.a=ed25519-sha256 header.s=susede2_ed25519 header.b=vrFmJJMH;
	dkim-atps=neutral
Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
	 key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256)
	(No client certificate requested)
	by lists.ozlabs.org (Postfix) with ESMTPS id 4Pchqg0z84z3c34
	for <linuxppc-dev@lists.ozlabs.org>; Thu, 16 Mar 2023 20:32:14 +1100 (AEDT)
Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
	 key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512)
	(No client certificate requested)
	by smtp-out1.suse.de (Postfix) with ESMTPS id E58EF21852;
	Thu, 16 Mar 2023 09:32:10 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa;
	t=1678959130; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:
	 mime-version:mime-version:content-type:content-type:
	 content-transfer-encoding:content-transfer-encoding:
	 in-reply-to:in-reply-to:references:references;
	bh=LgQ6KC2ybaQuhcxvNbj5yWV84Hdg07Eok3qBPbbxY94=;
	b=H9aXq6XWzgXy3V6gUMaMgKEQOC4u2wwCwuM31XT6epw3vcSavDTpG7v1DmpLitT3aAzvrb
	Uij7YtTtCMzKsnoVFyM4P36fE8radT/BVqUmJO5oz33vf5WRPcRwuHN+AiCd+tOxzXuyQp
	7HO6F55yqLXCARRi8VELwLEQafIfemA=
DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de;
	s=susede2_ed25519; t=1678959130;
	h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:
	 mime-version:mime-version:content-type:content-type:
	 content-transfer-encoding:content-transfer-encoding:
	 in-reply-to:in-reply-to:references:references;
	bh=LgQ6KC2ybaQuhcxvNbj5yWV84Hdg07Eok3qBPbbxY94=;
	b=vrFmJJMHnJyGAG4vQo3kBbAm10wkR+7Kjjsf57BxURiziFHRuM5x12k5OToJ7bvAHK8XWM
	BGwhhqI50dx0VBCA==
Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
	 key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512)
	(No client certificate requested)
	by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id DE806133E0;
	Thu, 16 Mar 2023 09:32:08 +0000 (UTC)
Received: from dovecot-director2.suse.de ([192.168.254.65])
	by imap2.suse-dmz.suse.de with ESMTPSA
	id 9o+MNRjiEmRkRAAAMHmgww
	(envelope-from <tiwai@suse.de>); Thu, 16 Mar 2023 09:32:08 +0000
Date: Thu, 16 Mar 2023 10:32:08 +0100
Message-ID: <87ttylhwiv.wl-tiwai@suse.de>
From: Takashi Iwai <tiwai@suse.de>
To: Uwe =?ISO-8859-1?Q?Kleine-K=F6nig?= <u.kleine-koenig@pengutronix.de>
Subject: Re: [PATCH 000/173] ALSA/ASoC: Convert to platform remove callback returning void
In-Reply-To: <20230315150745.67084-1-u.kleine-koenig@pengutronix.de>
References: <20230315150745.67084-1-u.kleine-koenig@pengutronix.de>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
X-Mailman-Approved-At: Fri, 17 Mar 2023 10:59:01 +1100
X-BeenThere: linuxppc-dev@lists.ozlabs.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Linux on PowerPC Developers Mail List <linuxppc-dev.lists.ozlabs.org>
List-Unsubscribe: <https://lists.ozlabs.org/options/linuxppc-dev>,
 <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>
List-Archive: <http://lists.ozlabs.org/pipermail/linuxppc-dev/>
List-Post: <mailto:linuxppc-dev@lists.ozlabs.org>
List-Help: <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>
List-Subscribe: <https://lists.ozlabs.org/listinfo/linuxppc-dev>,
 <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>
Cc: Andrew Lunn <andrew@lunn.ch>, Heiko Stuebner <heiko@sntech.de>, Xiubo Li <Xiubo.Lee@gmail.com>, Moises Cardona <moisesmcardona@gmail.com>, Jaroslav Kysela <perex@perex.cz>, Liam Girdwood <liam.r.girdwood@linux.intel.com>, Muralidhar Reddy <muralidhar.reddy@intel.com>, Sylwester Nawrocki <s.nawrocki@samsung.com>, Gongjun Song <gongjun.song@intel.com>, Gaosheng Cui <cuigaosheng1@huawei.com>, Marek Szyprowski <m.szyprowski@samsung.com>, Samuel Holland <samuel@sholland.org>, Ranjani Sridharan <ranjani.sridharan@linux.intel.com>, NXP Linux Team <linux-imx@nxp.com>, Olivier Moysan <olivier.moysan@foss.st.com>, Martin =?ISO-8859-2?Q?Povi=B9er?= <povik+lin@cutebit.org>, Linus Walleij <linus.walleij@linaro.org>, Sascha Hauer <s.hauer@pengutronix.de>, Nicholas Piggin <npiggin@gmail.com>, James Schulman <james.schulman@cirrus.com>, syed saba kareem <syed.sabakareem@amd.com>, Thomas Gleixner <tglx@linutronix.de>, linux-omap@vger.kernel.org, Alexander Sverdlin <alexander.sverdlin@gmail.com>, 
 Allison Randal <allison@lohutok.net>, Scott Branden <sbranden@broadcom.com>, Geoff Levand <geoff@infradead.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Zhang Qilong <zhangqilong3@huawei.com>, linux-stm32@st-md-mailman.stormreply.com, Shang XiaoJing <shangxiaojing@huawei.com>, Masami Hiramatsu <mhiramat@kernel.org>, kernel@pengutronix.de, Li Chen <lchen@ambarella.com>, Claudiu Beznea <claudiu.beznea@microchip.com>, Jarkko Nikula <jarkko.nikula@bitmer.com>, Alexandre Belloni <alexandre.belloni@bootlin.com>, Lucas Tanure <tanureal@opensource.cirrus.com>, David Rhodes <david.rhodes@cirrus.com>, Max Filippov <jcmvbkbc@gmail.com>, Thierry Reding <thierry.reding@gmail.com>, Srinivas Kandagatla <srinivas.kandagatla@linaro.org>, Mario Limonciello <mario.limonciello@amd.com>, Liang He <windhl@126.com>, Ban Tao <fengzheng923@gmail.com>, Hezi Shahmoon <hezi@marvell.com>, Chunyan Zhang <zhang.lyra@gmail.com>, YueHaibing <yuehaibing@huawei.com>, Pierre-Louis Bossart <pierre-louis.bossar
 t@linux.intel.com>, Minghao Chi <chi.minghao@zte.com.cn>, Jonathan Hunter <jonathanh@nvidia.com>, linux-rockchip@lists.infradead.org, Vijendar Mukunda <Vijendar.Mukunda@amd.com>, Jerome Brunet <jbrunet@baylibre.com>, Colin Ian King <colin.i.king@gmail.com>, linux-xtensa@linux-xtensa.org, Charles Keepax <ckeepax@opensource.cirrus.com>, Aidan MacDonald <aidanmacdonald.0x0@gmail.com>, alsa-devel@alsa-project.org, Martin Blumenstingl <martin.blumenstingl@googlemail.com>, Haojian Zhuang <haojian.zhuang@gmail.com>, Peter Ujfalusi <peter.ujfalusi@gmail.com>, Biju Das <biju.das.jz@bp.renesas.com>, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Neil Armstrong <neil.armstrong@linaro.org>, Kai Vehmanen <kai.vehmanen@linux.intel.com>, Meng Tang <tangmeng@uniontech.com>, Banajit Goswami <bgoswami@quicinc.com>, Amadeusz =?ISO-8859-2?Q?S=B3awi=F1ski?= <amadeuszx.slawinski@linux.intel.com>, linux-mediatek@lists.infradead.org, linux-sunxi@lists.linux.dev, Cezary Rojewski <c
 ezary.rojewski@intel.com>, Dan Carpenter <error27@gmail.com>, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>, Sameer Pujar <spujar@nvidia.com>, Li kunyu <kunyu@nfschina.com>, Miaoqian Lin <linmq006@gmail.com>, Michal Simek <michal.simek@xilinx.com>, linux-tegra@vger.kernel.org, Neta Zur Hershkovits <neta@marvell.com>, Bard Liao <yung-chuan.liao@linux.intel.com>, Marcin Wojtas <mw@semihalf.com>, Kunihiko Hayashi <hayashi.kunihiko@socionext.com>, Jernej Skrabec <jernej.skrabec@gmail.com>, Chen-Yu Tsai <wens@csie.org>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, Kevin Hilman <khilman@baylibre.com>, Yang Yingliang <yangyingliang@huawei.com>, Orson Zhai <orsonzhai@gmail.com>, Codrin Ciubotariu <codrin.ciubotariu@microchip.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, Andrey Turkin <andrey.turkin@gmail.com>, Arnd Bergmann <arnd@arndb.de>, Ray Jui <rjui@broadcom.com>, Jiasheng Jiang <jiasheng@iscas.ac.cn>, Richard Fitzge
 rald <rf@opensource.cirrus.com>, Mark Brown <broonie@kernel.org>, Baolin Wang <baolin.wang@linux.alibaba.com>, Shuming Fan <shumingf@realtek.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Mohan Kumar <mkumard@nvidia.com>, Zhen Ni <nizhen@uniontech.com>, Liam Girdwood <lgirdwood@gmail.com>, Takashi Sakamoto <o-takashi@sakamocchi.jp>, Linh Phung <linh.phung.jy@renesas.com>, asahi@lists.linux.dev, Akihiko Odaki <akihiko.odaki@gmail.com>, Ricardo Ribalda <ribalda@chromium.org>, Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>, Shawn Guo <shawnguo@kernel.org>, Peter Rosin <peda@axentia.se>, Lior Amsalem <alior@marvell.com>, Alexandre Torgue <alexandre.torgue@foss.st.com>, Nicolas Ferre <nicolas.ferre@microchip.com>, V sujith kumar Reddy <Vsujithkumar.Reddy@amd.com>, Robert Jarzmik <robert.jarzmik@free.fr>, Jiapeng Chong <jiapeng.chong@linux.alibaba.com>, Zhu Ning <zhuning0077@gmail.com>, Shengjiu Wang <shengjiu.wang@gmail.com>, Nicolas Frattaroli <frattaroli.nicolas@gmail.com>, Trevor 
 Wu <trevor.wu@mediatek.com>, Yong Zhi <yong.zhi@intel.com>, Maxime Coquelin <mcoquelin.stm32@gmail.com>, Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>, Nicolin Chen <nicoleotsuka@gmail.com>, Hans de Goede <hdegoede@redhat.com>, Christophe JAILLET <christophe.jaillet@wanadoo.fr>, Fabio Estevam <festevam@gmail.com>, Matthias Brugger <matthias.bgg@gmail.com>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Peter Ujfalusi <peter.ujfalusi@linux.intel.com>, Mikhail Rudenko <mike.rudenko@gmail.com>, patches@opensource.cirrus.com, Takashi Iwai <tiwai@suse.com>, Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, linuxppc-dev@lists.ozlabs.org, Daniel Mack <daniel@zonque.org>
Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org
Sender: "Linuxppc-dev" <linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org>

On Wed, 15 Mar 2023 16:04:52 +0100,
Uwe Kleine-König wrote:
> 
> Hello,
> 
> this series adapts the platform drivers below sound/ to use the .remove_new()
> callback. Compared to the traditional .remove() callback .remove_new() returns
> no value. This is a good thing because the driver core doesn't (and cannot)
> cope for errors during remove. The only effect of a non-zero return value in
> .remove() is that the driver core emits a warning. The device is removed anyhow
> and an early return from .remove() usually yields a resource leak.
> 
> By changing the remove callback to return void driver authors cannot
> reasonably assume any more that there is some kind of cleanup later.
> 
> The first two patches simplify a driver each to return zero unconditionally,
> and then all drivers are trivially converted to .remove_new().
> 
> There are nearly no interdependencies in this patch set---only 1 <- 11 and
> 2 <- 16. So even if some individual problems are found (I don't expect that),
> the other patches can (and from my POV should) still be applied.
> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (173):
(snip)

For the whole series,

Acked-by: Takashi Iwai <tiwai@suse.de>


thanks,

Takashi