2024-03-27 17:53:30

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] video: backlight: otm3225a: drop driver owner assignment

Core in spi_register_driver() already sets the .owner, so driver
does not need to.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/video/backlight/otm3225a.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/video/backlight/otm3225a.c b/drivers/video/backlight/otm3225a.c
index 2472e2167aae..4568020abf3c 100644
--- a/drivers/video/backlight/otm3225a.c
+++ b/drivers/video/backlight/otm3225a.c
@@ -239,7 +239,6 @@ static int otm3225a_probe(struct spi_device *spi)
static struct spi_driver otm3225a_driver = {
.driver = {
.name = "otm3225a",
- .owner = THIS_MODULE,
},
.probe = otm3225a_probe,
};
--
2.34.1



2024-04-02 11:07:35

by Daniel Thompson

[permalink] [raw]
Subject: Re: [PATCH] video: backlight: otm3225a: drop driver owner assignment

On Wed, Mar 27, 2024 at 06:47:14PM +0100, Krzysztof Kozlowski wrote:
> Core in spi_register_driver() already sets the .owner, so driver
> does not need to.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Daniel Thompson <[email protected]>


Daniel.

2024-04-11 11:40:24

by Lee Jones

[permalink] [raw]
Subject: Re: (subset) [PATCH] video: backlight: otm3225a: drop driver owner assignment

On Wed, 27 Mar 2024 18:47:14 +0100, Krzysztof Kozlowski wrote:
> Core in spi_register_driver() already sets the .owner, so driver
> does not need to.
>
>

Applied, thanks!

[1/1] video: backlight: otm3225a: drop driver owner assignment
commit: 9db7677e33b646a7fd60d35cbd4ab99886057a85

--
Lee Jones [李琼斯]