* [PATCH] clk: ti: dra7: remove two unused symbols
@ 2020-04-17 7:35 Jason Yan
2020-04-17 15:33 ` Tony Lindgren
2020-05-27 3:24 ` Stephen Boyd
0 siblings, 2 replies; 5+ messages in thread
From: Jason Yan @ 2020-04-17 7:35 UTC (permalink / raw)
To: t-kristo, mturquette, sboyd, tony, linux-omap, linux-clk, linux-kernel
Cc: Jason Yan, Hulk Robot
Fix the following gcc warning:
drivers/clk/ti/clk-7xx.c:320:43: warning: ‘dra7_gpu_sys_clk_data’
defined but not used [-Wunused-const-variable=]
static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
__initconst = {
^~~~~~~~~~~~~~~~~~~~~
drivers/clk/ti/clk-7xx.c:315:27: warning: ‘dra7_gpu_sys_clk_parents’
defined but not used [-Wunused-const-variable=]
static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
^~~~~~~~~~~~~~~~~~~~~~~~
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Jason Yan <yanaijie@huawei.com>
---
drivers/clk/ti/clk-7xx.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/clk/ti/clk-7xx.c b/drivers/clk/ti/clk-7xx.c
index 14b645093107..941a67dae22c 100644
--- a/drivers/clk/ti/clk-7xx.c
+++ b/drivers/clk/ti/clk-7xx.c
@@ -312,15 +312,6 @@ static const char * const dra7_gpu_hyd_mux_parents[] __initconst = {
NULL,
};
-static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
- "sys_clkin",
- NULL,
-};
-
-static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data __initconst = {
- .max_div = 2,
-};
-
static const struct omap_clkctrl_bit_data dra7_gpu_core_bit_data[] __initconst = {
{ 24, TI_CLK_MUX, dra7_gpu_core_mux_parents, NULL, },
{ 26, TI_CLK_MUX, dra7_gpu_hyd_mux_parents, NULL, },
--
2.21.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] clk: ti: dra7: remove two unused symbols
2020-04-17 7:35 [PATCH] clk: ti: dra7: remove two unused symbols Jason Yan
@ 2020-04-17 15:33 ` Tony Lindgren
2020-05-14 19:58 ` Stephen Boyd
2020-05-27 3:24 ` Stephen Boyd
1 sibling, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2020-04-17 15:33 UTC (permalink / raw)
To: Tero Kristo
Cc: Jason Yan, mturquette, sboyd, linux-omap, linux-clk,
linux-kernel, Hulk Robot
Tero,
* Jason Yan <yanaijie@huawei.com> [200417 07:09]:
> Fix the following gcc warning:
>
> drivers/clk/ti/clk-7xx.c:320:43: warning: ‘dra7_gpu_sys_clk_data’
> defined but not used [-Wunused-const-variable=]
> static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
> __initconst = {
> ^~~~~~~~~~~~~~~~~~~~~
> drivers/clk/ti/clk-7xx.c:315:27: warning: ‘dra7_gpu_sys_clk_parents’
> defined but not used [-Wunused-const-variable=]
> static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
> ^~~~~~~~~~~~~~~~~~~~~~~~
Looks like this started happening with commit 957ad44ff5f2
("clk: ti: add clkctrl data dra7 sgx") as I did it based on
omap5. But the clocks seem slightly different, care to check
sgx clocking against the TRM in case I also messed up something
else?
Regards,
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] clk: ti: dra7: remove two unused symbols
2020-04-17 15:33 ` Tony Lindgren
@ 2020-05-14 19:58 ` Stephen Boyd
2020-05-14 20:02 ` Tony Lindgren
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Boyd @ 2020-05-14 19:58 UTC (permalink / raw)
To: Tero Kristo, Tony Lindgren
Cc: Jason Yan, mturquette, linux-omap, linux-clk, linux-kernel, Hulk Robot
Quoting Tony Lindgren (2020-04-17 08:33:42)
> Tero,
>
> * Jason Yan <yanaijie@huawei.com> [200417 07:09]:
> > Fix the following gcc warning:
> >
> > drivers/clk/ti/clk-7xx.c:320:43: warning: \u2018dra7_gpu_sys_clk_data\u2019
> > defined but not used [-Wunused-const-variable=]
> > static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
> > __initconst = {
> > ^~~~~~~~~~~~~~~~~~~~~
> > drivers/clk/ti/clk-7xx.c:315:27: warning: \u2018dra7_gpu_sys_clk_parents\u2019
> > defined but not used [-Wunused-const-variable=]
> > static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
> > ^~~~~~~~~~~~~~~~~~~~~~~~
>
> Looks like this started happening with commit 957ad44ff5f2
> ("clk: ti: add clkctrl data dra7 sgx") as I did it based on
> omap5. But the clocks seem slightly different, care to check
> sgx clocking against the TRM in case I also messed up something
> else?
Is that an ack?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] clk: ti: dra7: remove two unused symbols
2020-05-14 19:58 ` Stephen Boyd
@ 2020-05-14 20:02 ` Tony Lindgren
0 siblings, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2020-05-14 20:02 UTC (permalink / raw)
To: Stephen Boyd
Cc: Tero Kristo, Jason Yan, mturquette, linux-omap, linux-clk,
linux-kernel, Hulk Robot
* Stephen Boyd <sboyd@kernel.org> [200514 19:59]:
> Quoting Tony Lindgren (2020-04-17 08:33:42)
> > Tero,
> >
> > * Jason Yan <yanaijie@huawei.com> [200417 07:09]:
> > > Fix the following gcc warning:
> > >
> > > drivers/clk/ti/clk-7xx.c:320:43: warning: \u2018dra7_gpu_sys_clk_data\u2019
> > > defined but not used [-Wunused-const-variable=]
> > > static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
> > > __initconst = {
> > > ^~~~~~~~~~~~~~~~~~~~~
> > > drivers/clk/ti/clk-7xx.c:315:27: warning: \u2018dra7_gpu_sys_clk_parents\u2019
> > > defined but not used [-Wunused-const-variable=]
> > > static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
> > > ^~~~~~~~~~~~~~~~~~~~~~~~
> >
> > Looks like this started happening with commit 957ad44ff5f2
> > ("clk: ti: add clkctrl data dra7 sgx") as I did it based on
> > omap5. But the clocks seem slightly different, care to check
> > sgx clocking against the TRM in case I also messed up something
> > else?
>
> Is that an ack?
Acked-by: Tony Lindgren <tony@atomide.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] clk: ti: dra7: remove two unused symbols
2020-04-17 7:35 [PATCH] clk: ti: dra7: remove two unused symbols Jason Yan
2020-04-17 15:33 ` Tony Lindgren
@ 2020-05-27 3:24 ` Stephen Boyd
1 sibling, 0 replies; 5+ messages in thread
From: Stephen Boyd @ 2020-05-27 3:24 UTC (permalink / raw)
To: Jason Yan, linux-clk, linux-kernel, linux-omap, mturquette,
t-kristo, tony
Cc: Jason Yan, Hulk Robot
Quoting Jason Yan (2020-04-17 00:35:23)
> Fix the following gcc warning:
>
> drivers/clk/ti/clk-7xx.c:320:43: warning: \u2018dra7_gpu_sys_clk_data\u2019
> defined but not used [-Wunused-const-variable=]
> static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
> __initconst = {
> ^~~~~~~~~~~~~~~~~~~~~
> drivers/clk/ti/clk-7xx.c:315:27: warning: \u2018dra7_gpu_sys_clk_parents\u2019
> defined but not used [-Wunused-const-variable=]
> static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
> ^~~~~~~~~~~~~~~~~~~~~~~~
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Jason Yan <yanaijie@huawei.com>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-05-27 3:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-17 7:35 [PATCH] clk: ti: dra7: remove two unused symbols Jason Yan
2020-04-17 15:33 ` Tony Lindgren
2020-05-14 19:58 ` Stephen Boyd
2020-05-14 20:02 ` Tony Lindgren
2020-05-27 3:24 ` 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).