diff mbox series

[04/19] platform: chrome: drop owner assignment

Message ID 20240327-b4-module-owner-acpi-v1-4-725241a2d224@linaro.org (mailing list archive)
State Superseded
Headers show
Series ACPI: store owner from modules with acpi_bus_register_driver() | expand

Commit Message

Krzysztof Kozlowski March 27, 2024, 7:43 a.m. UTC 
ACPI bus core already sets the .owner, so driver does not need to.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

Depends on the first patch.
---
 drivers/platform/chrome/wilco_ec/event.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Tzung-Bi Shih March 28, 2024, 10:20 a.m. UTC | #1 
On Wed, Mar 27, 2024 at 08:43:51AM +0100, Krzysztof Kozlowski wrote:
> ACPI bus core already sets the .owner, so driver does not need to.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Tzung-Bi Shih <tzungbi@kernel.org>

I would prefer to use "platform/chrome: wilco_ec: " prefix though.
diff mbox series

Patch

diff --git a/drivers/platform/chrome/wilco_ec/event.c b/drivers/platform/chrome/wilco_ec/event.c
index 13291fb4214e..ae34e81c5d18 100644
--- a/drivers/platform/chrome/wilco_ec/event.c
+++ b/drivers/platform/chrome/wilco_ec/event.c
@@ -523,7 +523,6 @@  static struct acpi_driver event_driver = {
 		.notify = event_device_notify,
 		.remove = event_device_remove,
 	},
-	.owner = THIS_MODULE,
 };
 
 static int __init event_module_init(void)