public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clocksource: sirf: remove unused variable
@ 2015-01-13 14:22 Arnd Bergmann
  2015-01-13 14:27 ` Daniel Lezcano
  0 siblings, 1 reply; 12+ messages in thread
From: Arnd Bergmann @ 2015-01-13 14:22 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: linux-arm-kernel, linux-kernel, Yanchang Li, Barry Song,
	Thomas Gleixner

A recent rework of the driver left an obviously unused variable
around, and now the compiler complains:

clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init':
clocksource/timer-marco.c:260:6: warning: unused variable 'timer_div' [-Wunused-variable]

Remove it.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: ef89af1f4380b92 ("clocksource: sirf: Remove hard-coded clock rate")

diff --git a/drivers/clocksource/timer-marco.c b/drivers/clocksource/timer-marco.c
index 361a789d4bee..3ddb81f7ee66 100644
--- a/drivers/clocksource/timer-marco.c
+++ b/drivers/clocksource/timer-marco.c
@@ -257,7 +257,6 @@ static void __init sirfsoc_clockevent_init(void)
 /* initialize the kernel jiffy timer source */
 static void __init sirfsoc_marco_timer_init(struct device_node *np)
 {
-	u32 timer_div;
 	struct clk *clk;
 
 	clk = of_clk_get(np, 0);


^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] clocksource: sirf: Remove unused variable
@ 2014-12-23 21:33 Fabio Estevam
  2015-01-11 14:45 ` Barry Song
  0 siblings, 1 reply; 12+ messages in thread
From: Fabio Estevam @ 2014-12-23 21:33 UTC (permalink / raw)
  To: daniel.lezcano; +Cc: baohua, linux-kernel, Fabio Estevam

Remove 'timer_div' variable to fix the following build warning:

drivers/clocksource/timer-marco.c: In function 'sirfsoc_marco_timer_init':
drivers/clocksource/timer-marco.c:260:6: warning: unused variable 'timer_div' [-Wunused-variable]

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 drivers/clocksource/timer-marco.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/clocksource/timer-marco.c b/drivers/clocksource/timer-marco.c
index 361a789..3ddb81f 100644
--- a/drivers/clocksource/timer-marco.c
+++ b/drivers/clocksource/timer-marco.c
@@ -257,7 +257,6 @@ static void __init sirfsoc_clockevent_init(void)
 /* initialize the kernel jiffy timer source */
 static void __init sirfsoc_marco_timer_init(struct device_node *np)
 {
-	u32 timer_div;
 	struct clk *clk;
 
 	clk = of_clk_get(np, 0);
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread
* [PATCH] clocksource: sirf: Remove unused variable
@ 2014-12-15 15:59 Thierry Reding
  2014-12-15 16:17 ` Daniel Lezcano
  0 siblings, 1 reply; 12+ messages in thread
From: Thierry Reding @ 2014-12-15 15:59 UTC (permalink / raw)
  To: Daniel Lezcano, Thomas Gleixner
  Cc: Barry Song, linux-arm-kernel, linux-kernel

From: Thierry Reding <treding@nvidia.com>

Commit ef89af1f4380 ("clocksource: sirf: Remove hard-coded clock rate")
removes all uses of the timer_div variable. Since the variable is no
longer used it should be removed.

Signed-off-by: Thierry Reding <treding@nvidia.com>
---
 drivers/clocksource/timer-marco.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/clocksource/timer-marco.c b/drivers/clocksource/timer-marco.c
index 361a789d4bee..3ddb81f7ee66 100644
--- a/drivers/clocksource/timer-marco.c
+++ b/drivers/clocksource/timer-marco.c
@@ -257,7 +257,6 @@ static void __init sirfsoc_clockevent_init(void)
 /* initialize the kernel jiffy timer source */
 static void __init sirfsoc_marco_timer_init(struct device_node *np)
 {
-	u32 timer_div;
 	struct clk *clk;
 
 	clk = of_clk_get(np, 0);
-- 
2.1.3


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

end of thread, other threads:[~2015-01-13 16:23 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-13 14:22 [PATCH] clocksource: sirf: remove unused variable Arnd Bergmann
2015-01-13 14:27 ` Daniel Lezcano
2015-01-13 15:24   ` Arnd Bergmann
2015-01-13 15:39     ` Daniel Lezcano
2015-01-13 15:24   ` Barry Song
2015-01-13 16:22     ` Arnd Bergmann
  -- strict thread matches above, loose matches on Subject: below --
2014-12-23 21:33 [PATCH] clocksource: sirf: Remove " Fabio Estevam
2015-01-11 14:45 ` Barry Song
2014-12-15 15:59 Thierry Reding
2014-12-15 16:17 ` Daniel Lezcano
2015-01-11 15:58   ` Barry Song
2015-01-11 20:51     ` Daniel Lezcano

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