lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 27 Sep 2023 10:10:13 +0200
From:   Uwe Kleine-König 
        <u.kleine-koenig@...gutronix.de>
To:     Benson Leung <bleung@...omium.org>,
        Tzung-Bi Shih <tzungbi@...nel.org>,
        Prashant Malani <pmalani@...omium.org>,
        Guenter Roeck <groeck@...omium.org>,
        Brian Norris <briannorris@...omium.org>,
        Zhengkang Huang <zkhuang@...t.edu.cn>,
        Dongliang Mu <dzm91@...t.edu.cn>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Hans de Goede <hdegoede@...hat.com>,
        Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
        Mark Gross <markgross@...nel.org>,
        Vadim Pasternak <vadimp@...dia.com>,
        Gerd Haeussler <gerd.haeussler.ext@...mens.com>,
        Xing Tong Wu <xingtong.wu@...mens.com>,
        Tobias Schaffner <tobias.schaffner@...mens.com>
Cc:     chrome-platform@...ts.linux.dev, linux-kernel@...r.kernel.org,
        platform-driver-x86@...r.kernel.org, kernel@...gutronix.de
Subject: [PATCH 00/27] platform: Convert to platform remove callback returning void

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

 drivers/platform/chrome/cros_ec_chardev.c            |  6 ++----
 drivers/platform/chrome/cros_ec_debugfs.c            |  6 ++----
 drivers/platform/chrome/cros_ec_lightbar.c           |  6 ++----
 drivers/platform/chrome/cros_ec_lpc.c                |  6 ++----
 drivers/platform/chrome/cros_ec_sysfs.c              |  6 ++----
 drivers/platform/chrome/cros_ec_vbc.c                |  6 ++----
 drivers/platform/chrome/cros_typec_switch.c          |  5 ++---
 drivers/platform/chrome/cros_usbpd_logger.c          |  6 ++----
 drivers/platform/chrome/cros_usbpd_notify.c          | 12 ++++--------
 drivers/platform/chrome/wilco_ec/core.c              |  5 ++---
 drivers/platform/chrome/wilco_ec/debugfs.c           |  6 ++----
 drivers/platform/chrome/wilco_ec/telemetry.c         |  6 ++----
 drivers/platform/goldfish/goldfish_pipe.c            |  5 ++---
 drivers/platform/mellanox/mlxbf-bootctl.c            |  6 ++----
 drivers/platform/mellanox/mlxbf-tmfifo.c             |  6 ++----
 drivers/platform/mellanox/mlxreg-hotplug.c           |  6 ++----
 drivers/platform/mellanox/mlxreg-io.c                |  6 ++----
 drivers/platform/mellanox/mlxreg-lc.c                |  8 +++-----
 drivers/platform/mellanox/nvsw-sn2201.c              |  6 ++----
 drivers/platform/x86/intel/bytcrc_pwrsrc.c           |  5 ++---
 drivers/platform/x86/mlx-platform.c                  |  5 ++---
 drivers/platform/x86/sel3350-platform.c              |  6 ++----
 .../x86/siemens/simatic-ipc-batt-apollolake.c        |  6 +++---
 .../x86/siemens/simatic-ipc-batt-elkhartlake.c       |  6 +++---
 .../platform/x86/siemens/simatic-ipc-batt-f7188x.c   |  6 +++---
 drivers/platform/x86/siemens/simatic-ipc-batt.c      |  9 ++++-----
 drivers/platform/x86/siemens/simatic-ipc-batt.h      |  4 ++--
 27 files changed, 62 insertions(+), 104 deletions(-)

base-commit: 18030226a48de1fbfabf4ae16aaa2695a484254f
-- 
2.40.1

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ