linux-clk.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table
@ 2020-11-03 16:24 Krzysztof Kozlowski
  2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
                   ` (7 more replies)
  0 siblings, 8 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might be not relevant here).  This
fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):

    drivers/clk/clk-pwm.c:139:34: warning:
        ‘clk_pwm_dt_ids’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/clk-pwm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/clk-pwm.c b/drivers/clk/clk-pwm.c
index 86f2e2d3fc02..da2c8eddfd9f 100644
--- a/drivers/clk/clk-pwm.c
+++ b/drivers/clk/clk-pwm.c
@@ -147,7 +147,7 @@ static struct platform_driver clk_pwm_driver = {
 	.remove = clk_pwm_remove,
 	.driver = {
 		.name = "pwm-clock",
-		.of_match_table = of_match_ptr(clk_pwm_dt_ids),
+		.of_match_table = clk_pwm_dt_ids,
 	},
 };
 
-- 
2.25.1


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

* [PATCH 2/8] clk: imx8mm: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  2020-11-03 16:24 ` [PATCH 3/8] clk: imx8mn: " Krzysztof Kozlowski
                   ` (6 subsequent siblings)
  7 siblings, 2 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might be not relevant here).  This
fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):

    drivers/clk/imx/clk-imx8mm.c:641:34: warning:
        ‘imx8mm_clk_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/imx/clk-imx8mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx8mm.c b/drivers/clk/imx/clk-imx8mm.c
index 0de0be0cf548..e27890b61fb6 100644
--- a/drivers/clk/imx/clk-imx8mm.c
+++ b/drivers/clk/imx/clk-imx8mm.c
@@ -653,7 +653,7 @@ static struct platform_driver imx8mm_clk_driver = {
 		 * reloading the driver will crash or break devices.
 		 */
 		.suppress_bind_attrs = true,
-		.of_match_table = of_match_ptr(imx8mm_clk_of_match),
+		.of_match_table = imx8mm_clk_of_match,
 	},
 };
 module_platform_driver(imx8mm_clk_driver);
-- 
2.25.1


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

* [PATCH 3/8] clk: imx8mn: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
  2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  2020-11-03 16:24 ` [PATCH 4/8] clk: imx8mp: " Krzysztof Kozlowski
                   ` (5 subsequent siblings)
  7 siblings, 2 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might be not relevant here).  This
fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):

    drivers/clk/imx/clk-imx8mn.c:592:34: warning:
        ‘imx8mn_clk_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/imx/clk-imx8mn.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx8mn.c b/drivers/clk/imx/clk-imx8mn.c
index e984de543f0b..18f81419f355 100644
--- a/drivers/clk/imx/clk-imx8mn.c
+++ b/drivers/clk/imx/clk-imx8mn.c
@@ -604,7 +604,7 @@ static struct platform_driver imx8mn_clk_driver = {
 		 * reloading the driver will crash or break devices.
 		 */
 		.suppress_bind_attrs = true,
-		.of_match_table = of_match_ptr(imx8mn_clk_of_match),
+		.of_match_table = imx8mn_clk_of_match,
 	},
 };
 module_platform_driver(imx8mn_clk_driver);
-- 
2.25.1


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

* [PATCH 4/8] clk: imx8mp: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
  2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
  2020-11-03 16:24 ` [PATCH 3/8] clk: imx8mn: " Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  2020-11-03 16:24 ` [PATCH 5/8] clk: imx8mq: " Krzysztof Kozlowski
                   ` (4 subsequent siblings)
  7 siblings, 2 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski, kernel test robot

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might be not relevant here).  This
fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):

    drivers/clk/imx/clk-imx8mp.c:751:34: warning:
        ‘imx8mp_clk_of_match’ defined but not used [-Wunused-const-variable=]

Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/imx/clk-imx8mp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c
index 12ce4770f702..5295a5156ab0 100644
--- a/drivers/clk/imx/clk-imx8mp.c
+++ b/drivers/clk/imx/clk-imx8mp.c
@@ -763,7 +763,7 @@ static struct platform_driver imx8mp_clk_driver = {
 		 * reloading the driver will crash or break devices.
 		 */
 		.suppress_bind_attrs = true,
-		.of_match_table = of_match_ptr(imx8mp_clk_of_match),
+		.of_match_table = imx8mp_clk_of_match,
 	},
 };
 module_platform_driver(imx8mp_clk_driver);
-- 
2.25.1


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

* [PATCH 5/8] clk: imx8mq: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
                   ` (2 preceding siblings ...)
  2020-11-03 16:24 ` [PATCH 4/8] clk: imx8mp: " Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  2020-11-03 16:24 ` [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function Krzysztof Kozlowski
                   ` (3 subsequent siblings)
  7 siblings, 2 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might be not relevant here).  This
fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):

    drivers/clk/imx/clk-imx8mq.c:626:34: warning:
        ‘imx8mq_clk_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/imx/clk-imx8mq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx8mq.c b/drivers/clk/imx/clk-imx8mq.c
index 8265d1d48af4..75186888ecae 100644
--- a/drivers/clk/imx/clk-imx8mq.c
+++ b/drivers/clk/imx/clk-imx8mq.c
@@ -639,7 +639,7 @@ static struct platform_driver imx8mq_clk_driver = {
 		 * reloading the driver will crash or break devices.
 		 */
 		.suppress_bind_attrs = true,
-		.of_match_table = of_match_ptr(imx8mq_clk_of_match),
+		.of_match_table = imx8mq_clk_of_match,
 	},
 };
 module_platform_driver(imx8mq_clk_driver);
-- 
2.25.1


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

* [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
                   ` (3 preceding siblings ...)
  2020-11-03 16:24 ` [PATCH 5/8] clk: imx8mq: " Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-04 10:53   ` Geert Uytterhoeven
  2020-11-03 16:24 ` [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv Krzysztof Kozlowski
                   ` (2 subsequent siblings)
  7 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The function rcar_r8a779a0_cpg_clk_register() is not used outside of the
unit so it can be made static to fix compilation warning:

    drivers/clk/renesas/r8a779a0-cpg-mssr.c:156:21: warning:
        no previous prototype for ‘rcar_r8a779a0_cpg_clk_register’ [-Wmissing-prototypes]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/renesas/r8a779a0-cpg-mssr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/renesas/r8a779a0-cpg-mssr.c b/drivers/clk/renesas/r8a779a0-cpg-mssr.c
index 17ebbac7ddfb..7e25b3b8945b 100644
--- a/drivers/clk/renesas/r8a779a0-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a779a0-cpg-mssr.c
@@ -153,7 +153,7 @@ static const struct rcar_r8a779a0_cpg_pll_config *cpg_pll_config __initdata;
 static unsigned int cpg_clk_extalr __initdata;
 static u32 cpg_mode __initdata;
 
-struct clk * __init rcar_r8a779a0_cpg_clk_register(struct device *dev,
+static struct clk * __init rcar_r8a779a0_cpg_clk_register(struct device *dev,
 	const struct cpg_core_clk *core, const struct cpg_mssr_info *info,
 	struct clk **clks, void __iomem *base,
 	struct raw_notifier_head *notifiers)
-- 
2.25.1


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

* [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
                   ` (4 preceding siblings ...)
  2020-11-03 16:24 ` [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-04 10:52   ` Geert Uytterhoeven
  2020-11-03 16:24 ` [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused Krzysztof Kozlowski
  2020-12-10 20:24 ` [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Stephen Boyd
  7 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The struct cpg_mssr_priv missed proper formatting:

    drivers/clk/renesas/renesas-cpg-mssr.c:142: warning:
        cannot understand function prototype: 'struct cpg_mssr_priv '

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/renesas/renesas-cpg-mssr.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/renesas/renesas-cpg-mssr.c b/drivers/clk/renesas/renesas-cpg-mssr.c
index 94db88370337..1c3215dc4877 100644
--- a/drivers/clk/renesas/renesas-cpg-mssr.c
+++ b/drivers/clk/renesas/renesas-cpg-mssr.c
@@ -119,7 +119,8 @@ static const u16 srstclr_for_v3u[] = {
 };
 
 /**
- * Clock Pulse Generator / Module Standby and Software Reset Private Data
+ * struct cpg_mssr_priv - Clock Pulse Generator / Module Standby
+ *                        and Software Reset Private Data
  *
  * @rcdev: Optional reset controller entity
  * @dev: CPG/MSSR device
-- 
2.25.1


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

* [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
                   ` (5 preceding siblings ...)
  2020-11-03 16:24 ` [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv Krzysztof Kozlowski
@ 2020-11-03 16:24 ` Krzysztof Kozlowski
  2020-11-04  9:00   ` Sudeep Holla
  2020-12-10 20:24   ` Stephen Boyd
  2020-12-10 20:24 ` [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Stephen Boyd
  7 siblings, 2 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2020-11-03 16:24 UTC (permalink / raw)
  To: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, linux-kernel, linux-arm-kernel,
	linux-renesas-soc
  Cc: Krzysztof Kozlowski

The scpi_clk_match (struct of_device_id) is referenced only with
CONFIG_OF builds thus mark it as __maybe_unused:

    drivers/clk/clk-scpi.c:132:34: warning:
        ‘scpi_clk_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/clk/clk-scpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/clk-scpi.c b/drivers/clk/clk-scpi.c
index 5a9b140dd8c8..a39af7616b13 100644
--- a/drivers/clk/clk-scpi.c
+++ b/drivers/clk/clk-scpi.c
@@ -129,7 +129,7 @@ static const struct clk_ops scpi_dvfs_ops = {
 	.set_rate = scpi_dvfs_set_rate,
 };
 
-static const struct of_device_id scpi_clk_match[] = {
+static const struct of_device_id scpi_clk_match[] __maybe_unused = {
 	{ .compatible = "arm,scpi-dvfs-clocks", .data = &scpi_dvfs_ops, },
 	{ .compatible = "arm,scpi-variable-clocks", .data = &scpi_clk_ops, },
 	{}
-- 
2.25.1


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

* Re: [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused
  2020-11-03 16:24 ` [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused Krzysztof Kozlowski
@ 2020-11-04  9:00   ` Sudeep Holla
  2020-12-10 20:24   ` Stephen Boyd
  1 sibling, 0 replies; 21+ messages in thread
From: Sudeep Holla @ 2020-11-04  9:00 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Geert Uytterhoeven, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	linux-kernel, linux-arm-kernel, linux-renesas-soc

On Tue, Nov 03, 2020 at 05:24:35PM +0100, Krzysztof Kozlowski wrote:
> The scpi_clk_match (struct of_device_id) is referenced only with
> CONFIG_OF builds thus mark it as __maybe_unused:
>
>     drivers/clk/clk-scpi.c:132:34: warning:
>         ‘scpi_clk_match’ defined but not used [-Wunused-const-variable=]
>

Acked-by: Sudeep Holla <sudeep.holla@arm.com>

--
Regards,
Sudeep

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

* Re: [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv
  2020-11-03 16:24 ` [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv Krzysztof Kozlowski
@ 2020-11-04 10:52   ` Geert Uytterhoeven
  0 siblings, 0 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-11-04 10:52 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	Linux Kernel Mailing List, Linux ARM, Linux-Renesas

On Tue, Nov 3, 2020 at 5:25 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> The struct cpg_mssr_priv missed proper formatting:
>
>     drivers/clk/renesas/renesas-cpg-mssr.c:142: warning:
>         cannot understand function prototype: 'struct cpg_mssr_priv '
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-clk-for-v5.11.

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

* Re: [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function
  2020-11-03 16:24 ` [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function Krzysztof Kozlowski
@ 2020-11-04 10:53   ` Geert Uytterhoeven
  0 siblings, 0 replies; 21+ messages in thread
From: Geert Uytterhoeven @ 2020-11-04 10:53 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	NXP Linux Team, Geert Uytterhoeven, Peng Fan, Abel Vesa,
	Anson Huang, linux-clk, Linux Kernel Mailing List, Linux ARM,
	Linux-Renesas

Hi Krzysztof,

On Tue, Nov 3, 2020 at 5:25 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> The function rcar_r8a779a0_cpg_clk_register() is not used outside of the
> unit so it can be made static to fix compilation warning:
>
>     drivers/clk/renesas/r8a779a0-cpg-mssr.c:156:21: warning:
>         no previous prototype for ‘rcar_r8a779a0_cpg_clk_register’ [-Wmissing-prototypes]
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Thanks for your patch!
A fix for this is already queued in renesas-clk-for-v5.11:
https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=renesas-clk-for-v5.11&id=c3f207f6d23d01c0fa817fcd6118bbbc509abe39

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

* Re: [PATCH 2/8] clk: imx8mm: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
@ 2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-11-05  1:11 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski

Quoting Krzysztof Kozlowski (2020-11-03 08:24:29)
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mm.c:641:34: warning:
>         ‘imx8mm_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Acked-by: Stephen Boyd <sboyd@kernel.org>

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

* Re: [PATCH 3/8] clk: imx8mn: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 3/8] clk: imx8mn: " Krzysztof Kozlowski
@ 2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-11-05  1:11 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski

Quoting Krzysztof Kozlowski (2020-11-03 08:24:30)
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mn.c:592:34: warning:
>         ‘imx8mn_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Acked-by: Stephen Boyd <sboyd@kernel.org>

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

* Re: [PATCH 4/8] clk: imx8mp: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 4/8] clk: imx8mp: " Krzysztof Kozlowski
@ 2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-11-05  1:11 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski, kernel test robot

Quoting Krzysztof Kozlowski (2020-11-03 08:24:31)
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mp.c:751:34: warning:
>         ‘imx8mp_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Acked-by: Stephen Boyd <sboyd@kernel.org>

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

* Re: [PATCH 5/8] clk: imx8mq: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 5/8] clk: imx8mq: " Krzysztof Kozlowski
@ 2020-11-05  1:11   ` Stephen Boyd
  2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-11-05  1:11 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski

Quoting Krzysztof Kozlowski (2020-11-03 08:24:32)
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mq.c:626:34: warning:
>         ‘imx8mq_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Acked-by: Stephen Boyd <sboyd@kernel.org>

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

* Re: [PATCH 2/8] clk: imx8mm: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
@ 2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2020-11-10  1:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Geert Uytterhoeven, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	linux-kernel, linux-arm-kernel, linux-renesas-soc

On Tue, Nov 03, 2020 at 05:24:29PM +0100, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mm.c:641:34: warning:
>         ‘imx8mm_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Applied, thanks.

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

* Re: [PATCH 3/8] clk: imx8mn: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 3/8] clk: imx8mn: " Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
@ 2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2020-11-10  1:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Geert Uytterhoeven, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	linux-kernel, linux-arm-kernel, linux-renesas-soc

On Tue, Nov 03, 2020 at 05:24:30PM +0100, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mn.c:592:34: warning:
>         ‘imx8mn_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Applied, thanks.

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

* Re: [PATCH 4/8] clk: imx8mp: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 4/8] clk: imx8mp: " Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
@ 2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2020-11-10  1:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Geert Uytterhoeven, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	linux-kernel, linux-arm-kernel, linux-renesas-soc,
	kernel test robot

On Tue, Nov 03, 2020 at 05:24:31PM +0100, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mp.c:751:34: warning:
>         ‘imx8mp_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Applied, thanks.

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

* Re: [PATCH 5/8] clk: imx8mq: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 ` [PATCH 5/8] clk: imx8mq: " Krzysztof Kozlowski
  2020-11-05  1:11   ` Stephen Boyd
@ 2020-11-10  1:54   ` Shawn Guo
  1 sibling, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2020-11-10  1:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Michael Turquette, Stephen Boyd, Sudeep Holla, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
	Geert Uytterhoeven, Peng Fan, Abel Vesa, Anson Huang, linux-clk,
	linux-kernel, linux-arm-kernel, linux-renesas-soc

On Tue, Nov 03, 2020 at 05:24:32PM +0100, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/imx/clk-imx8mq.c:626:34: warning:
>         ‘imx8mq_clk_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Applied, thanks.

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

* Re: [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table
  2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
                   ` (6 preceding siblings ...)
  2020-11-03 16:24 ` [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused Krzysztof Kozlowski
@ 2020-12-10 20:24 ` Stephen Boyd
  7 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-12-10 20:24 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski

Quoting Krzysztof Kozlowski (2020-11-03 08:24:28)
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it might be not relevant here).  This
> fixes compile warning (!CONFIG_OF && !CONFIG_MODULES):
> 
>     drivers/clk/clk-pwm.c:139:34: warning:
>         ‘clk_pwm_dt_ids’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Applied to clk-next

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

* Re: [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused
  2020-11-03 16:24 ` [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused Krzysztof Kozlowski
  2020-11-04  9:00   ` Sudeep Holla
@ 2020-12-10 20:24   ` Stephen Boyd
  1 sibling, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-12-10 20:24 UTC (permalink / raw)
  To: Abel Vesa, Anson Huang, Fabio Estevam, Geert Uytterhoeven,
	Krzysztof Kozlowski, Michael Turquette, NXP Linux Team, Peng Fan,
	Pengutronix Kernel Team, Sascha Hauer, Shawn Guo, Sudeep Holla,
	linux-arm-kernel, linux-clk, linux-kernel, linux-renesas-soc
  Cc: Krzysztof Kozlowski

Quoting Krzysztof Kozlowski (2020-11-03 08:24:35)
> The scpi_clk_match (struct of_device_id) is referenced only with
> CONFIG_OF builds thus mark it as __maybe_unused:
> 
>     drivers/clk/clk-scpi.c:132:34: warning:
>         ‘scpi_clk_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Applied to clk-next

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

end of thread, other threads:[~2020-12-10 20:25 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-03 16:24 [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Krzysztof Kozlowski
2020-11-03 16:24 ` [PATCH 2/8] clk: imx8mm: " Krzysztof Kozlowski
2020-11-05  1:11   ` Stephen Boyd
2020-11-10  1:54   ` Shawn Guo
2020-11-03 16:24 ` [PATCH 3/8] clk: imx8mn: " Krzysztof Kozlowski
2020-11-05  1:11   ` Stephen Boyd
2020-11-10  1:54   ` Shawn Guo
2020-11-03 16:24 ` [PATCH 4/8] clk: imx8mp: " Krzysztof Kozlowski
2020-11-05  1:11   ` Stephen Boyd
2020-11-10  1:54   ` Shawn Guo
2020-11-03 16:24 ` [PATCH 5/8] clk: imx8mq: " Krzysztof Kozlowski
2020-11-05  1:11   ` Stephen Boyd
2020-11-10  1:54   ` Shawn Guo
2020-11-03 16:24 ` [PATCH 6/8] clk: renesas: r8a779a0-cpg-mssr: add static to local function Krzysztof Kozlowski
2020-11-04 10:53   ` Geert Uytterhoeven
2020-11-03 16:24 ` [PATCH 7/8] clk: renesas: renesas-cpg-mssr: fix kerneldoc of cpg_mssr_priv Krzysztof Kozlowski
2020-11-04 10:52   ` Geert Uytterhoeven
2020-11-03 16:24 ` [PATCH 8/8] clk: scpi: mark scpi_clk_match as maybe unused Krzysztof Kozlowski
2020-11-04  9:00   ` Sudeep Holla
2020-12-10 20:24   ` Stephen Boyd
2020-12-10 20:24 ` [PATCH 1/8] clk: pwm: drop of_match_ptr from of_device_id table Stephen Boyd

This is a public inbox, see mirroring instructions
on how to clone and mirror all data and code used for this inbox