Skip to contentSkip to conversation

Cleanup sched_clock.h users

PATCH In review Stephen Boyd 2 participants 5 messages Jul 18, 2013

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

cover Cleanup sched_clock.h users

This patchset cleans up the last few stragglers using the sched_clock.h header file in asm and removes the asm header.

A new one appeared in drivers/clocksource so I'm not sure who wants to take that one.

Stephen Boyd (3):
clocksource: orion: Use linux/sched_clock.h
ARM: clps711x: Use linux/sched_clock.h
ARM: Remove temporary sched_clock.h header

 arch/arm/include/asm/sched_clock.h | 4 ----
 arch/arm/mach-clps711x/common.c    | 2 +-
 drivers/clocksource/time-orion.c   | 2 +-
 3 files changed, 2 insertions(+), 6 deletions(-)
 delete mode 100644 arch/arm/include/asm/sched_clock.h
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
#1 lore raw

PATCH 1/3 clocksource: orion: Use linux/sched_clock.h

The sched_clock.h include is under include/linux now.

Cc Sebastian Hesselbarth <[email protected]>
---
 drivers/clocksource/time-orion.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clocksource/time-orion.c +1 -1
index ecbeb68..9c7f018 100644
@@ -19,7 +19,7 @@
 #include <linux/of_address.h>
 #include <linux/of_irq.h>
 #include <linux/spinlock.h>
-#include <asm/sched_clock.h>
+#include <linux/sched_clock.h>
 
 #define TIMER_CTRL		0x00
 #define  TIMER0_EN		BIT(0)
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
#2 lore raw

PATCH 2/3 ARM: clps711x: Use linux/sched_clock.h

In reply to Stephen Boyd

The sched_clock.h include is under include/linux now.

Cc Alexander Shiyan <[email protected]>
---
 arch/arm/mach-clps711x/common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-clps711x/common.c +1 -1
index 4ca2f3c..134641d 100644
@@ -29,12 +29,12 @@
 #include <linux/clockchips.h>
 #include <linux/clocksource.h>
 #include <linux/clk-provider.h>
+#include <linux/sched_clock.h>
 
 #include <asm/exception.h>
 #include <asm/mach/irq.h>
 #include <asm/mach/map.h>
 #include <asm/mach/time.h>
-#include <asm/sched_clock.h>
 #include <asm/system_misc.h>
 
 #include <mach/hardware.h>
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
#3 lore raw

PATCH 3/3 ARM: Remove temporary sched_clock.h header

In reply to Stephen Boyd

This header file is no longer needed now that the ARM sched_clock framework is generic and all users have moved to linux/sched_clock.h instead of asm/sched_clock.h. Remove it.

---
 arch/arm/include/asm/sched_clock.h | 4 ----
 1 file changed, 4 deletions(-)
 delete mode 100644 arch/arm/include/asm/sched_clock.h
arch/arm/include/asm/sched_clock.h deleted +0 -4
deleted file mode 100644
index 2389b71..0000000
@@ -1,4 +0,0 @@
-/* You shouldn't include this file. Use linux/sched_clock.h instead.
- * Temporary file until all asm/sched_clock.h users are gone
- */
-#include <linux/sched_clock.h>
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
#4 lore raw
In reply to Stephen Boyd
Quoted 5 lines · Stephen Boyd

This patchset cleans up the last few stragglers using the sched_clock.h header file in asm and removes the asm header.

A new one appeared in drivers/clocksource so I'm not sure who wants to take that one.

Thanks for doing this!

I'll queue the drivers/clocksource one on its own in a separate branch, that way I'll merge it into my own 3.12 queue, and Arnd/Olof can also merge just that change it in before they drop the asm/sched_clock.h header.

I just pushed it out here:

git://git.linaro.org/people/jstultz/linux.git fortglx/3.12/sched-clock-cleanups

thanks
-john