Skip to contentSkip to conversation

clocksource: sirf: Remove unused variable

PATCH Applied Thierry Reding 3 participants 4 messages Dec 15, 2014 – Jan 11, 2015 Clocksource, clockevent drivers

lore
Share & tools
lore

Thierry Reding

author
#0 lore raw

PATCH 1/1 clocksource: sirf: Remove unused variable

From: Thierry Reding <[email protected]>

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 <[email protected]>
---
 drivers/clocksource/timer-marco.c | 1 -
 1 file changed, 1 deletion(-)
drivers/clocksource/timer-marco.c +0 -1
index 361a789d4bee..3ddb81f7ee66 100644
@@ -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);
Signature · 1 line
2.1.3
Applied
#1 lore raw
Quoted 8 lines · Thierry Reding

From: Thierry Reding <[email protected]>

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 <[email protected]>

---

Applied to my tree. Thanks!

-- Daniel

Signature · 5 lines
  <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
Acked-by Arnd Bergmann Acked-by Daniel Lezcano
#2 lore raw

2014-12-16 0:17 GMT+08:00 Daniel Lezcano <[email protected]>:

Quoted 8 lines · Thierry Reding
Quoted 7 lines

From: Thierry Reding <[email protected]>

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.

Quoted 5 lines
Quoted 2 lines
Signed-off-by Thierry Reding <[email protected]>

---

Applied to my tree. Thanks!

Daniel,

i didn't find this commit in any tree in kernel.org. am i missing something?

so i manually rebased "clocksource: marco: rename marco to atlas7" against this commit.

can you help to apply the attached patch to your tree? i guess pulling by arm-soc will cause conflicts.

Quoted 3 lines

-- Daniel

-barry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-clocksource-marco-rename-marco-to-atlas7.patch
Type: text/x-patch
Size: 20190 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150111/d2680e92/attachment-0001.bin>

#3 lore raw
Quoted 13 lines · Barry Song

2014-12-16 0:17 GMT+08:00 Daniel Lezcano <[email protected]>:

Quoted 12 lines · Thierry Reding
Quoted 9 lines

From: Thierry Reding <[email protected]>

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 <[email protected]>

---

Quoted 5 lines
Quoted 1 line

Applied to my tree. Thanks!

Daniel,

i didn't find this commit in any tree in kernel.org. am i missing something?

Nope, I did probably something wrong.

Quoted 5 lines

so i manually rebased "clocksource: marco: rename marco to atlas7" against this commit.

can you help to apply the attached patch to your tree? i guess pulling by arm-soc will cause conflicts.

Ok, done. Both patches are in my tree [1].

Thanks
-- Daniel

[1]
https://git.linaro.org/people/daniel.lezcano/linux.git/shortlog/refs/heads/clockevents/3.20

Signature · 5 lines
  <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