From: Sudeep KarkadaNagesha <Sudeep.KarkadaNagesha@arm.com> To: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: Sudeep.KarkadaNagesha@arm.com, Mark Rutland <mark.rutland@arm.com>, Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>, Russell King <linux@arm.linux.org.uk>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Sudeep KarkadaNagesha <sudeep.karkadanagesha@arm.com> Subject: [PATCH v2 2/3] ARM: arch_timer: add macros for bits in control register Date: Mon, 22 Jul 2013 12:21:21 +0100 [thread overview] Message-ID: <1374492082-13686-3-git-send-email-Sudeep.KarkadaNagesha@arm.com> (raw) In-Reply-To: <1374492082-13686-1-git-send-email-Sudeep.KarkadaNagesha@arm.com> From: Sudeep KarkadaNagesha <sudeep.karkadanagesha@arm.com> Add macros to describe the bitfields in the ARM architected timer control register to make code easy to understand. Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Reviewed-by: Will Deacon <will.deacon@arm.com> Signed-off-by: Sudeep KarkadaNagesha <sudeep.karkadanagesha@arm.com> --- arch/arm/include/asm/arch_timer.h | 9 +++++++-- arch/arm64/include/asm/arch_timer.h | 10 ++++++++-- include/clocksource/arm_arch_timer.h | 8 ++++++++ 3 files changed, 23 insertions(+), 4 deletions(-) diff --git a/arch/arm/include/asm/arch_timer.h b/arch/arm/include/asm/arch_timer.h index c3d9ef7..26c65ae 100644 --- a/arch/arm/include/asm/arch_timer.h +++ b/arch/arm/include/asm/arch_timer.h @@ -96,8 +96,13 @@ static inline void arch_counter_set_user_access(int divider) asm volatile("mrc p15, 0, %0, c14, c1, 0" : "=r" (cntkctl)); /* disable user access to everything */ - cntkctl &= ~((3 << 8) | (0xf << 4) | (3 << 0)); - cntkctl |= (divider << 4) | (1 << 2); + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN + | ARCH_TIMER_USR_VT_ACCESS_EN + | ARCH_TIMER_EVT_TRIGGER_MASK + | ARCH_TIMER_USR_VCT_ACCESS_EN + | ARCH_TIMER_USR_PCT_ACCESS_EN); + cntkctl |= (divider << ARCH_TIMER_EVT_TRIGGER_SHIFT) + | ARCH_TIMER_VIRT_EVT_EN; asm volatile("mcr p15, 0, %0, c14, c1, 0" : : "r" (cntkctl)); } diff --git a/arch/arm64/include/asm/arch_timer.h b/arch/arm64/include/asm/arch_timer.h index 8c1e42f..3494102 100644 --- a/arch/arm64/include/asm/arch_timer.h +++ b/arch/arm64/include/asm/arch_timer.h @@ -103,10 +103,16 @@ static inline void arch_counter_set_user_access(int divider) /* Disable user access to the timers and the physical counter. */ asm volatile("mrs %0, cntkctl_el1" : "=r" (cntkctl)); - cntkctl &= ~((3 << 8) | (0xf << 4) | (1 << 0)); + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN + | ARCH_TIMER_USR_VT_ACCESS_EN + | ARCH_TIMER_EVT_TRIGGER_MASK + | ARCH_TIMER_USR_PCT_ACCESS_EN) /* Enable user access to the virtual counter and frequency. */ - cntkctl |= (divider << 4) | (1 << 2) | (1 << 1); + cntkctl |= (divider << ARCH_TIMER_EVT_TRIGGER_SHIFT) + | ARCH_TIMER_VIRT_EVT_EN + | ARCH_TIMER_USR_VCT_ACCESS_EN; + asm volatile("msr cntkctl_el1, %0" : : "r" (cntkctl)); } diff --git a/include/clocksource/arm_arch_timer.h b/include/clocksource/arm_arch_timer.h index 24dc140..c4d0fc4 100644 --- a/include/clocksource/arm_arch_timer.h +++ b/include/clocksource/arm_arch_timer.h @@ -29,6 +29,14 @@ #define ARCH_TIMER_PHYS_ACCESS 0 #define ARCH_TIMER_VIRT_ACCESS 1 +#define ARCH_TIMER_USR_PCT_ACCESS_EN (1 << 0) /* physical counter */ +#define ARCH_TIMER_USR_VCT_ACCESS_EN (1 << 1) /* virtual counter */ +#define ARCH_TIMER_VIRT_EVT_EN (1 << 2) +#define ARCH_TIMER_EVT_TRIGGER_SHIFT (4) +#define ARCH_TIMER_EVT_TRIGGER_MASK (0xF << ARCH_TIMER_EVT_TRIGGER_SHIFT) +#define ARCH_TIMER_USR_VT_ACCESS_EN (1 << 8) /* virtual timer registers */ +#define ARCH_TIMER_USR_PT_ACCESS_EN (1 << 9) /* physical timer registers */ + #define ARCH_TIMER_EVT_STREAM_FREQ 10000 /* 100us */ #ifdef CONFIG_ARM_ARCH_TIMER -- 1.8.1.2
WARNING: multiple messages have this Message-ID (diff)
From: Sudeep.KarkadaNagesha@arm.com (Sudeep KarkadaNagesha) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 2/3] ARM: arch_timer: add macros for bits in control register Date: Mon, 22 Jul 2013 12:21:21 +0100 [thread overview] Message-ID: <1374492082-13686-3-git-send-email-Sudeep.KarkadaNagesha@arm.com> (raw) In-Reply-To: <1374492082-13686-1-git-send-email-Sudeep.KarkadaNagesha@arm.com> From: Sudeep KarkadaNagesha <sudeep.karkadanagesha@arm.com> Add macros to describe the bitfields in the ARM architected timer control register to make code easy to understand. Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Reviewed-by: Will Deacon <will.deacon@arm.com> Signed-off-by: Sudeep KarkadaNagesha <sudeep.karkadanagesha@arm.com> --- arch/arm/include/asm/arch_timer.h | 9 +++++++-- arch/arm64/include/asm/arch_timer.h | 10 ++++++++-- include/clocksource/arm_arch_timer.h | 8 ++++++++ 3 files changed, 23 insertions(+), 4 deletions(-) diff --git a/arch/arm/include/asm/arch_timer.h b/arch/arm/include/asm/arch_timer.h index c3d9ef7..26c65ae 100644 --- a/arch/arm/include/asm/arch_timer.h +++ b/arch/arm/include/asm/arch_timer.h @@ -96,8 +96,13 @@ static inline void arch_counter_set_user_access(int divider) asm volatile("mrc p15, 0, %0, c14, c1, 0" : "=r" (cntkctl)); /* disable user access to everything */ - cntkctl &= ~((3 << 8) | (0xf << 4) | (3 << 0)); - cntkctl |= (divider << 4) | (1 << 2); + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN + | ARCH_TIMER_USR_VT_ACCESS_EN + | ARCH_TIMER_EVT_TRIGGER_MASK + | ARCH_TIMER_USR_VCT_ACCESS_EN + | ARCH_TIMER_USR_PCT_ACCESS_EN); + cntkctl |= (divider << ARCH_TIMER_EVT_TRIGGER_SHIFT) + | ARCH_TIMER_VIRT_EVT_EN; asm volatile("mcr p15, 0, %0, c14, c1, 0" : : "r" (cntkctl)); } diff --git a/arch/arm64/include/asm/arch_timer.h b/arch/arm64/include/asm/arch_timer.h index 8c1e42f..3494102 100644 --- a/arch/arm64/include/asm/arch_timer.h +++ b/arch/arm64/include/asm/arch_timer.h @@ -103,10 +103,16 @@ static inline void arch_counter_set_user_access(int divider) /* Disable user access to the timers and the physical counter. */ asm volatile("mrs %0, cntkctl_el1" : "=r" (cntkctl)); - cntkctl &= ~((3 << 8) | (0xf << 4) | (1 << 0)); + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN + | ARCH_TIMER_USR_VT_ACCESS_EN + | ARCH_TIMER_EVT_TRIGGER_MASK + | ARCH_TIMER_USR_PCT_ACCESS_EN) /* Enable user access to the virtual counter and frequency. */ - cntkctl |= (divider << 4) | (1 << 2) | (1 << 1); + cntkctl |= (divider << ARCH_TIMER_EVT_TRIGGER_SHIFT) + | ARCH_TIMER_VIRT_EVT_EN + | ARCH_TIMER_USR_VCT_ACCESS_EN; + asm volatile("msr cntkctl_el1, %0" : : "r" (cntkctl)); } diff --git a/include/clocksource/arm_arch_timer.h b/include/clocksource/arm_arch_timer.h index 24dc140..c4d0fc4 100644 --- a/include/clocksource/arm_arch_timer.h +++ b/include/clocksource/arm_arch_timer.h @@ -29,6 +29,14 @@ #define ARCH_TIMER_PHYS_ACCESS 0 #define ARCH_TIMER_VIRT_ACCESS 1 +#define ARCH_TIMER_USR_PCT_ACCESS_EN (1 << 0) /* physical counter */ +#define ARCH_TIMER_USR_VCT_ACCESS_EN (1 << 1) /* virtual counter */ +#define ARCH_TIMER_VIRT_EVT_EN (1 << 2) +#define ARCH_TIMER_EVT_TRIGGER_SHIFT (4) +#define ARCH_TIMER_EVT_TRIGGER_MASK (0xF << ARCH_TIMER_EVT_TRIGGER_SHIFT) +#define ARCH_TIMER_USR_VT_ACCESS_EN (1 << 8) /* virtual timer registers */ +#define ARCH_TIMER_USR_PT_ACCESS_EN (1 << 9) /* physical timer registers */ + #define ARCH_TIMER_EVT_STREAM_FREQ 10000 /* 100us */ #ifdef CONFIG_ARM_ARCH_TIMER -- 1.8.1.2
next prev parent reply other threads:[~2013-07-22 11:21 UTC|newest] Thread overview: 98+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-06-18 17:06 [PATCH 0/4] ARM/ARM64 architected timer updates Sudeep KarkadaNagesha 2013-06-18 17:06 ` Sudeep KarkadaNagesha 2013-06-18 17:07 ` [PATCH 1/4] drivers: clocksource: configure event stream for ARM arch timer Sudeep KarkadaNagesha 2013-06-18 17:07 ` Sudeep KarkadaNagesha 2013-06-18 17:07 ` [PATCH 2/4] ARM: arch_timer: add macros for bits in control register Sudeep KarkadaNagesha 2013-06-18 17:07 ` Sudeep KarkadaNagesha 2013-06-18 17:07 ` [PATCH 3/4] ARM/ARM64: arch_timer: remove __cpuinit attribute for arch_counter_set_user_access Sudeep KarkadaNagesha 2013-06-18 17:07 ` Sudeep KarkadaNagesha 2013-06-18 17:58 ` Sudeep KarkadaNagesha 2013-06-18 17:58 ` Sudeep KarkadaNagesha 2013-06-18 17:07 ` [PATCH 4/4] drivers: clocksource: add CPU PM notifier for ARM architected timer Sudeep KarkadaNagesha 2013-06-18 17:07 ` Sudeep KarkadaNagesha 2013-07-02 16:09 ` Jon Medhurst (Tixy) 2013-07-02 16:09 ` Jon Medhurst (Tixy) 2013-07-02 17:24 ` Sudeep KarkadaNagesha 2013-07-02 17:24 ` Sudeep KarkadaNagesha 2013-07-22 11:21 ` [PATCH v2 0/3] ARM/ARM64 architected timer updates Sudeep KarkadaNagesha 2013-07-22 11:21 ` Sudeep KarkadaNagesha 2013-07-22 11:21 ` [PATCH v2 1/3] drivers: clocksource: configure event stream for ARM arch timer Sudeep KarkadaNagesha 2013-07-22 11:21 ` Sudeep KarkadaNagesha 2013-07-23 10:23 ` Catalin Marinas 2013-07-23 10:23 ` Catalin Marinas 2013-07-23 10:33 ` Will Deacon 2013-07-23 10:33 ` Will Deacon 2013-07-23 11:12 ` Catalin Marinas 2013-07-23 11:12 ` Catalin Marinas 2013-07-25 9:45 ` [PATCH v3 " Sudeep KarkadaNagesha 2013-07-25 9:45 ` Sudeep KarkadaNagesha 2013-08-12 13:32 ` Sudeep KarkadaNagesha 2013-08-12 13:32 ` Sudeep KarkadaNagesha 2013-07-22 11:21 ` Sudeep KarkadaNagesha [this message] 2013-07-22 11:21 ` [PATCH v2 2/3] ARM: arch_timer: add macros for bits in control register Sudeep KarkadaNagesha 2013-07-22 11:21 ` [PATCH v2 3/3] drivers: clocksource: add CPU PM notifier for ARM architected timer Sudeep KarkadaNagesha 2013-07-22 11:21 ` Sudeep KarkadaNagesha 2013-08-02 9:32 ` [PATCH v2 0/3] ARM/ARM64 architected timer updates Sudeep KarkadaNagesha 2013-08-02 9:32 ` Sudeep KarkadaNagesha 2013-08-02 9:48 ` Daniel Lezcano 2013-08-02 9:48 ` Daniel Lezcano 2013-08-02 9:55 ` Sudeep KarkadaNagesha 2013-08-02 9:55 ` Sudeep KarkadaNagesha 2013-08-02 12:12 ` Daniel Lezcano 2013-08-02 12:12 ` Daniel Lezcano 2013-08-05 16:25 ` Sudeep KarkadaNagesha 2013-08-05 16:25 ` Sudeep KarkadaNagesha 2013-08-06 8:09 ` Daniel Lezcano 2013-08-06 8:09 ` Daniel Lezcano 2013-08-06 9:01 ` Sudeep KarkadaNagesha 2013-08-06 9:01 ` Sudeep KarkadaNagesha 2013-08-12 13:16 ` Daniel Lezcano 2013-08-12 13:16 ` Daniel Lezcano 2013-08-13 17:29 ` [PATCH v3 0/6] " Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-13 17:29 ` [PATCH v3 1/6] ARM/ARM64: arch_timer: add macros for bits in control register Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-14 8:50 ` Daniel Lezcano 2013-08-14 8:50 ` Daniel Lezcano 2013-08-14 9:03 ` Marc Zyngier 2013-08-14 9:03 ` Marc Zyngier 2013-08-14 9:23 ` Daniel Lezcano 2013-08-14 9:23 ` Daniel Lezcano 2013-08-13 17:29 ` [PATCH v3 2/6] drivers: clocksource: configure event stream for ARM arch timer Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-13 17:29 ` [PATCH v3 3/6] ARM: arch_timer: configure and enable event stream Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-13 17:29 ` [PATCH v3 4/6] ARM64: " Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-20 13:27 ` Will Deacon 2013-08-20 13:27 ` Will Deacon 2013-08-20 14:27 ` Sudeep KarkadaNagesha 2013-08-20 14:27 ` Sudeep KarkadaNagesha 2013-08-20 16:16 ` Will Deacon 2013-08-20 16:16 ` Will Deacon 2013-08-20 16:23 ` Sudeep KarkadaNagesha 2013-08-20 16:23 ` Sudeep KarkadaNagesha 2013-08-20 17:08 ` Sudeep KarkadaNagesha 2013-08-20 17:08 ` Sudeep KarkadaNagesha 2013-08-21 17:52 ` Will Deacon 2013-08-21 17:52 ` Will Deacon 2013-08-13 17:29 ` [PATCH v3 5/6] drivers: clocksource: enable hwcaps for event stream on ARM arch timer Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-13 17:29 ` [PATCH v3 6/6] drivers: clocksource: add CPU PM notifier for ARM architected timer Sudeep KarkadaNagesha 2013-08-13 17:29 ` Sudeep KarkadaNagesha 2013-08-14 9:08 ` [PATCH v3 0/6] ARM/ARM64 architected timer updates Daniel Lezcano 2013-08-14 9:08 ` Daniel Lezcano 2013-08-14 9:17 ` Sudeep KarkadaNagesha 2013-08-14 9:17 ` Sudeep KarkadaNagesha 2013-08-14 10:51 ` Daniel Lezcano 2013-08-14 10:51 ` Daniel Lezcano 2013-08-14 11:03 ` Sudeep KarkadaNagesha 2013-08-14 11:03 ` Sudeep KarkadaNagesha 2013-08-14 15:00 ` Daniel Lezcano 2013-08-14 15:00 ` Daniel Lezcano 2013-08-20 17:14 ` Sudeep KarkadaNagesha 2013-08-20 17:14 ` Sudeep KarkadaNagesha 2013-08-23 9:26 ` Catalin Marinas 2013-08-23 9:26 ` Catalin Marinas 2013-08-23 9:57 ` Sudeep KarkadaNagesha 2013-08-23 9:57 ` Sudeep KarkadaNagesha
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1374492082-13686-3-git-send-email-Sudeep.KarkadaNagesha@arm.com \ --to=sudeep.karkadanagesha@arm.com \ --cc=catalin.marinas@arm.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=lorenzo.pieralisi@arm.com \ --cc=mark.rutland@arm.com \ --cc=will.deacon@arm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.