* [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 related [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 related [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 related [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 related [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 related [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 related [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 related [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 related [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
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).