linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding
@ 2018-04-18 14:50 Geert Uytterhoeven
  2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
                   ` (4 more replies)
  0 siblings, 5 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

	Hi Mike, Stephen, et al,

This patch series replaces open-coded versions of
of_clk_get_parent_count() by calls to the helper function.

Changes compared to v2:
  - Add Acked-by, Reviewed-by,
  - Add SPDX-License-Identifier to <linux/of_clk.h>,
  - Add <linux/of_clk.h> to clock section in MAINTAINERS (note that Rob
    is still listed as a maintainer, too, due to the include/linux/of*.h
    catch-all rule),

Changes compared to v1:
  - Move OF clock helpers (a.o. of_clk_get_parent_count()) to their own
    include file <linux/of_clk.h>, as suggested by Stephen,
  - Update includes for the above change,
  - Fix timer-sp804, now a dummy is available in the !CONFIG_COMMON_CLK
    case,
  - Fix a new occurrence.

Note that patches 2-5 depend on the first patch.

Thanks!

Geert Uytterhoeven (5):
  clk: Extract OF clock helpers in <linux/of_clk.h>
  ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding
  soc: rockchip: power-domain: Use of_clk_get_parent_count() instead of
    open coding
  soc/tegra: pmc: Use of_clk_get_parent_count() instead of open coding
  pinctrl: sunxi: Use of_clk_get_parent_count() instead of open coding

 MAINTAINERS                           |  1 +
 drivers/clocksource/timer-sp804.c     |  3 ++-
 drivers/pinctrl/sunxi/pinctrl-sunxi.c |  4 ++--
 drivers/soc/rockchip/pm_domains.c     |  4 ++--
 drivers/soc/tegra/pmc.c               |  3 ++-
 include/linux/clk-provider.h          | 14 +-------------
 include/linux/of_clk.h                | 30 ++++++++++++++++++++++++++++++
 7 files changed, 40 insertions(+), 19 deletions(-)
 create mode 100644 include/linux/of_clk.h

-- 
2.7.4

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

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

* [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h>
  2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
@ 2018-04-18 14:50 ` Geert Uytterhoeven
  2018-04-19 22:25   ` Stephen Boyd
  2018-05-02 15:38   ` Stephen Boyd
  2018-04-18 14:50 ` [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
                   ` (3 subsequent siblings)
  4 siblings, 2 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

The use of of_clk_get_parent_{count,name}() and of_clk_init() is not
limited to clock providers.

Hence move these helpers into their own header file, so callers that are
not clock providers no longer have to include <linux/clk-provider.h>.

Suggested-by: Stephen Boyd <sboyd@kernel.org>
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
---
v3:
  - Add Reviewed-by,
  - Add SPDX-License-Identifier,
  - Add to clock section in MAINTAINERS (note that Rob is still listed
    as a maintainer, too, due to the include/linux/of*.h catch-all
    rule),

v2:
  - New.
---
 MAINTAINERS                  |  1 +
 include/linux/clk-provider.h | 14 +-------------
 include/linux/of_clk.h       | 30 ++++++++++++++++++++++++++++++
 3 files changed, 32 insertions(+), 13 deletions(-)
 create mode 100644 include/linux/of_clk.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 50927cba875771e8..abf08b50a85d969d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3556,6 +3556,7 @@ F:	drivers/clk/
 X:	drivers/clk/clkdev.c
 F:	include/linux/clk-pr*
 F:	include/linux/clk/
+F:	include/linux/of_clk.h
 
 COMMON INTERNET FILE SYSTEM (CIFS)
 M:	Steve French <sfrench@samba.org>
diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h
index 210a890008f9e129..61cb4729f22acd6b 100644
--- a/include/linux/clk-provider.h
+++ b/include/linux/clk-provider.h
@@ -13,6 +13,7 @@
 
 #include <linux/io.h>
 #include <linux/of.h>
+#include <linux/of_clk.h>
 
 #ifdef CONFIG_COMMON_CLK
 
@@ -890,13 +891,10 @@ struct clk_hw *of_clk_hw_simple_get(struct of_phandle_args *clkspec,
 struct clk *of_clk_src_onecell_get(struct of_phandle_args *clkspec, void *data);
 struct clk_hw *of_clk_hw_onecell_get(struct of_phandle_args *clkspec,
 				     void *data);
-unsigned int of_clk_get_parent_count(struct device_node *np);
 int of_clk_parent_fill(struct device_node *np, const char **parents,
 		       unsigned int size);
-const char *of_clk_get_parent_name(struct device_node *np, int index);
 int of_clk_detect_critical(struct device_node *np, int index,
 			    unsigned long *flags);
-void of_clk_init(const struct of_device_id *matches);
 
 #else /* !CONFIG_OF */
 
@@ -943,26 +941,16 @@ of_clk_hw_onecell_get(struct of_phandle_args *clkspec, void *data)
 {
 	return ERR_PTR(-ENOENT);
 }
-static inline unsigned int of_clk_get_parent_count(struct device_node *np)
-{
-	return 0;
-}
 static inline int of_clk_parent_fill(struct device_node *np,
 				     const char **parents, unsigned int size)
 {
 	return 0;
 }
-static inline const char *of_clk_get_parent_name(struct device_node *np,
-						 int index)
-{
-	return NULL;
-}
 static inline int of_clk_detect_critical(struct device_node *np, int index,
 					  unsigned long *flags)
 {
 	return 0;
 }
-static inline void of_clk_init(const struct of_device_id *matches) {}
 #endif /* CONFIG_OF */
 
 /*
diff --git a/include/linux/of_clk.h b/include/linux/of_clk.h
new file mode 100644
index 0000000000000000..b27da9f164cbd221
--- /dev/null
+++ b/include/linux/of_clk.h
@@ -0,0 +1,30 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * OF clock helpers
+ */
+
+#ifndef __LINUX_OF_CLK_H
+#define __LINUX_OF_CLK_H
+
+#if defined(CONFIG_COMMON_CLK) && defined(CONFIG_OF)
+
+unsigned int of_clk_get_parent_count(struct device_node *np);
+const char *of_clk_get_parent_name(struct device_node *np, int index);
+void of_clk_init(const struct of_device_id *matches);
+
+#else /* !CONFIG_COMMON_CLK || !CONFIG_OF */
+
+static inline unsigned int of_clk_get_parent_count(struct device_node *np)
+{
+	return 0;
+}
+static inline const char *of_clk_get_parent_name(struct device_node *np,
+						 int index)
+{
+	return NULL;
+}
+static inline void of_clk_init(const struct of_device_id *matches) {}
+
+#endif /* !CONFIG_COMMON_CLK || !CONFIG_OF */
+
+#endif /* __LINUX_OF_CLK_H */
-- 
2.7.4

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

* [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
  2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
@ 2018-04-18 14:50 ` Geert Uytterhoeven
  2018-05-02 15:38   ` Stephen Boyd
  2018-04-18 14:50 ` [PATCH v3 3/5] soc: rockchip: power-domain: " Geert Uytterhoeven
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
<linux/of_clk.h>".

v3:
  - Add Acked-by,

v2:
  - of_clk_get_parent_count() was moved to <linux/of_clk.h>,
  - Dropped RFC, as a dummy is now available in the !CONFIG_COMMON_CLK
    case.
---
 drivers/clocksource/timer-sp804.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clocksource/timer-sp804.c b/drivers/clocksource/timer-sp804.c
index 3ac9dec9a03852f9..e01222ea888ff9d2 100644
--- a/drivers/clocksource/timer-sp804.c
+++ b/drivers/clocksource/timer-sp804.c
@@ -27,6 +27,7 @@
 #include <linux/io.h>
 #include <linux/of.h>
 #include <linux/of_address.h>
+#include <linux/of_clk.h>
 #include <linux/of_irq.h>
 #include <linux/sched_clock.h>
 
@@ -245,7 +246,7 @@ static int __init sp804_of_init(struct device_node *np)
 		clk1 = NULL;
 
 	/* Get the 2nd clock if the timer has 3 timer clocks */
-	if (of_count_phandle_with_args(np, "clocks", "#clock-cells") == 3) {
+	if (of_clk_get_parent_count(np) == 3) {
 		clk2 = of_clk_get(np, 1);
 		if (IS_ERR(clk2)) {
 			pr_err("sp804: %s clock not found: %d\n", np->name,
-- 
2.7.4

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

* [PATCH v3 3/5] soc: rockchip: power-domain: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
  2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
  2018-04-18 14:50 ` [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
@ 2018-04-18 14:50 ` Geert Uytterhoeven
  2018-04-30 13:41   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  2018-04-18 14:50 ` [PATCH v3 4/5] soc/tegra: pmc: " Geert Uytterhoeven
  2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
  4 siblings, 2 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

As of_clk_get_parent_count() returns zero on failure, while
of_count_phandle_with_args() might return a negative error code, this
also fixes the issue of possibly using a negative number in the
allocation below.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
---
This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
<linux/of_clk.h>".

v3:
  - Add Reviewed-by,

v2:
  - of_clk_get_parent_count() was moved to <linux/of_clk.h>.
---
 drivers/soc/rockchip/pm_domains.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/soc/rockchip/pm_domains.c b/drivers/soc/rockchip/pm_domains.c
index 53efc386b1ada8cf..13913d40c8213e36 100644
--- a/drivers/soc/rockchip/pm_domains.c
+++ b/drivers/soc/rockchip/pm_domains.c
@@ -14,6 +14,7 @@
 #include <linux/pm_clock.h>
 #include <linux/pm_domain.h>
 #include <linux/of_address.h>
+#include <linux/of_clk.h>
 #include <linux/of_platform.h>
 #include <linux/clk.h>
 #include <linux/regmap.h>
@@ -400,8 +401,7 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 	pd->info = pd_info;
 	pd->pmu = pmu;
 
-	pd->num_clks = of_count_phandle_with_args(node, "clocks",
-						  "#clock-cells");
+	pd->num_clks = of_clk_get_parent_count(node);
 	if (pd->num_clks > 0) {
 		pd->clks = devm_kcalloc(pmu->dev, pd->num_clks,
 					sizeof(*pd->clks), GFP_KERNEL);
-- 
2.7.4

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

* [PATCH v3 4/5] soc/tegra: pmc: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
                   ` (2 preceding siblings ...)
  2018-04-18 14:50 ` [PATCH v3 3/5] soc: rockchip: power-domain: " Geert Uytterhoeven
@ 2018-04-18 14:50 ` Geert Uytterhoeven
  2018-04-30 13:44   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
  4 siblings, 2 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

As of_clk_get_parent_count() returns zero on failure, while
of_count_phandle_with_args() might return a negative error code, this
also fixes the issue of possibly using a very big number in the
allocation below.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Jon Hunter <jonathanh@nvidia.com>
---
This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
<linux/of_clk.h>".

v3:
  - Add Acked-by,

v2:
  - of_clk_get_parent_count() was moved to <linux/of_clk.h>.
---
 drivers/soc/tegra/pmc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index d9fcdb592b3966a5..d8cb48a4b8eb1b78 100644
--- a/drivers/soc/tegra/pmc.c
+++ b/drivers/soc/tegra/pmc.c
@@ -31,6 +31,7 @@
 #include <linux/iopoll.h>
 #include <linux/of.h>
 #include <linux/of_address.h>
+#include <linux/of_clk.h>
 #include <linux/of_platform.h>
 #include <linux/platform_device.h>
 #include <linux/pm_domain.h>
@@ -725,7 +726,7 @@ static int tegra_powergate_of_get_clks(struct tegra_powergate *pg,
 	unsigned int i, count;
 	int err;
 
-	count = of_count_phandle_with_args(np, "clocks", "#clock-cells");
+	count = of_clk_get_parent_count(np);
 	if (count == 0)
 		return -ENODEV;
 
-- 
2.7.4

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

* [PATCH v3 5/5] pinctrl: sunxi: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
                   ` (3 preceding siblings ...)
  2018-04-18 14:50 ` [PATCH v3 4/5] soc/tegra: pmc: " Geert Uytterhoeven
@ 2018-04-18 14:50 ` Geert Uytterhoeven
  2018-04-26 11:57   ` Linus Walleij
                     ` (2 more replies)
  4 siblings, 3 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-18 14:50 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

A new open coder has crept in since 470b73a38470e8ba ("pinctrl: sunxi:
Use of_clk_get_parent_count() instead of open coding"), replace it.

of_clk_get_parent_count() was moved to <linux/of_clk.h>, so include that
instead of <linux/clk-provider.h>.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
---
This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
<linux/of_clk.h>".

v3:
  - Add Acked-by,

v2:
  - New.
---
 drivers/pinctrl/sunxi/pinctrl-sunxi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
index 020d6d84639ca002..25e80a5370ca02f6 100644
--- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
+++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
@@ -12,12 +12,12 @@
 
 #include <linux/io.h>
 #include <linux/clk.h>
-#include <linux/clk-provider.h>
 #include <linux/gpio/driver.h>
 #include <linux/irqdomain.h>
 #include <linux/irqchip/chained_irq.h>
 #include <linux/export.h>
 #include <linux/of.h>
+#include <linux/of_clk.h>
 #include <linux/of_address.h>
 #include <linux/of_device.h>
 #include <linux/of_irq.h>
@@ -1361,7 +1361,7 @@ int sunxi_pinctrl_init_with_variant(struct platform_device *pdev,
 			goto gpiochip_error;
 	}
 
-	ret = of_count_phandle_with_args(node, "clocks", "#clock-cells");
+	ret = of_clk_get_parent_count(node);
 	clk = devm_clk_get(&pdev->dev, ret == 1 ? NULL : "apb");
 	if (IS_ERR(clk)) {
 		ret = PTR_ERR(clk);
-- 
2.7.4

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

* Re: [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h>
  2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
@ 2018-04-19 22:25   ` Stephen Boyd
  2018-04-20  6:28     ` Geert Uytterhoeven
  2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 1 reply; 18+ messages in thread
From: Stephen Boyd @ 2018-04-19 22:25 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:01)
> The use of of_clk_get_parent_{count,name}() and of_clk_init() is not
> limited to clock providers.
> 
> Hence move these helpers into their own header file, so callers that are
> not clock providers no longer have to include <linux/clk-provider.h>.
> 
> Suggested-by: Stephen Boyd <sboyd@kernel.org>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---
> v3:
>   - Add Reviewed-by,
>   - Add SPDX-License-Identifier,
>   - Add to clock section in MAINTAINERS (note that Rob is still listed
>     as a maintainer, too, due to the include/linux/of*.h catch-all
>     rule),

Can you X: out this file so Rob is happy? Or that doesn't work?

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

* Re: [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h>
  2018-04-19 22:25   ` Stephen Boyd
@ 2018-04-20  6:28     ` Geert Uytterhoeven
  2018-04-24 15:44       ` Rob Herring
  0 siblings, 1 reply; 18+ messages in thread
From: Geert Uytterhoeven @ 2018-04-20  6:28 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Geert Uytterhoeven, Michael Turquette, Rob Herring, Frank Rowand,
	Daniel Lezcano, Thomas Gleixner, Maxime Ripard, Chen-Yu Tsai,
	Linus Walleij, Heiko Stuebner, Thierry Reding, Jonathan Hunter,
	linux-clk, Linux ARM, open list:GPIO SUBSYSTEM,
	open list:ARM/Rockchip SoC...,
	linux-tegra,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Linux Kernel Mailing List

Hi Stephen, Rob,

On Fri, Apr 20, 2018 at 12:25 AM, Stephen Boyd <sboyd@kernel.org> wrote:
> Quoting Geert Uytterhoeven (2018-04-18 07:50:01)
>> The use of of_clk_get_parent_{count,name}() and of_clk_init() is not
>> limited to clock providers.
>>
>> Hence move these helpers into their own header file, so callers that are
>> not clock providers no longer have to include <linux/clk-provider.h>.
>>
>> Suggested-by: Stephen Boyd <sboyd@kernel.org>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
>> ---
>> v3:
>>   - Add Reviewed-by,
>>   - Add SPDX-License-Identifier,
>>   - Add to clock section in MAINTAINERS (note that Rob is still listed
>>     as a maintainer, too, due to the include/linux/of*.h catch-all
>>     rule),
>
> Can you X: out this file so Rob is happy? Or that doesn't work?

I guess that should work.

My point here is that due to the catch-all rule, he's listed as maintainer
for of_{dma,gpio,irq,iommu,mdio,net,pci} too, which are all helpers for
other subsystems. Perhaps these should be X'd-out too?
Or is it OK without X-ing them out, as the clock maintainers are now shown, too?

Rob: What's your preference?

Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h>
  2018-04-20  6:28     ` Geert Uytterhoeven
@ 2018-04-24 15:44       ` Rob Herring
  0 siblings, 0 replies; 18+ messages in thread
From: Rob Herring @ 2018-04-24 15:44 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Stephen Boyd, Geert Uytterhoeven, Michael Turquette,
	Frank Rowand, Daniel Lezcano, Thomas Gleixner, Maxime Ripard,
	Chen-Yu Tsai, Linus Walleij, Heiko Stuebner, Thierry Reding,
	Jonathan Hunter, linux-clk, Linux ARM, open list:GPIO SUBSYSTEM,
	open list:ARM/Rockchip SoC...,
	linux-tegra,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Linux Kernel Mailing List

On Fri, Apr 20, 2018 at 08:28:28AM +0200, Geert Uytterhoeven wrote:
> Hi Stephen, Rob,
> 
> On Fri, Apr 20, 2018 at 12:25 AM, Stephen Boyd <sboyd@kernel.org> wrote:
> > Quoting Geert Uytterhoeven (2018-04-18 07:50:01)
> >> The use of of_clk_get_parent_{count,name}() and of_clk_init() is not
> >> limited to clock providers.
> >>
> >> Hence move these helpers into their own header file, so callers that are
> >> not clock providers no longer have to include <linux/clk-provider.h>.
> >>
> >> Suggested-by: Stephen Boyd <sboyd@kernel.org>
> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> >> ---
> >> v3:
> >>   - Add Reviewed-by,
> >>   - Add SPDX-License-Identifier,
> >>   - Add to clock section in MAINTAINERS (note that Rob is still listed
> >>     as a maintainer, too, due to the include/linux/of*.h catch-all
> >>     rule),
> >
> > Can you X: out this file so Rob is happy? Or that doesn't work?
> 
> I guess that should work.
> 
> My point here is that due to the catch-all rule, he's listed as maintainer
> for of_{dma,gpio,irq,iommu,mdio,net,pci} too, which are all helpers for
> other subsystems. Perhaps these should be X'd-out too?
> Or is it OK without X-ing them out, as the clock maintainers are now shown, too?
> 
> Rob: What's your preference?

It's fine as-is.

Rob

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

* Re: [PATCH v3 5/5] pinctrl: sunxi: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
@ 2018-04-26 11:57   ` Linus Walleij
  2018-04-30 13:46   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  2 siblings, 0 replies; 18+ messages in thread
From: Linus Walleij @ 2018-04-26 11:57 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Michael Turquette, Stephen Boyd, Rob Herring, Frank Rowand,
	Daniel Lezcano, Thomas Gleixner, Maxime Ripard, Chen-Yu Tsai,
	Heiko Stuebner, Thierry Reding, Jonathan Hunter, linux-clk,
	Linux ARM, open list:GPIO SUBSYSTEM,
	open list:ARM/Rockchip SoC...,
	linux-tegra,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-kernel

On Wed, Apr 18, 2018 at 4:50 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:

> A new open coder has crept in since 470b73a38470e8ba ("pinctrl: sunxi:
> Use of_clk_get_parent_count() instead of open coding"), replace it.
>
> of_clk_get_parent_count() was moved to <linux/of_clk.h>, so include that
> instead of <linux/clk-provider.h>.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
> ---
> This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
> <linux/of_clk.h>".
>
> v3:
>   - Add Acked-by,

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Please merge this through the clk tree.

Yours,
Linus Walleij

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

* Re: [PATCH v3 3/5] soc: rockchip: power-domain: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 3/5] soc: rockchip: power-domain: " Geert Uytterhoeven
@ 2018-04-30 13:41   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 0 replies; 18+ messages in thread
From: Daniel Lezcano @ 2018-04-30 13:41 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Michael Turquette, Stephen Boyd, Rob Herring, Frank Rowand,
	Thomas Gleixner, Maxime Ripard, Chen-Yu Tsai, Linus Walleij,
	Heiko Stuebner, Thierry Reding, Jonathan Hunter, linux-clk,
	linux-arm-kernel, linux-gpio, linux-rockchip, linux-tegra,
	devicetree, linux-kernel

On Wed, Apr 18, 2018 at 04:50:03PM +0200, Geert Uytterhoeven wrote:
> As of_clk_get_parent_count() returns zero on failure, while
> of_count_phandle_with_args() might return a negative error code, this
> also fixes the issue of possibly using a negative number in the
> allocation below.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>

Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>

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

* Re: [PATCH v3 4/5] soc/tegra: pmc: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 4/5] soc/tegra: pmc: " Geert Uytterhoeven
@ 2018-04-30 13:44   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 0 replies; 18+ messages in thread
From: Daniel Lezcano @ 2018-04-30 13:44 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Michael Turquette, Stephen Boyd, Rob Herring, Frank Rowand,
	Thomas Gleixner, Maxime Ripard, Chen-Yu Tsai, Linus Walleij,
	Heiko Stuebner, Thierry Reding, Jonathan Hunter, linux-clk,
	linux-arm-kernel, linux-gpio, linux-rockchip, linux-tegra,
	devicetree, linux-kernel

On Wed, Apr 18, 2018 at 04:50:04PM +0200, Geert Uytterhoeven wrote:
> As of_clk_get_parent_count() returns zero on failure, while
> of_count_phandle_with_args() might return a negative error code, this
> also fixes the issue of possibly using a very big number in the
> allocation below.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Jon Hunter <jonathanh@nvidia.com>
> ---
> This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
> <linux/of_clk.h>".
> 
> v3:
>   - Add Acked-by,
> 
> v2:
>   - of_clk_get_parent_count() was moved to <linux/of_clk.h>.
> ---

Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>




 <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] 18+ messages in thread

* Re: [PATCH v3 5/5] pinctrl: sunxi: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
  2018-04-26 11:57   ` Linus Walleij
@ 2018-04-30 13:46   ` Daniel Lezcano
  2018-05-02 15:38   ` Stephen Boyd
  2 siblings, 0 replies; 18+ messages in thread
From: Daniel Lezcano @ 2018-04-30 13:46 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Michael Turquette, Stephen Boyd, Rob Herring, Frank Rowand,
	Thomas Gleixner, Maxime Ripard, Chen-Yu Tsai, Linus Walleij,
	Heiko Stuebner, Thierry Reding, Jonathan Hunter, linux-clk,
	linux-arm-kernel, linux-gpio, linux-rockchip, linux-tegra,
	devicetree, linux-kernel

On Wed, Apr 18, 2018 at 04:50:05PM +0200, Geert Uytterhoeven wrote:
> A new open coder has crept in since 470b73a38470e8ba ("pinctrl: sunxi:
> Use of_clk_get_parent_count() instead of open coding"), replace it.
> 
> of_clk_get_parent_count() was moved to <linux/of_clk.h>, so include that
> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
> ---
> This depends on "[PATCH v3 1/5] clk: Extract OF clock helpers in
> <linux/of_clk.h>".
> 
> v3:
>   - Add Acked-by,
> 
> v2:
>   - New.
> ---

Reviewed-by: Daniel Lezcano <daniel.lezcano@linaro.org>

-- 

 <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] 18+ messages in thread

* Re: [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h>
  2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
  2018-04-19 22:25   ` Stephen Boyd
@ 2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 0 replies; 18+ messages in thread
From: Stephen Boyd @ 2018-05-02 15:38 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:01)
> The use of of_clk_get_parent_{count,name}() and of_clk_init() is not
> limited to clock providers.
> 
> Hence move these helpers into their own header file, so callers that are
> not clock providers no longer have to include <linux/clk-provider.h>.
> 
> Suggested-by: Stephen Boyd <sboyd@kernel.org>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---

Applied to clk-next

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

* Re: [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
@ 2018-05-02 15:38   ` Stephen Boyd
  0 siblings, 0 replies; 18+ messages in thread
From: Stephen Boyd @ 2018-05-02 15:38 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:02)
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---

Applied to clk-next

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

* Re: [PATCH v3 3/5] soc: rockchip: power-domain: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 3/5] soc: rockchip: power-domain: " Geert Uytterhoeven
  2018-04-30 13:41   ` Daniel Lezcano
@ 2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 0 replies; 18+ messages in thread
From: Stephen Boyd @ 2018-05-02 15:38 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:03)
> As of_clk_get_parent_count() returns zero on failure, while
> of_count_phandle_with_args() might return a negative error code, this
> also fixes the issue of possibly using a negative number in the
> allocation below.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> ---

Applied to clk-next

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

* Re: [PATCH v3 4/5] soc/tegra: pmc: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 4/5] soc/tegra: pmc: " Geert Uytterhoeven
  2018-04-30 13:44   ` Daniel Lezcano
@ 2018-05-02 15:38   ` Stephen Boyd
  1 sibling, 0 replies; 18+ messages in thread
From: Stephen Boyd @ 2018-05-02 15:38 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:04)
> As of_clk_get_parent_count() returns zero on failure, while
> of_count_phandle_with_args() might return a negative error code, this
> also fixes the issue of possibly using a very big number in the
> allocation below.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Jon Hunter <jonathanh@nvidia.com>
> ---

Applied to clk-next

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

* Re: [PATCH v3 5/5] pinctrl: sunxi: Use of_clk_get_parent_count() instead of open coding
  2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
  2018-04-26 11:57   ` Linus Walleij
  2018-04-30 13:46   ` Daniel Lezcano
@ 2018-05-02 15:38   ` Stephen Boyd
  2 siblings, 0 replies; 18+ messages in thread
From: Stephen Boyd @ 2018-05-02 15:38 UTC (permalink / raw)
  To: Geert Uytterhoeven, Michael Turquette
  Cc: Rob Herring, Frank Rowand, Daniel Lezcano, Thomas Gleixner,
	Maxime Ripard, Chen-Yu Tsai, Linus Walleij, Heiko Stuebner,
	Thierry Reding, Jonathan Hunter, linux-clk, linux-arm-kernel,
	linux-gpio, linux-rockchip, linux-tegra, devicetree,
	linux-kernel, Geert Uytterhoeven

Quoting Geert Uytterhoeven (2018-04-18 07:50:05)
> A new open coder has crept in since 470b73a38470e8ba ("pinctrl: sunxi:
> Use of_clk_get_parent_count() instead of open coding"), replace it.
> 
> of_clk_get_parent_count() was moved to <linux/of_clk.h>, so include that
> instead of <linux/clk-provider.h>.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
> ---

Applied to clk-next

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

end of thread, other threads:[~2018-05-02 15:39 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-18 14:50 [PATCH v3 0/5] Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
2018-04-18 14:50 ` [PATCH v3 1/5] clk: Extract OF clock helpers in <linux/of_clk.h> Geert Uytterhoeven
2018-04-19 22:25   ` Stephen Boyd
2018-04-20  6:28     ` Geert Uytterhoeven
2018-04-24 15:44       ` Rob Herring
2018-05-02 15:38   ` Stephen Boyd
2018-04-18 14:50 ` [PATCH v3 2/5] ARM: timer-sp: Use of_clk_get_parent_count() instead of open coding Geert Uytterhoeven
2018-05-02 15:38   ` Stephen Boyd
2018-04-18 14:50 ` [PATCH v3 3/5] soc: rockchip: power-domain: " Geert Uytterhoeven
2018-04-30 13:41   ` Daniel Lezcano
2018-05-02 15:38   ` Stephen Boyd
2018-04-18 14:50 ` [PATCH v3 4/5] soc/tegra: pmc: " Geert Uytterhoeven
2018-04-30 13:44   ` Daniel Lezcano
2018-05-02 15:38   ` Stephen Boyd
2018-04-18 14:50 ` [PATCH v3 5/5] pinctrl: sunxi: " Geert Uytterhoeven
2018-04-26 11:57   ` Linus Walleij
2018-04-30 13:46   ` Daniel Lezcano
2018-05-02 15:38   ` Stephen Boyd

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).