[PATCH 000/117] media: Convert to platform remove callback returning void

[Biju Das]

Hi Uwe,

> -----Original Message-----
> From: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> Sent: Monday, April 17, 2023 8:31 AM
> To: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Cc: Heiko Stuebner <heiko at sntech.de>; Eddie James <eajames at linux.ibm.com>;
> Hans Verkuil <hverkuil at xs4all.nl>; Alim Akhtar <alim.akhtar at samsung.com>;
> Dmitry Osipenko <digetx at gmail.com>; linux-stm32 at st-md-
> mailman.stormreply.com; Marek Szyprowski <m.szyprowski at samsung.com>; linux-
> samsung-soc at vger.kernel.org; Robert Foss <rfoss at kernel.org>; Dafna
> Hirschfeld <dafna at fastmail.com>; Samuel Holland <samuel at sholland.org>; Kevin
> Hilman <khilman at baylibre.com>; Michal Simek <michal.simek at xilinx.com>; Antti
> Palosaari <crope at iki.fi>; NXP Linux Team <linux-imx at nxp.com>; Jerome Brunet
> <jbrunet at baylibre.com>; linux-sunxi at lists.linux.dev; ye xingchen
> <ye.xingchen at zte.com.cn>; Sascha Hauer <s.hauer at pengutronix.de>; Łukasz
> Stelmach <l.stelmach at samsung.com>; Eugen Hristev
> <eugen.hristev at collabora.com>; Shuah Khan <skhan at linuxfoundation.org>; Hyun
> Kwon <hyun.kwon at xilinx.com>; Andrew Jeffery <andrew at aj.id.au>; Michael
> Tretter <m.tretter at pengutronix.de>; Moudy Ho <moudy.ho at mediatek.com>;
> kernel at pengutronix.de; Hans Verkuil <hverkuil-cisco at xs4all.nl>; Claudiu
> Beznea <claudiu.beznea at microchip.com>; Ming Qian <ming.qian at nxp.com>;
> Andrew-CT Chen <andrew-ct.chen at mediatek.com>; Alexandre Belloni
> <alexandre.belloni at bootlin.com>; Sylwester Nawrocki
> <s.nawrocki at samsung.com>; linux-aspeed at lists.ozlabs.org; Yunfei Dong
> <yunfei.dong at mediatek.com>; Lad, Prabhakar <prabhakar.csengg at gmail.com>;
> Thierry Reding <thierry.reding at gmail.com>; Guenter Roeck
> <groeck at chromium.org>; chrome-platform at lists.linux.dev; Jonathan Hunter
> <jonathanh at nvidia.com>; linux-rockchip at lists.infradead.org; Fabien Dessenne
> <fabien.dessenne at foss.st.com>; Ezequiel Garcia
> <ezequiel at vanguardiasur.com.ar>; Alain Volmat <alain.volmat at foss.st.com>;
> Andrzej Pietrasiewicz <andrzejtp2010 at gmail.com>; Colin Ian King
> <colin.i.king at gmail.com>; linux-media at vger.kernel.org; Jacopo Mondi
> <jacopo at jmondi.org>; Rory Liu <hellojacky0226 at hotmail.com>; Martin
> Blumenstingl <martin.blumenstingl at googlemail.com>; linux-arm-
> msm at vger.kernel.org; Sean Wang <sean.wang at mediatek.com>; Maxime Ripard
> <mripard at kernel.org>; Fabrizio Castro <fabrizio.castro.jz at renesas.com>;
> linux-amlogic at lists.infradead.org; linux-arm-kernel at lists.infradead.org;
> Neil Armstrong <neil.armstrong at linaro.org>; Zhou Peng <eagle.zhou at nxp.com>;
> Paul Kocialkowski <paul.kocialkowski at bootlin.com>; Maxime Coquelin
> <mcoquelin.stm32 at gmail.com>; linux-mediatek at lists.infradead.org; Jacek
> Anaszewski <jacek.anaszewski at gmail.com>; Dan Carpenter <error27 at gmail.com>;
> Sean Young <sean at mess.org>; Xavier Roumegue <xavier.roumegue at oss.nxp.com>;
> Ettore Chimenti <ek5.chimenti at gmail.com>; Vikash Garodia
> <quic_vgarodia at quicinc.com>; linux-tegra at vger.kernel.org; Eduardo Valentin
> <edubezval at gmail.com>; Andrzej Hajda <andrzej.hajda at intel.com>; Fabio
> Estevam <festevam at gmail.com>; Jean-Christophe Trotin <jean-
> christophe.trotin at foss.st.com>; Stanimir Varbanov
> <stanimir.k.varbanov at gmail.com>; Kieran Bingham
> <kieran.bingham at ideasonboard.com>; Jernej Skrabec
> <jernej.skrabec at gmail.com>; Chen-Yu Tsai <wens at csie.org>; Jacob Chen <jacob-
> chen at iotwrt.com>; Joel Stanley <joel at jms.id.au>; Yang Yingliang
> <yangyingliang at huawei.com>; Patrice Chotard <patrice.chotard at foss.st.com>;
> Bin Liu <bin.liu at mediatek.com>; Nathan Chancellor <nathan at kernel.org>;
> Sylwester Nawrocki <sylvester.nawrocki at gmail.com>; Mauro Carvalho Chehab
> <mchehab at kernel.org>; Benson Leung <bleung at chromium.org>; Daniel W. S.
> Almeida <dwlsalmeida at gmail.com>; Qiheng Lin <linqiheng at huawei.com>; Konrad
> Dybcio <konrad.dybcio at linaro.org>; Kieran Bingham
> <kieran.bingham+renesas at ideasonboard.com>; Yang Li
> <yang.lee at linux.alibaba.com>; Sakari Ailus <sakari.ailus at linux.intel.com>;
> Ricardo Ribalda <ribalda at chromium.org>; Shawn Guo <shawnguo at kernel.org>;
> Minghsiu Tsai <minghsiu.tsai at mediatek.com>; Daniel Almeida
> <daniel.almeida at collabora.com>; Alexandre Torgue
> <alexandre.torgue at foss.st.com>; Todor Tomov <todor.too at gmail.com>; Mirela
> Rabulea <mirela.rabulea at nxp.com>; Ajye Huang <ajye_huang at compal.corp-
> partner.google.com>; Scott Chao <scott_chao at wistron.corp-
> partner.google.com>; linux-renesas-soc at vger.kernel.org; Hugues Fruchet
> <hugues.fruchet at foss.st.com>; openbmc at lists.ozlabs.org; Andy Gross
> <agross at kernel.org>; Mikhail Ulyanov <mikhail.ulyanov at cogentembedded.com>;
> Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj at bp.renesas.com>; Benoit
> Parrot <bparrot at ti.com>; Rui Miguel Silva <rmfrfs at gmail.com>; Christophe
> JAILLET <christophe.jaillet at wanadoo.fr>; Yong Deng <yong.deng at magewell.com>;
> Matthias Brugger <matthias.bgg at gmail.com>; Tiffany Lin
> <tiffany.lin at mediatek.com>; AngeloGioacchino Del Regno
> <angelogioacchino.delregno at collabora.com>; Bjorn Andersson
> <andersson at kernel.org>; Nicolas Ferre <nicolas.ferre at microchip.com>; Houlong
> Wei <houlong.wei at mediatek.com>; Krzysztof Kozlowski
> <krzysztof.kozlowski at linaro.org>; Philipp Zabel <p.zabel at pengutronix.de>;
> Niklas Söderlund <niklas.soderlund at ragnatech.se>
> Subject: Re: [PATCH 000/117] media: Convert to platform remove callback
> returning void
> 
> Hello Laurent,
> 
> On Mon, Apr 17, 2023 at 09:19:28AM +0300, Laurent Pinchart wrote:
> > On Mon, Apr 17, 2023 at 08:02:03AM +0200, Uwe Kleine-König wrote:
> > > On Sun, Mar 26, 2023 at 04:30:25PM +0200, Uwe Kleine-König wrote:
> > > > Hello,
> > > >
> > > > this series adapts the platform drivers below drivers/pci to use
> > > > the
> > >
> > > copy&paste failure here: s/pci/media/ of course.
> > >
> > > > .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.
> > > >
> > > > Only three drivers needed some preparation first to make sure they
> > > > return 0 unconditionally in their remove callback. Then all
> > > > drivers could be trivially converted without side effects to
> .remove_new().
> > > >
> > > > The changes to the individual drivers are all orthogonal. If I
> > > > need to resend some patches because of some review feedback, I'd
> > > > like to only send the patches that actually needed changes, so
> > > > please pick up the remaining patches that don't need changing to
> reduce the amount of mail.
> > >
> > > I didn't hear anything back about application of this series. Is
> > > there a blocker somewhere?
> >
> > I think the series got applied to the master branch of
> > git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git.
> > It should thus appear in v6.4.
> 
> I guess that linux-stable.git is a copy&paste failure (and it's not there).
> I don't see the series in the master branch of
> git://linuxtv.org/media_tree.git either.
> 
> .. a bit later ...
> 
> ah, it's in git://linuxtv.org/mchehab/media-next.git
> 
> I guess I was just to quick and probably the series will be included in
> today's next.

I believe patchwork <patchwork at linuxtv.org> will send notification to
author and along with people who applied tags for that patch.

I normally get notification from patchwork <patchwork at linuxtv.org>
When the state of patch changes.

Cheers,
Biju