2020-03-10 15:44:37

by Stephen Boyd

[permalink] [raw]
Subject: [PATCH v2 3/3] i2c: qcom-geni: Drop of_platform.h include

This driver doesn't call any DT platform functions like of_platform_*().
Remove the include as it isn't used.

Cc: Alok Chauhan <[email protected]>
Reviewed-by: Douglas Anderson <[email protected]>
Reviewed-by: Brendan Higgins <[email protected]>
Signed-off-by: Stephen Boyd <[email protected]>
---
drivers/i2c/busses/i2c-qcom-geni.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c
index 2f5fb2e83f95..18d1e4fd4cf3 100644
--- a/drivers/i2c/busses/i2c-qcom-geni.c
+++ b/drivers/i2c/busses/i2c-qcom-geni.c
@@ -10,7 +10,6 @@
#include <linux/io.h>
#include <linux/module.h>
#include <linux/of.h>
-#include <linux/of_platform.h>
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
#include <linux/qcom-geni-se.h>
--
Sent by a computer, using git, on the internet


2020-03-10 19:43:31

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH v2 3/3] i2c: qcom-geni: Drop of_platform.h include

On Tue 10 Mar 08:43 PDT 2020, Stephen Boyd wrote:

> This driver doesn't call any DT platform functions like of_platform_*().
> Remove the include as it isn't used.
>
> Cc: Alok Chauhan <[email protected]>
> Reviewed-by: Douglas Anderson <[email protected]>
> Reviewed-by: Brendan Higgins <[email protected]>

Reviewed-by: Bjorn Andersson <[email protected]>

> Signed-off-by: Stephen Boyd <[email protected]>
> ---
> drivers/i2c/busses/i2c-qcom-geni.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c
> index 2f5fb2e83f95..18d1e4fd4cf3 100644
> --- a/drivers/i2c/busses/i2c-qcom-geni.c
> +++ b/drivers/i2c/busses/i2c-qcom-geni.c
> @@ -10,7 +10,6 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> -#include <linux/of_platform.h>
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> #include <linux/qcom-geni-se.h>
> --
> Sent by a computer, using git, on the internet
>

2020-03-11 07:30:58

by Amit Kucheria

[permalink] [raw]
Subject: Re: [PATCH v2 3/3] i2c: qcom-geni: Drop of_platform.h include

On Tue, Mar 10, 2020 at 9:14 PM Stephen Boyd <[email protected]> wrote:
>
> This driver doesn't call any DT platform functions like of_platform_*().
> Remove the include as it isn't used.
>

Reviewed-by: Amit Kucheria <[email protected]>


> Cc: Alok Chauhan <[email protected]>
> Reviewed-by: Douglas Anderson <[email protected]>
> Reviewed-by: Brendan Higgins <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>
> ---
> drivers/i2c/busses/i2c-qcom-geni.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c
> index 2f5fb2e83f95..18d1e4fd4cf3 100644
> --- a/drivers/i2c/busses/i2c-qcom-geni.c
> +++ b/drivers/i2c/busses/i2c-qcom-geni.c
> @@ -10,7 +10,6 @@
> #include <linux/io.h>
> #include <linux/module.h>
> #include <linux/of.h>
> -#include <linux/of_platform.h>
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> #include <linux/qcom-geni-se.h>
> --
> Sent by a computer, using git, on the internet
>

2020-03-13 14:23:39

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH v2 3/3] i2c: qcom-geni: Drop of_platform.h include

On Tue, Mar 10, 2020 at 08:43:58AM -0700, Stephen Boyd wrote:
> This driver doesn't call any DT platform functions like of_platform_*().
> Remove the include as it isn't used.
>
> Cc: Alok Chauhan <[email protected]>
> Reviewed-by: Douglas Anderson <[email protected]>
> Reviewed-by: Brendan Higgins <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>

Applied to for-next, thanks!


Attachments:
(No filename) (430.00 B)
signature.asc (849.00 B)
Download all attachments