Re: [PATCH 00/27] platform: Convert to platform remove callback returning void

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 4152 bytes --]

On Wed, 27 Sep 2023, Uwe Kleine-König wrote:

> Hello,
> 
> this series converts all platform drivers below drivers/platform 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.
> 
> See commit 5c5a7680e67b ("platform: Provide a remove callback that
> returns no value") for an extended explanation and the eventual goal.
> 
> As there is no single maintainer team for drivers/platform, I suggest
> the individual maintainers to pick up "their" patches. The only
> inter-dependency in this series is that the patches #24 - #27 depend on
> #23. Otherwise there are no inter-dependencies, so that should work
> fine. As there are still quite a few drivers to convert in other areas
> than drivers/platform, I'm happy about every patch that makes it in and
> there is no need for further coordination. So even if there is a merge
> conflict with one patch until you apply or a subject prefix is
> suboptimal, please don't let you stop by negative feedback for other
> patches (unless it applies to "your" patches, too, of course).
> 
> Best regards and thanks for considering,
> Uwe
> 
> Uwe Kleine-König (27):
>   platform/chrome: cros_ec_chardev: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_ec_debugfs: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_ec_lightbar: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_ec_lpc: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_ec_sysfs: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_ec_vbc: Convert to platform remove callback
>     returning void
>   platform/chrome: cros_typec_switch: Convert to platform remove
>     callback returning void
>   platform/chrome: cros_usbpd_logger: Convert to platform remove
>     callback returning void
>   platform/chrome: cros_usbpd_notify: Convert to platform remove
>     callback returning void
>   platform/chrome/wilco_ec: core: Convert to platform remove callback
>     returning void
>   platform/chrome/wilco_ec: debugfs: Convert to platform remove callback
>     returning void
>   platform/chrome/wilco_ec: telemetry: Convert to platform remove
>     callback returning void
>   platform/goldfish: goldfish_pipe: Convert to platform remove callback
>     returning void
>   platform/mellanox: mlxbf-bootctl: Convert to platform remove callback
>     returning void
>   platform/mellanox: mlxbf-tmfifo: Convert to platform remove callback
>     returning void
>   platform/mellanox: mlxreg-hotplug: Convert to platform remove callback
>     returning void
>   platform/mellanox: mlxreg-io: Convert to platform remove callback
>     returning void
>   platform/mellanox: mlxreg-lc: Convert to platform remove callback
>     returning void
>   platform/mellanox: nvsw-sn2201: Convert to platform remove callback
>     returning void
>   platform/x86/intel: bytcrc_pwrsrc: Convert to platform remove callback
>     returning void
>   platform/x86: mlx-platform: Convert to platform remove callback
>     returning void
>   platform/x86: sel3350-platform: Convert to platform remove callback
>     returning void
>   platform/x86/siemens: simatic-ipc-batt: Simplify
>     simatic_ipc_batt_remove()
>   platform/x86/siemens: simatic-ipc-batt: Convert to platform remove
>     callback returning void
>   platform/x86/siemens: simatic-ipc-batt-apollolake: Convert to platform
>     remove callback returning void
>   platform/x86/siemens: simatic-ipc-batt-elkhartlake: Convert to
>     platform remove callback returning void
>   platform/x86/siemens: simatic-ipc-batt-f7188x: Convert to platform
>     remove callback returning void

Thanks for the patches.

I don't know if you got the auto generated thank you email as it seemed to 
hit a character encoding problem while sending the email out.

But in any case, I've applied patches 14-27 to my local review-ilpo branch 
(they will eventually appear on the public repo too and propagate to 
for-next).


-- 
 i.