Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1181976 > unrolled thread

[PATCH 0/7] Remove clk.h from clk-provider.h (ARM part)

Started byStephen Boyd <sboyd@codeaurora.org>
First post2015-07-11 00:50 +0200
Last post2015-07-13 11:40 +0200
Articles 7 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/7] Remove clk.h from clk-provider.h (ARM part) Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
    [PATCH 4/7] ARM: mvebu: Remove clk-provider.h include Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
    [PATCH 3/7] ARM: keystone: Remove clk-provider.h include Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
    [PATCH 1/7] ARM: at91: Remove clk-provider.h include Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
    [PATCH 5/7] ARM: orion5x: Remove clk-provider.h include Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
    [PATCH 7/7] ARM: OMAP2+: Include clk.h Stephen Boyd <sboyd@codeaurora.org> - 2015-07-11 00:50 +0200
      Re: [PATCH 7/7] ARM: OMAP2+: Include clk.h Tero Kristo <t-kristo@ti.com> - 2015-07-13 11:40 +0200

#1181976 — [PATCH 0/7] Remove clk.h from clk-provider.h (ARM part)

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 0/7] Remove clk.h from clk-provider.h (ARM part)
Message-ID<pKPhg-3mu-7@gated-at.bofh.it>
This is the first set in a series of patches that removes
clk.h from clk-provider.h. This allows us to clearly see what
provider drivers are using the consumer API (clk.h) by
checking the includes. Currently clk.h is included by
clk-provider.h even though it doesn't need to so quite a few
clk provider drivers are relying on the implicit include.

Stephen Boyd (7):
  ARM: at91: Remove clk-provider.h include
  ARM: hisi: Remove clk-provider.h include
  ARM: keystone: Remove clk-provider.h include
  ARM: mvebu: Remove clk-provider.h include
  ARM: orion5x: Remove clk-provider.h include
  ARM: s3c64xx: Remove clk-provider.h include
  ARM: OMAP2+: Include clk.h

 arch/arm/mach-at91/at91rm9200.c    | 1 -
 arch/arm/mach-hisi/hisilicon.c     | 1 -
 arch/arm/mach-keystone/pm_domain.c | 1 -
 arch/arm/mach-mvebu/board-v7.c     | 1 -
 arch/arm/mach-omap2/clock.c        | 1 +
 arch/arm/mach-omap2/omap_hwmod.c   | 1 +
 arch/arm/mach-omap2/pm24xx.c       | 1 +
 arch/arm/mach-orion5x/board-dt.c   | 1 -
 arch/arm/mach-s3c64xx/common.c     | 1 -
 9 files changed, 3 insertions(+), 6 deletions(-)

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1181977 — [PATCH 4/7] ARM: mvebu: Remove clk-provider.h include

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 4/7] ARM: mvebu: Remove clk-provider.h include
Message-ID<pKPhg-3mu-11@gated-at.bofh.it>
In reply to#1181976
This file doesn't use the clk provider APIs. Remove the include.

Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---

Please ack if you want this to go through clk-tree.

 arch/arm/mach-mvebu/board-v7.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-mvebu/board-v7.c b/arch/arm/mach-mvebu/board-v7.c
index afee9083ad92..9f739f3cad4c 100644
--- a/arch/arm/mach-mvebu/board-v7.c
+++ b/arch/arm/mach-mvebu/board-v7.c
@@ -14,7 +14,6 @@
 
 #include <linux/kernel.h>
 #include <linux/init.h>
-#include <linux/clk-provider.h>
 #include <linux/of_address.h>
 #include <linux/of_fdt.h>
 #include <linux/of_platform.h>
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1181978 — [PATCH 3/7] ARM: keystone: Remove clk-provider.h include

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 3/7] ARM: keystone: Remove clk-provider.h include
Message-ID<pKPhg-3mu-13@gated-at.bofh.it>
In reply to#1181976
This file doesn't use the clk provider APIs. Remove the include.

Cc: Nishanth Menon <nm@ti.com>
Cc: Sekhar Nori <nsekhar@ti.com>
Cc: Santosh Shilimkar <ssantosh@kernel.org>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---

Please ack if you want this to go through clk-tree.

 arch/arm/mach-keystone/pm_domain.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-keystone/pm_domain.c b/arch/arm/mach-keystone/pm_domain.c
index edea697e8253..e283939a216f 100644
--- a/arch/arm/mach-keystone/pm_domain.c
+++ b/arch/arm/mach-keystone/pm_domain.c
@@ -16,7 +16,6 @@
 #include <linux/pm_runtime.h>
 #include <linux/pm_clock.h>
 #include <linux/platform_device.h>
-#include <linux/clk-provider.h>
 #include <linux/of.h>
 
 static struct dev_pm_domain keystone_pm_domain = {
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1181979 — [PATCH 1/7] ARM: at91: Remove clk-provider.h include

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 1/7] ARM: at91: Remove clk-provider.h include
Message-ID<pKPhg-3mu-15@gated-at.bofh.it>
In reply to#1181976
This file doesn't use the clk provider APIs. Remove the include.

Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---

Please ack if you want this to go through clk-tree.

 arch/arm/mach-at91/at91rm9200.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-at91/at91rm9200.c b/arch/arm/mach-at91/at91rm9200.c
index eaf58f88ef5d..685826c4a710 100644
--- a/arch/arm/mach-at91/at91rm9200.c
+++ b/arch/arm/mach-at91/at91rm9200.c
@@ -8,7 +8,6 @@
  * Licensed under GPLv2 or later.
  */
 
-#include <linux/clk-provider.h>
 #include <linux/of.h>
 #include <linux/of_platform.h>
 
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1181981 — [PATCH 5/7] ARM: orion5x: Remove clk-provider.h include

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 5/7] ARM: orion5x: Remove clk-provider.h include
Message-ID<pKPhg-3mu-19@gated-at.bofh.it>
In reply to#1181976
This file doesn't use the clk provider APIs. Remove the include.

Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---

Please ack if you want this to go through clk-tree.

 arch/arm/mach-orion5x/board-dt.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/mach-orion5x/board-dt.c b/arch/arm/mach-orion5x/board-dt.c
index 79f033b1ddff..d0871786dd8a 100644
--- a/arch/arm/mach-orion5x/board-dt.c
+++ b/arch/arm/mach-orion5x/board-dt.c
@@ -16,7 +16,6 @@
 #include <linux/of_platform.h>
 #include <linux/cpu.h>
 #include <linux/mbus.h>
-#include <linux/clk-provider.h>
 #include <linux/clocksource.h>
 #include <asm/system_misc.h>
 #include <asm/mach/arch.h>
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1181982 — [PATCH 7/7] ARM: OMAP2+: Include clk.h

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-07-11 00:50 +0200
Subject[PATCH 7/7] ARM: OMAP2+: Include clk.h
Message-ID<pKPhg-3mu-21@gated-at.bofh.it>
In reply to#1181976
These files use the consumer API, so include clk.h explicitly.

Cc: Tero Kristo <t-kristo@ti.com>
Cc: Tony Lindgren <tony@atomide.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---

Please ack so this can be routed through clk-tree. Otherwise
when clk.h is removed from clk-provider.h these files will
fail to compile.

 arch/arm/mach-omap2/clock.c      | 1 +
 arch/arm/mach-omap2/omap_hwmod.c | 1 +
 arch/arm/mach-omap2/pm24xx.c     | 1 +
 3 files changed, 3 insertions(+)

diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
index a699d7169307..f664dc664ded 100644
--- a/arch/arm/mach-omap2/clock.c
+++ b/arch/arm/mach-omap2/clock.c
@@ -20,6 +20,7 @@
 #include <linux/errno.h>
 #include <linux/err.h>
 #include <linux/delay.h>
+#include <linux/clk.h>
 #include <linux/clk-provider.h>
 #include <linux/io.h>
 #include <linux/bitops.h>
diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c
index d78c12e7cb5e..de43e4c9a865 100644
--- a/arch/arm/mach-omap2/omap_hwmod.c
+++ b/arch/arm/mach-omap2/omap_hwmod.c
@@ -130,6 +130,7 @@
 #include <linux/kernel.h>
 #include <linux/errno.h>
 #include <linux/io.h>
+#include <linux/clk.h>
 #include <linux/clk-provider.h>
 #include <linux/delay.h>
 #include <linux/err.h>
diff --git a/arch/arm/mach-omap2/pm24xx.c b/arch/arm/mach-omap2/pm24xx.c
index b1aad7e1426c..2a1a4180d5d0 100644
--- a/arch/arm/mach-omap2/pm24xx.c
+++ b/arch/arm/mach-omap2/pm24xx.c
@@ -25,6 +25,7 @@
 #include <linux/sysfs.h>
 #include <linux/module.h>
 #include <linux/delay.h>
+#include <linux/clk.h>
 #include <linux/clk-provider.h>
 #include <linux/irq.h>
 #include <linux/time.h>
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1182650 — Re: [PATCH 7/7] ARM: OMAP2+: Include clk.h

FromTero Kristo <t-kristo@ti.com>
Date2015-07-13 11:40 +0200
SubjectRe: [PATCH 7/7] ARM: OMAP2+: Include clk.h
Message-ID<pLInq-3kU-47@gated-at.bofh.it>
In reply to#1181982
On 07/11/2015 01:45 AM, Stephen Boyd wrote:
> These files use the consumer API, so include clk.h explicitly.
>
> Cc: Tero Kristo <t-kristo@ti.com>
> Cc: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> ---
>
> Please ack so this can be routed through clk-tree. Otherwise
> when clk.h is removed from clk-provider.h these files will
> fail to compile.

Acked-by: Tero Kristo <t-kristo@ti.com>

>
>   arch/arm/mach-omap2/clock.c      | 1 +
>   arch/arm/mach-omap2/omap_hwmod.c | 1 +
>   arch/arm/mach-omap2/pm24xx.c     | 1 +
>   3 files changed, 3 insertions(+)
>
> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
> index a699d7169307..f664dc664ded 100644
> --- a/arch/arm/mach-omap2/clock.c
> +++ b/arch/arm/mach-omap2/clock.c
> @@ -20,6 +20,7 @@
>   #include <linux/errno.h>
>   #include <linux/err.h>
>   #include <linux/delay.h>
> +#include <linux/clk.h>
>   #include <linux/clk-provider.h>
>   #include <linux/io.h>
>   #include <linux/bitops.h>
> diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c
> index d78c12e7cb5e..de43e4c9a865 100644
> --- a/arch/arm/mach-omap2/omap_hwmod.c
> +++ b/arch/arm/mach-omap2/omap_hwmod.c
> @@ -130,6 +130,7 @@
>   #include <linux/kernel.h>
>   #include <linux/errno.h>
>   #include <linux/io.h>
> +#include <linux/clk.h>
>   #include <linux/clk-provider.h>
>   #include <linux/delay.h>
>   #include <linux/err.h>
> diff --git a/arch/arm/mach-omap2/pm24xx.c b/arch/arm/mach-omap2/pm24xx.c
> index b1aad7e1426c..2a1a4180d5d0 100644
> --- a/arch/arm/mach-omap2/pm24xx.c
> +++ b/arch/arm/mach-omap2/pm24xx.c
> @@ -25,6 +25,7 @@
>   #include <linux/sysfs.h>
>   #include <linux/module.h>
>   #include <linux/delay.h>
> +#include <linux/clk.h>
>   #include <linux/clk-provider.h>
>   #include <linux/irq.h>
>   #include <linux/time.h>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web