* [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header
@ 2025-03-31 7:25 Andy Shevchenko
2025-04-03 8:10 ` Tomi Valkeinen
2025-05-02 14:12 ` Andy Shevchenko
0 siblings, 2 replies; 3+ messages in thread
From: Andy Shevchenko @ 2025-03-31 7:25 UTC (permalink / raw)
To: Tomi Valkeinen, linux-media, linux-kernel
Cc: Mauro Carvalho Chehab, Andy Shevchenko
The fwnode.h is not supposed to be used by the drivers as it
has the definitions for the core parts for different device
property provider implementations. Drop it.
Note, that fwnode API for drivers is provided in property.h
which is included here.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
drivers/media/i2c/ds90ub913.c | 1 -
drivers/media/i2c/ds90ub953.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/media/i2c/ds90ub913.c b/drivers/media/i2c/ds90ub913.c
index 834df8d508e6..352b3d26f376 100644
--- a/drivers/media/i2c/ds90ub913.c
+++ b/drivers/media/i2c/ds90ub913.c
@@ -12,7 +12,6 @@
#include <linux/clk-provider.h>
#include <linux/clk.h>
#include <linux/delay.h>
-#include <linux/fwnode.h>
#include <linux/gpio/driver.h>
#include <linux/i2c-atr.h>
#include <linux/i2c.h>
diff --git a/drivers/media/i2c/ds90ub953.c b/drivers/media/i2c/ds90ub953.c
index 1cd5ac7157e4..dc50517fd228 100644
--- a/drivers/media/i2c/ds90ub953.c
+++ b/drivers/media/i2c/ds90ub953.c
@@ -11,7 +11,6 @@
#include <linux/clk-provider.h>
#include <linux/clk.h>
#include <linux/delay.h>
-#include <linux/fwnode.h>
#include <linux/gpio/driver.h>
#include <linux/i2c-atr.h>
#include <linux/i2c.h>
--
2.47.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header
2025-03-31 7:25 [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header Andy Shevchenko
@ 2025-04-03 8:10 ` Tomi Valkeinen
2025-05-02 14:12 ` Andy Shevchenko
1 sibling, 0 replies; 3+ messages in thread
From: Tomi Valkeinen @ 2025-04-03 8:10 UTC (permalink / raw)
To: Andy Shevchenko, linux-media, linux-kernel; +Cc: Mauro Carvalho Chehab
Hi,
On 31/03/2025 10:25, Andy Shevchenko wrote:
> The fwnode.h is not supposed to be used by the drivers as it
> has the definitions for the core parts for different device
> property provider implementations. Drop it.
>
> Note, that fwnode API for drivers is provided in property.h
> which is included here.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> drivers/media/i2c/ds90ub913.c | 1 -
> drivers/media/i2c/ds90ub953.c | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/drivers/media/i2c/ds90ub913.c b/drivers/media/i2c/ds90ub913.c
> index 834df8d508e6..352b3d26f376 100644
> --- a/drivers/media/i2c/ds90ub913.c
> +++ b/drivers/media/i2c/ds90ub913.c
> @@ -12,7 +12,6 @@
> #include <linux/clk-provider.h>
> #include <linux/clk.h>
> #include <linux/delay.h>
> -#include <linux/fwnode.h>
> #include <linux/gpio/driver.h>
> #include <linux/i2c-atr.h>
> #include <linux/i2c.h>
> diff --git a/drivers/media/i2c/ds90ub953.c b/drivers/media/i2c/ds90ub953.c
> index 1cd5ac7157e4..dc50517fd228 100644
> --- a/drivers/media/i2c/ds90ub953.c
> +++ b/drivers/media/i2c/ds90ub953.c
> @@ -11,7 +11,6 @@
> #include <linux/clk-provider.h>
> #include <linux/clk.h>
> #include <linux/delay.h>
> -#include <linux/fwnode.h>
> #include <linux/gpio/driver.h>
> #include <linux/i2c-atr.h>
> #include <linux/i2c.h>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Tomi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header
2025-03-31 7:25 [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header Andy Shevchenko
2025-04-03 8:10 ` Tomi Valkeinen
@ 2025-05-02 14:12 ` Andy Shevchenko
1 sibling, 0 replies; 3+ messages in thread
From: Andy Shevchenko @ 2025-05-02 14:12 UTC (permalink / raw)
To: Tomi Valkeinen, linux-media, linux-kernel; +Cc: Mauro Carvalho Chehab
On Mon, Mar 31, 2025 at 10:25:07AM +0300, Andy Shevchenko wrote:
> The fwnode.h is not supposed to be used by the drivers as it
> has the definitions for the core parts for different device
> property provider implementations. Drop it.
>
> Note, that fwnode API for drivers is provided in property.h
> which is included here.
Any comments? Can it be applied?
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-05-02 14:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-31 7:25 [PATCH v1 1/1] media: i2c: ds90ub9x3: Remove (explicitly) unused header Andy Shevchenko
2025-04-03 8:10 ` Tomi Valkeinen
2025-05-02 14:12 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox