From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1DE47CE7A88 for ; Sat, 23 Sep 2023 17:35:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231363AbjIWRgC (ORCPT ); Sat, 23 Sep 2023 13:36:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59584 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229513AbjIWRgC (ORCPT ); Sat, 23 Sep 2023 13:36:02 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 98135AF; Sat, 23 Sep 2023 10:35:55 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D6892C433C8; Sat, 23 Sep 2023 17:35:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695490555; bh=ejyzahKJAJt9lCn+qnlEr4NX27qh7eXBTFD4U9A1SjU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=QeNSFcPUCH4xy1kS395jk0Cox1UddxCRcd2x2EjvDTmX1KYbrQuhpNkUBHZuAdGiq kV/zqk9FM6KQdGbKVVGecf20HtNQA4lrjEzI1IHuo241x/0RYTT5fXqOUMeOOazA9y F0FVmVyOTX6AAw2bUbxgcpC84d3pJKkcepeoFTgGlCP78lQyX+SvNtZtau22rCgC4/ bwWPsiuKyaqhArk9vylBqt5J9Btdd1N/7augiDEwhu9H8ZC1G8uqJ49n70kE6lDL7s NcrYZc9Sw96xq7YYgbmFpI2BGcHBxNfFb6xFbzBBBhqVtPjvDzhKJcKO/ffOBuo3f5 9fuEbW1t3hSNw== Date: Sat, 23 Sep 2023 18:35:24 +0100 From: Jonathan Cameron To: Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= Cc: Jiri Kosina , Srinivas Pandruvada , Lars-Peter Clausen , linux-input@vger.kernel.org, linux-iio@vger.kernel.org, kernel@pengutronix.de, Linus Walleij , linux-arm-kernel@lists.infradead.org, Eugen Hristev , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Jinjie Ruan , Rob Herring , Heiko Stuebner , Yang Yingliang , Chen-Yu Tsai , Aidan MacDonald , Andy Shevchenko , Ray Jui , Scott Branden , Broadcom internal kernel review list , Hartley Sweeten , Alexander Sverdlin , Krzysztof Kozlowski , Alim Akhtar , linux-samsung-soc@vger.kernel.org, Shawn Guo , Sascha Hauer , Fabio Estevam , NXP Linux Team , Andreas Klinger , Cai Huoqing , Haibo Chen , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , George Stark , Andy Shevchenko , Nuno =?UTF-8?B?U8Oh?= , linux-amlogic@lists.infradead.org, Saravanan Sekar , Jiakai Luo , Dongliang Mu , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , openbmc@lists.ozlabs.org, Andy Gross , Bjorn Andersson , Konrad Dybcio , linux-arm-msm@vger.kernel.org, Marek Vasut , Maxime Coquelin , Alexandre Torgue , Olivier Moysan , Fabrice Gasnier , Zhang Shurong , Yangtao Li , linux-stm32@st-md-mailman.stormreply.com, Sean Nyekjaer , Tom Rix , Jernej Skrabec , Samuel Holland , "Rafael J. Wysocki" , Damien Le Moal , Mark Brown , Ido Schimmel , Daniel Lezcano , linux-sunxi@lists.linux.dev, Dmitry Torokhov , Andreas Kemnade , Peter Rosin , Vladimir Zapolskiy , Kevin Tsai , Benson Leung , Guenter Roeck , chrome-platform@lists.linux.dev Subject: Re: [PATCH 00/49] iio: Convert to platform remove callback returning void Message-ID: <20230923183513.717f3014@jic23-huawei> In-Reply-To: <20230919174931.1417681-1-u.kleine-koenig@pengutronix.de> References: <20230919174931.1417681-1-u.kleine-koenig@pengutronix.de> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org On Tue, 19 Sep 2023 19:48:42 +0200 Uwe Kleine-K=C3=B6nig wrote: > this series converts all platform drivers below drivers/iio to use > .remove_new(). The motivation is to get rid of an integer return code > that is (mostly) ignored by the platform driver core and error prone on > the driver side. As all platform drivers return zero unconditionally in t= heir > remove callback up to now, the conversions are "trivial". >=20 > See commit 5c5a7680e67b ("platform: Provide a remove callback that > returns no value") for an extended explanation and the eventual goal. >=20 > There are no interdependencies between the patches. As there are still > quite a few drivers to convert, I'm happy about every patch that makes > it in. So even if there is a merge conflict with one patch until you > apply or I picked a wrong subject prefix, please apply the remainder of > this series anyhow. >=20 Series applied to the togreg branch of iio.git and pushed out as testing=20 to see if 0-day finds anything we are missing. I've picked up all tags give as of early today. Thanks, Jonathan > Best regards > Uwe >=20 > Uwe Kleine-K=C3=B6nig (49): > iio: accel: hid-sensor-accel-3d: Convert to platform remove callback > returning void > iio: adc: ab8500-gpadc: Convert to platform remove callback returning > void > iio: adc: at91-sama5d2: Convert to platform remove callback returning > void > iio: adc: at91: Convert to platform remove callback returning void > iio: adc: axp20x: Convert to platform remove callback returning void > iio: adc: bcm_iproc: Convert to platform remove callback returning > void > iio: adc: dln2: Convert to platform remove callback returning void > iio: adc: ep93xx: Convert to platform remove callback returning void > iio: adc: exynos: Convert to platform remove callback returning void > iio: adc: fsl-imx25-gcq: Convert to platform remove callback returning > void > iio: adc: hx711: Convert to platform remove callback returning void > iio: adc: imx8qxp: Convert to platform remove callback returning void > iio: adc: imx93: Convert to platform remove callback returning void > iio: adc: meson_saradc: Convert to platform remove callback returning > void > iio: adc: mp2629: Convert to platform remove callback returning void > iio: adc: mxs-lradc: Convert to platform remove callback returning > void > iio: adc: npcm: Convert to platform remove callback returning void > iio: adc: qcom-pm8xxx-xoadc: Convert to platform remove callback > returning void > iio: adc: rcar-gyroadc: Convert to platform remove callback returning > void > iio: adc: stm32-adc-core: Convert to platform remove callback > returning void > iio: adc: stm32-adc: Convert to platform remove callback returning > void > iio: adc: stm32-dfsdm-adc: Convert to platform remove callback > returning void > iio: adc: stm32-dfsdm-core: Convert to platform remove callback > returning void > iio: adc: sun4i-gpadc-iio: Convert to platform remove callback > returning void > iio: adc: ti_am335x_adc: Convert to platform remove callback returning > void > iio: adc: twl4030-madc: Convert to platform remove callback returning > void > iio: adc: twl6030-gpadc: Convert to platform remove callback returning > void > iio: adc: vf610_adc: Convert to platform remove callback returning > void > iio: dac: dpot-dac: Convert to platform remove callback returning void > iio: dac: lpc18xx_dac: Convert to platform remove callback returning > void > iio: dac: stm32-dac-core: Convert to platform remove callback > returning void > iio: dac: stm32-dac: Convert to platform remove callback returning > void > iio: dac: vf610: Convert to platform remove callback returning void > iio: gyro: hid-sensor-gyro-3d: Convert to platform remove callback > returning void > iio: humidity: hid-sensor-humidity: Convert to platform remove > callback returning void > iio: light: cm3605: Convert to platform remove callback returning void > iio: light: hid-sensor-als: Convert to platform remove callback > returning void > iio: light: hid-sensor-prox: Convert to platform remove callback > returning void > iio: light: lm3533-als: Convert to platform remove callback returning > void > iio: magnetometer: hid-sensor-magn-3d: Convert to platform remove > callback returning void > iio: orientation: hid-sensor-incl-3d: Convert to platform remove > callback returning void > iio: orientation: hid-sensor-rotation: Convert to platform remove > callback returning void > iio: position: hid-sensor-custom-intel-hinge: Convert to platform > remove callback returning void > iio: pressure: hid-sensor: Convert to platform remove callback > returning void > iio: proximity: cros_ec_mkbp: Convert to platform remove callback > returning void > iio: proximity: srf04: Convert to platform remove callback returning > void > iio: temperature: hid-sensor: Convert to platform remove callback > returning void > iio: trigger: iio-trig-interrupt: Convert to platform remove callback > returning void > iio: trigger: stm32-timer: Convert to platform remove callback > returning void >=20 > drivers/iio/accel/hid-sensor-accel-3d.c | 6 ++---- > drivers/iio/adc/ab8500-gpadc.c | 6 ++---- > drivers/iio/adc/at91-sama5d2_adc.c | 6 ++---- > drivers/iio/adc/at91_adc.c | 6 ++---- > drivers/iio/adc/axp20x_adc.c | 6 ++---- > drivers/iio/adc/bcm_iproc_adc.c | 6 ++---- > drivers/iio/adc/dln2-adc.c | 5 ++--- > drivers/iio/adc/ep93xx_adc.c | 6 ++---- > drivers/iio/adc/exynos_adc.c | 6 ++---- > drivers/iio/adc/fsl-imx25-gcq.c | 6 ++---- > drivers/iio/adc/hx711.c | 6 ++---- > drivers/iio/adc/imx8qxp-adc.c | 6 ++---- > drivers/iio/adc/imx93_adc.c | 6 ++---- > drivers/iio/adc/meson_saradc.c | 6 ++---- > drivers/iio/adc/mp2629_adc.c | 6 ++---- > drivers/iio/adc/mxs-lradc-adc.c | 6 ++---- > drivers/iio/adc/npcm_adc.c | 6 ++---- > drivers/iio/adc/qcom-pm8xxx-xoadc.c | 6 ++---- > drivers/iio/adc/rcar-gyroadc.c | 6 ++---- > drivers/iio/adc/stm32-adc-core.c | 6 ++---- > drivers/iio/adc/stm32-adc.c | 6 ++---- > drivers/iio/adc/stm32-dfsdm-adc.c | 6 ++---- > drivers/iio/adc/stm32-dfsdm-core.c | 6 ++---- > drivers/iio/adc/sun4i-gpadc-iio.c | 8 +++----- > drivers/iio/adc/ti_am335x_adc.c | 6 ++---- > drivers/iio/adc/twl4030-madc.c | 6 ++---- > drivers/iio/adc/twl6030-gpadc.c | 6 ++---- > drivers/iio/adc/vf610_adc.c | 6 ++---- > drivers/iio/dac/dpot-dac.c | 6 ++---- > drivers/iio/dac/lpc18xx_dac.c | 6 ++---- > drivers/iio/dac/stm32-dac-core.c | 6 ++---- > drivers/iio/dac/stm32-dac.c | 6 ++---- > drivers/iio/dac/vf610_dac.c | 6 ++---- > drivers/iio/gyro/hid-sensor-gyro-3d.c | 6 ++---- > drivers/iio/humidity/hid-sensor-humidity.c | 6 ++---- > drivers/iio/light/cm3605.c | 6 ++---- > drivers/iio/light/hid-sensor-als.c | 6 ++---- > drivers/iio/light/hid-sensor-prox.c | 6 ++---- > drivers/iio/light/lm3533-als.c | 6 ++---- > drivers/iio/magnetometer/hid-sensor-magn-3d.c | 6 ++---- > drivers/iio/orientation/hid-sensor-incl-3d.c | 6 ++---- > drivers/iio/orientation/hid-sensor-rotation.c | 6 ++---- > drivers/iio/position/hid-sensor-custom-intel-hinge.c | 6 ++---- > drivers/iio/pressure/hid-sensor-press.c | 6 ++---- > drivers/iio/proximity/cros_ec_mkbp_proximity.c | 6 ++---- > drivers/iio/proximity/srf04.c | 6 ++---- > drivers/iio/temperature/hid-sensor-temperature.c | 6 ++---- > drivers/iio/trigger/iio-trig-interrupt.c | 6 ++---- > drivers/iio/trigger/stm32-timer-trigger.c | 6 ++---- > 49 files changed, 99 insertions(+), 196 deletions(-) >=20 >=20 > base-commit: 29e400e3ea486bf942b214769fc9778098114113