Linux-OMAP Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH v2 0/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion
@ 2021-04-06  9:19 Zhen Lei
  2021-04-06  9:19 ` [PATCH v2 1/1] " Zhen Lei
  0 siblings, 1 reply; 4+ messages in thread
From: Zhen Lei @ 2021-04-06  9:19 UTC (permalink / raw)
  To: Eduardo Valentin, Keerthy, Zhang Rui, Daniel Lezcano,
	Amit Kucheria, linux-pm, linux-omap, linux-kernel
  Cc: Zhen Lei

v1 --> v2:
Arrange all included header files in alphabetical order.


Zhen Lei (1):
  thermal: ti-soc-thermal: Remove duplicated header file inclusion

 drivers/thermal/ti-soc-thermal/ti-bandgap.c | 35 ++++++++++++++---------------
 1 file changed, 17 insertions(+), 18 deletions(-)

-- 
1.8.3



^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v2 1/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion
  2021-04-06  9:19 [PATCH v2 0/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion Zhen Lei
@ 2021-04-06  9:19 ` Zhen Lei
  2021-04-06 10:38   ` J, KEERTHY
  2021-04-20 17:06   ` Daniel Lezcano
  0 siblings, 2 replies; 4+ messages in thread
From: Zhen Lei @ 2021-04-06  9:19 UTC (permalink / raw)
  To: Eduardo Valentin, Keerthy, Zhang Rui, Daniel Lezcano,
	Amit Kucheria, linux-pm, linux-omap, linux-kernel
  Cc: Zhen Lei

Delete one of the header files <linux/of_device.h> that are included
twice, all included header files are then rearranged alphabetically.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 drivers/thermal/ti-soc-thermal/ti-bandgap.c | 35 ++++++++++++++---------------
 1 file changed, 17 insertions(+), 18 deletions(-)

diff --git a/drivers/thermal/ti-soc-thermal/ti-bandgap.c b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
index 8a3646e26ddd208..5e7e80b4a8171c4 100644
--- a/drivers/thermal/ti-soc-thermal/ti-bandgap.c
+++ b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
@@ -9,30 +9,29 @@
  *   Eduardo Valentin <eduardo.valentin@ti.com>
  */
 
-#include <linux/module.h>
+#include <linux/clk.h>
+#include <linux/cpu_pm.h>
+#include <linux/device.h>
+#include <linux/err.h>
 #include <linux/export.h>
+#include <linux/gpio/consumer.h>
 #include <linux/init.h>
-#include <linux/kernel.h>
 #include <linux/interrupt.h>
-#include <linux/clk.h>
-#include <linux/gpio/consumer.h>
-#include <linux/platform_device.h>
-#include <linux/err.h>
-#include <linux/types.h>
-#include <linux/spinlock.h>
-#include <linux/sys_soc.h>
-#include <linux/reboot.h>
-#include <linux/of_device.h>
-#include <linux/of_platform.h>
-#include <linux/of_irq.h>
 #include <linux/io.h>
 #include <linux/iopoll.h>
-#include <linux/cpu_pm.h>
-#include <linux/device.h>
-#include <linux/pm_runtime.h>
-#include <linux/pm.h>
-#include <linux/of.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
 #include <linux/of_device.h>
+#include <linux/of.h>
+#include <linux/of_irq.h>
+#include <linux/of_platform.h>
+#include <linux/platform_device.h>
+#include <linux/pm.h>
+#include <linux/pm_runtime.h>
+#include <linux/reboot.h>
+#include <linux/spinlock.h>
+#include <linux/sys_soc.h>
+#include <linux/types.h>
 
 #include "ti-bandgap.h"
 
-- 
1.8.3



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion
  2021-04-06  9:19 ` [PATCH v2 1/1] " Zhen Lei
@ 2021-04-06 10:38   ` J, KEERTHY
  2021-04-20 17:06   ` Daniel Lezcano
  1 sibling, 0 replies; 4+ messages in thread
From: J, KEERTHY @ 2021-04-06 10:38 UTC (permalink / raw)
  To: Zhen Lei, Eduardo Valentin, Zhang Rui, Daniel Lezcano,
	Amit Kucheria, linux-pm, linux-omap, linux-kernel



On 4/6/2021 2:49 PM, Zhen Lei wrote:
> Delete one of the header files <linux/of_device.h> that are included
> twice, all included header files are then rearranged alphabetically.

Reviewed-by: Keerthy <j-keerthy@ti.com>

> 
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
> ---
>   drivers/thermal/ti-soc-thermal/ti-bandgap.c | 35 ++++++++++++++---------------
>   1 file changed, 17 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/thermal/ti-soc-thermal/ti-bandgap.c b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
> index 8a3646e26ddd208..5e7e80b4a8171c4 100644
> --- a/drivers/thermal/ti-soc-thermal/ti-bandgap.c
> +++ b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
> @@ -9,30 +9,29 @@
>    *   Eduardo Valentin <eduardo.valentin@ti.com>
>    */
>   
> -#include <linux/module.h>
> +#include <linux/clk.h>
> +#include <linux/cpu_pm.h>
> +#include <linux/device.h>
> +#include <linux/err.h>
>   #include <linux/export.h>
> +#include <linux/gpio/consumer.h>
>   #include <linux/init.h>
> -#include <linux/kernel.h>
>   #include <linux/interrupt.h>
> -#include <linux/clk.h>
> -#include <linux/gpio/consumer.h>
> -#include <linux/platform_device.h>
> -#include <linux/err.h>
> -#include <linux/types.h>
> -#include <linux/spinlock.h>
> -#include <linux/sys_soc.h>
> -#include <linux/reboot.h>
> -#include <linux/of_device.h>
> -#include <linux/of_platform.h>
> -#include <linux/of_irq.h>
>   #include <linux/io.h>
>   #include <linux/iopoll.h>
> -#include <linux/cpu_pm.h>
> -#include <linux/device.h>
> -#include <linux/pm_runtime.h>
> -#include <linux/pm.h>
> -#include <linux/of.h>
> +#include <linux/kernel.h>
> +#include <linux/module.h>
>   #include <linux/of_device.h>
> +#include <linux/of.h>
> +#include <linux/of_irq.h>
> +#include <linux/of_platform.h>
> +#include <linux/platform_device.h>
> +#include <linux/pm.h>
> +#include <linux/pm_runtime.h>
> +#include <linux/reboot.h>
> +#include <linux/spinlock.h>
> +#include <linux/sys_soc.h>
> +#include <linux/types.h>
>   
>   #include "ti-bandgap.h"
>   
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2 1/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion
  2021-04-06  9:19 ` [PATCH v2 1/1] " Zhen Lei
  2021-04-06 10:38   ` J, KEERTHY
@ 2021-04-20 17:06   ` Daniel Lezcano
  1 sibling, 0 replies; 4+ messages in thread
From: Daniel Lezcano @ 2021-04-20 17:06 UTC (permalink / raw)
  To: Zhen Lei, Eduardo Valentin, Keerthy, Zhang Rui, Amit Kucheria,
	linux-pm, linux-omap, linux-kernel

On 06/04/2021 11:19, Zhen Lei wrote:
> Delete one of the header files <linux/of_device.h> that are included
> twice, all included header files are then rearranged alphabetically.

The duplicate header file inclusion has been already fixed in a previous
patch.

Applied this patch by massaging the changelog accordingly and fixing the
conflict.

Thanks

  -- Daniel

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-06  9:19 [PATCH v2 0/1] thermal: ti-soc-thermal: Remove duplicated header file inclusion Zhen Lei
2021-04-06  9:19 ` [PATCH v2 1/1] " Zhen Lei
2021-04-06 10:38   ` J, KEERTHY
2021-04-20 17:06   ` Daniel Lezcano

Linux-OMAP Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-omap/0 linux-omap/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-omap linux-omap/ https://lore.kernel.org/linux-omap \
		linux-omap@vger.kernel.org
	public-inbox-index linux-omap

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-omap


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git