From: "Rafael J. Wysocki" <rafael@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
"Len Brown" <lenb@kernel.org>,
"Robert Moore" <robert.moore@intel.com>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
"Eric Dumazet" <edumazet@google.com>,
"Jakub Kicinski" <kuba@kernel.org>,
"Paolo Abeni" <pabeni@redhat.com>,
"Benson Leung" <bleung@chromium.org>,
"Tzung-Bi Shih" <tzungbi@kernel.org>,
"Corentin Chary" <corentin.chary@gmail.com>,
"Luke D. Jones" <luke@ljones.dev>,
"Hans de Goede" <hdegoede@redhat.com>,
"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
"Thadeu Lima de Souza Cascardo" <cascardo@holoscopio.com>,
"Pali Rohár" <pali@kernel.org>,
"Matan Ziv-Av" <matan@svgalib.org>,
"Mattia Dongili" <malattia@linux.it>,
"Azael Avalos" <coproscefalo@gmail.com>,
"Ajay Kaher" <akaher@vmware.com>,
"Alexey Makhalov" <amakhalov@vmware.com>,
"VMware PV-Drivers Reviewers" <pv-drivers@vmware.com>,
"Richard Cochran" <richardcochran@gmail.com>,
"Theodore Ts'o" <tytso@mit.edu>,
"Jason A. Donenfeld" <Jason@zx2c4.com>,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
acpica-devel@lists.linux.dev, linux-input@vger.kernel.org,
netdev@vger.kernel.org, chrome-platform@lists.linux.dev,
platform-driver-x86@vger.kernel.org,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
Subject: Re: [PATCH v2 00/19] ACPI: store owner from modules with acpi_bus_register_driver()
Date: Mon, 8 Apr 2024 16:18:54 +0200 [thread overview]
Message-ID: <CAJZ5v0jd7FegkcnJyDn61uBEdGRKHWoaQ8SKJE+JX18W0QHh1Q@mail.gmail.com> (raw)
In-Reply-To: <20240328-b4-module-owner-acpi-v2-0-1e5552c2c69f@linaro.org>
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!
prev parent reply other threads:[~2024-04-08 14:19 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-28 19:49 [PATCH v2 00/19] ACPI: store owner from modules with acpi_bus_register_driver() Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 01/19] " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 02/19] Input: atlas - drop owner assignment Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 03/19] net: fjes: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 04/19] platform/chrome: wilco_ec: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 05/19] platform: asus-laptop: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 06/19] platform: classmate-laptop: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 07/19] platform/x86/dell: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 08/19] platform/x86/eeepc: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 09/19] platform/x86/intel/rst: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 10/19] platform/x86/intel/smartconnect: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 11/19] platform/x86/lg-laptop: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 12/19] platform/x86/sony-laptop: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 13/19] platform/x86/toshiba_acpi: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 14/19] platform/x86/toshiba_bluetooth: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 15/19] platform/x86/toshiba_haps: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 16/19] platform/x86/wireless-hotkey: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 17/19] ptp: vmw: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 18/19] virt: vmgenid: " Krzysztof Kozlowski
2024-03-28 19:49 ` [PATCH v2 19/19] ACPI: drop redundant owner from acpi_driver Krzysztof Kozlowski
2024-04-08 14:18 ` Rafael J. Wysocki [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAJZ5v0jd7FegkcnJyDn61uBEdGRKHWoaQ8SKJE+JX18W0QHh1Q@mail.gmail.com \
--to=rafael@kernel.org \
--cc=Jason@zx2c4.com \
--cc=acpica-devel@lists.linux.dev \
--cc=akaher@vmware.com \
--cc=amakhalov@vmware.com \
--cc=bleung@chromium.org \
--cc=cascardo@holoscopio.com \
--cc=chrome-platform@lists.linux.dev \
--cc=coproscefalo@gmail.com \
--cc=corentin.chary@gmail.com \
--cc=davem@davemloft.net \
--cc=dmitry.torokhov@gmail.com \
--cc=edumazet@google.com \
--cc=hdegoede@redhat.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=krzysztof.kozlowski@linaro.org \
--cc=kuba@kernel.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luke@ljones.dev \
--cc=malattia@linux.it \
--cc=matan@svgalib.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=pali@kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=pv-drivers@vmware.com \
--cc=rafael.j.wysocki@intel.com \
--cc=richardcochran@gmail.com \
--cc=robert.moore@intel.com \
--cc=tytso@mit.edu \
--cc=tzungbi@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).