mbox series

[v2,00/19] ACPI: store owner from modules with acpi_bus_register_driver()

Message ID 20240328-b4-module-owner-acpi-v2-0-1e5552c2c69f@linaro.org (mailing list archive)
Headers show
Series ACPI: store owner from modules with acpi_bus_register_driver() | expand

Message

Krzysztof Kozlowski March 28, 2024, 7:49 p.m. UTC
Changes in v2:
- Correct input and platform/chrome subjects.
- Add acks.
- Link to v1: https://lore.kernel.org/r/20240327-b4-module-owner-acpi-v1-0-725241a2d224@linaro.org

Merging
=======
All further patches depend on the first amba patch, therefore one way is
to ack and take it via one tree, e.g. ACPI.

Description
===========
Modules registering driver with acpi_bus_register_driver() often forget to
set .owner field.

Solve the problem by moving this task away from the drivers to the core
amba bus code, just like we did for platform_driver in commit
9447057eaff8 ("platform_device: use a macro instead of
platform_driver_register").

Best regards,
Krzysztof

---
Krzysztof Kozlowski (19):
      ACPI: store owner from modules with acpi_bus_register_driver()
      Input: atlas - drop owner assignment
      net: fjes: drop owner assignment
      platform/chrome: wilco_ec: drop owner assignment
      platform: asus-laptop: drop owner assignment
      platform: classmate-laptop: drop owner assignment
      platform/x86/dell: drop owner assignment
      platform/x86/eeepc: drop owner assignment
      platform/x86/intel/rst: drop owner assignment
      platform/x86/intel/smartconnect: drop owner assignment
      platform/x86/lg-laptop: drop owner assignment
      platform/x86/sony-laptop: drop owner assignment
      platform/x86/toshiba_acpi: drop owner assignment
      platform/x86/toshiba_bluetooth: drop owner assignment
      platform/x86/toshiba_haps: drop owner assignment
      platform/x86/wireless-hotkey: drop owner assignment
      ptp: vmw: drop owner assignment
      virt: vmgenid: drop owner assignment
      ACPI: drop redundant owner from acpi_driver

 drivers/acpi/bus.c                        | 9 +++++----
 drivers/input/misc/atlas_btns.c           | 1 -
 drivers/net/fjes/fjes_main.c              | 1 -
 drivers/platform/chrome/wilco_ec/event.c  | 1 -
 drivers/platform/x86/asus-laptop.c        | 1 -
 drivers/platform/x86/classmate-laptop.c   | 5 -----
 drivers/platform/x86/dell/dell-rbtn.c     | 1 -
 drivers/platform/x86/eeepc-laptop.c       | 1 -
 drivers/platform/x86/intel/rst.c          | 1 -
 drivers/platform/x86/intel/smartconnect.c | 1 -
 drivers/platform/x86/lg-laptop.c          | 1 -
 drivers/platform/x86/sony-laptop.c        | 2 --
 drivers/platform/x86/toshiba_acpi.c       | 1 -
 drivers/platform/x86/toshiba_bluetooth.c  | 1 -
 drivers/platform/x86/toshiba_haps.c       | 1 -
 drivers/platform/x86/wireless-hotkey.c    | 1 -
 drivers/ptp/ptp_vmw.c                     | 1 -
 drivers/virt/vmgenid.c                    | 1 -
 include/acpi/acpi_bus.h                   | 8 ++++++--
 19 files changed, 11 insertions(+), 28 deletions(-)
---
base-commit: 02e345c7445b96358707ee2e5ca1dfc9b3458d39
change-id: 20240327-b4-module-owner-acpi-d4948a922351

Best regards,

Comments

Rafael J. Wysocki April 8, 2024, 2:18 p.m. UTC | #1
On Thu, Mar 28, 2024 at 8:49 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> Changes in v2:
> - Correct input and platform/chrome subjects.
> - Add acks.
> - Link to v1: https://lore.kernel.org/r/20240327-b4-module-owner-acpi-v1-0-725241a2d224@linaro.org
>
> Merging
> =======
> All further patches depend on the first amba patch, therefore one way is
> to ack and take it via one tree, e.g. ACPI.
>
> Description
> ===========
> Modules registering driver with acpi_bus_register_driver() often forget to
> set .owner field.
>
> Solve the problem by moving this task away from the drivers to the core
> amba bus code, just like we did for platform_driver in commit
> 9447057eaff8 ("platform_device: use a macro instead of
> platform_driver_register").
>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (19):
>       ACPI: store owner from modules with acpi_bus_register_driver()
>       Input: atlas - drop owner assignment
>       net: fjes: drop owner assignment
>       platform/chrome: wilco_ec: drop owner assignment
>       platform: asus-laptop: drop owner assignment
>       platform: classmate-laptop: drop owner assignment
>       platform/x86/dell: drop owner assignment
>       platform/x86/eeepc: drop owner assignment
>       platform/x86/intel/rst: drop owner assignment
>       platform/x86/intel/smartconnect: drop owner assignment
>       platform/x86/lg-laptop: drop owner assignment
>       platform/x86/sony-laptop: drop owner assignment
>       platform/x86/toshiba_acpi: drop owner assignment
>       platform/x86/toshiba_bluetooth: drop owner assignment
>       platform/x86/toshiba_haps: drop owner assignment
>       platform/x86/wireless-hotkey: drop owner assignment
>       ptp: vmw: drop owner assignment
>       virt: vmgenid: drop owner assignment
>       ACPI: drop redundant owner from acpi_driver
>
>  drivers/acpi/bus.c                        | 9 +++++----
>  drivers/input/misc/atlas_btns.c           | 1 -
>  drivers/net/fjes/fjes_main.c              | 1 -
>  drivers/platform/chrome/wilco_ec/event.c  | 1 -
>  drivers/platform/x86/asus-laptop.c        | 1 -
>  drivers/platform/x86/classmate-laptop.c   | 5 -----
>  drivers/platform/x86/dell/dell-rbtn.c     | 1 -
>  drivers/platform/x86/eeepc-laptop.c       | 1 -
>  drivers/platform/x86/intel/rst.c          | 1 -
>  drivers/platform/x86/intel/smartconnect.c | 1 -
>  drivers/platform/x86/lg-laptop.c          | 1 -
>  drivers/platform/x86/sony-laptop.c        | 2 --
>  drivers/platform/x86/toshiba_acpi.c       | 1 -
>  drivers/platform/x86/toshiba_bluetooth.c  | 1 -
>  drivers/platform/x86/toshiba_haps.c       | 1 -
>  drivers/platform/x86/wireless-hotkey.c    | 1 -
>  drivers/ptp/ptp_vmw.c                     | 1 -
>  drivers/virt/vmgenid.c                    | 1 -
>  include/acpi/acpi_bus.h                   | 8 ++++++--
>  19 files changed, 11 insertions(+), 28 deletions(-)
> ---

Whole series applied as 6.10 material, thanks!
patchwork-bot+chrome-platform@kernel.org May 27, 2024, 2:55 a.m. UTC | #2
Hello:

This series was applied to chrome-platform/linux.git (for-kernelci)
by Rafael J. Wysocki <rafael.j.wysocki@intel.com>:

On Thu, 28 Mar 2024 20:49:10 +0100 you wrote:
> Changes in v2:
> - Correct input and platform/chrome subjects.
> - Add acks.
> - Link to v1: https://lore.kernel.org/r/20240327-b4-module-owner-acpi-v1-0-725241a2d224@linaro.org
> 
> Merging
> =======
> All further patches depend on the first amba patch, therefore one way is
> to ack and take it via one tree, e.g. ACPI.
> 
> [...]

Here is the summary with links:
  - [v2,01/19] ACPI: store owner from modules with acpi_bus_register_driver()
    https://git.kernel.org/chrome-platform/c/48b9c4862bd3
  - [v2,02/19] Input: atlas - drop owner assignment
    https://git.kernel.org/chrome-platform/c/726c149e0798
  - [v2,03/19] net: fjes: drop owner assignment
    https://git.kernel.org/chrome-platform/c/3bdef399d00e
  - [v2,04/19] platform/chrome: wilco_ec: drop owner assignment
    https://git.kernel.org/chrome-platform/c/245d97ff3473
  - [v2,05/19] platform: asus-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/eda8304c74f0
  - [v2,06/19] platform: classmate-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/be24e9a09337
  - [v2,07/19] platform/x86/dell: drop owner assignment
    https://git.kernel.org/chrome-platform/c/1baad72e9026
  - [v2,08/19] platform/x86/eeepc: drop owner assignment
    https://git.kernel.org/chrome-platform/c/4313188f8128
  - [v2,09/19] platform/x86/intel/rst: drop owner assignment
    https://git.kernel.org/chrome-platform/c/68370cc2e32a
  - [v2,10/19] platform/x86/intel/smartconnect: drop owner assignment
    https://git.kernel.org/chrome-platform/c/e84a761f1215
  - [v2,11/19] platform/x86/lg-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/2929a735d92e
  - [v2,12/19] platform/x86/sony-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/562231f34cea
  - [v2,13/19] platform/x86/toshiba_acpi: drop owner assignment
    https://git.kernel.org/chrome-platform/c/b655cda9f089
  - [v2,14/19] platform/x86/toshiba_bluetooth: drop owner assignment
    https://git.kernel.org/chrome-platform/c/ce69eeb2ccb7
  - [v2,15/19] platform/x86/toshiba_haps: drop owner assignment
    https://git.kernel.org/chrome-platform/c/eb22f3ba0c2e
  - [v2,16/19] platform/x86/wireless-hotkey: drop owner assignment
    https://git.kernel.org/chrome-platform/c/d49c09ddfbd5
  - [v2,17/19] ptp: vmw: drop owner assignment
    https://git.kernel.org/chrome-platform/c/cd3eda2e3508
  - [v2,18/19] virt: vmgenid: drop owner assignment
    https://git.kernel.org/chrome-platform/c/00e8b52bf9f9
  - [v2,19/19] ACPI: drop redundant owner from acpi_driver
    https://git.kernel.org/chrome-platform/c/cc85f9c05bba

You are awesome, thank you!
patchwork-bot+chrome-platform@kernel.org May 27, 2024, 3:07 a.m. UTC | #3
Hello:

This series was applied to chrome-platform/linux.git (for-next)
by Rafael J. Wysocki <rafael.j.wysocki@intel.com>:

On Thu, 28 Mar 2024 20:49:10 +0100 you wrote:
> Changes in v2:
> - Correct input and platform/chrome subjects.
> - Add acks.
> - Link to v1: https://lore.kernel.org/r/20240327-b4-module-owner-acpi-v1-0-725241a2d224@linaro.org
> 
> Merging
> =======
> All further patches depend on the first amba patch, therefore one way is
> to ack and take it via one tree, e.g. ACPI.
> 
> [...]

Here is the summary with links:
  - [v2,01/19] ACPI: store owner from modules with acpi_bus_register_driver()
    https://git.kernel.org/chrome-platform/c/48b9c4862bd3
  - [v2,02/19] Input: atlas - drop owner assignment
    https://git.kernel.org/chrome-platform/c/726c149e0798
  - [v2,03/19] net: fjes: drop owner assignment
    https://git.kernel.org/chrome-platform/c/3bdef399d00e
  - [v2,04/19] platform/chrome: wilco_ec: drop owner assignment
    https://git.kernel.org/chrome-platform/c/245d97ff3473
  - [v2,05/19] platform: asus-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/eda8304c74f0
  - [v2,06/19] platform: classmate-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/be24e9a09337
  - [v2,07/19] platform/x86/dell: drop owner assignment
    https://git.kernel.org/chrome-platform/c/1baad72e9026
  - [v2,08/19] platform/x86/eeepc: drop owner assignment
    https://git.kernel.org/chrome-platform/c/4313188f8128
  - [v2,09/19] platform/x86/intel/rst: drop owner assignment
    https://git.kernel.org/chrome-platform/c/68370cc2e32a
  - [v2,10/19] platform/x86/intel/smartconnect: drop owner assignment
    https://git.kernel.org/chrome-platform/c/e84a761f1215
  - [v2,11/19] platform/x86/lg-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/2929a735d92e
  - [v2,12/19] platform/x86/sony-laptop: drop owner assignment
    https://git.kernel.org/chrome-platform/c/562231f34cea
  - [v2,13/19] platform/x86/toshiba_acpi: drop owner assignment
    https://git.kernel.org/chrome-platform/c/b655cda9f089
  - [v2,14/19] platform/x86/toshiba_bluetooth: drop owner assignment
    https://git.kernel.org/chrome-platform/c/ce69eeb2ccb7
  - [v2,15/19] platform/x86/toshiba_haps: drop owner assignment
    https://git.kernel.org/chrome-platform/c/eb22f3ba0c2e
  - [v2,16/19] platform/x86/wireless-hotkey: drop owner assignment
    https://git.kernel.org/chrome-platform/c/d49c09ddfbd5
  - [v2,17/19] ptp: vmw: drop owner assignment
    https://git.kernel.org/chrome-platform/c/cd3eda2e3508
  - [v2,18/19] virt: vmgenid: drop owner assignment
    https://git.kernel.org/chrome-platform/c/00e8b52bf9f9
  - [v2,19/19] ACPI: drop redundant owner from acpi_driver
    https://git.kernel.org/chrome-platform/c/cc85f9c05bba

You are awesome, thank you!