From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Sudeep KarkadaNagesha <Sudeep.KarkadaNagesha@arm.com> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Thomas Gleixner <tglx@linutronix.de> Subject: Re: [PATCH v3 1/6] ARM/ARM64: arch_timer: add macros for bits in control register Date: Wed, 14 Aug 2013 10:50:23 +0200 [thread overview] Message-ID: <520B44CF.6040403@linaro.org> (raw) In-Reply-To: <1376414984-14182-2-git-send-email-Sudeep.KarkadaNagesha@arm.com> On 08/13/2013 07:29 PM, Sudeep KarkadaNagesha wrote: > 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. > > Cc: Catalin Marinas <catalin.marinas@arm.com> > 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 | 12 ++++++++---- > include/clocksource/arm_arch_timer.h | 8 ++++++++ > 3 files changed, 23 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/include/asm/arch_timer.h b/arch/arm/include/asm/arch_timer.h > index e406d57..9ef74da 100644 > --- a/arch/arm/include/asm/arch_timer.h > +++ b/arch/arm/include/asm/arch_timer.h > @@ -95,8 +95,13 @@ static inline void arch_counter_set_user_access(void) > > asm volatile("mrc p15, 0, %0, c14, c1, 0" : "=r" (cntkctl)); > > - /* disable user access to everything */ > - cntkctl &= ~((3 << 8) | (7 << 0)); > + /* Disable user access to both physical/virtual counters/timers. */ > + /* Also disable virtual event stream */ > + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN > + | ARCH_TIMER_USR_VT_ACCESS_EN > + | ARCH_TIMER_VIRT_EVT_EN > + | ARCH_TIMER_USR_VCT_ACCESS_EN > + | ARCH_TIMER_USR_PCT_ACCESS_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 98abd47..00b09d0 100644 > --- a/arch/arm64/include/asm/arch_timer.h > +++ b/arch/arm64/include/asm/arch_timer.h > @@ -101,12 +101,16 @@ static inline void arch_counter_set_user_access(void) > { > u32 cntkctl; > > - /* Disable user access to the timers and the physical counter. */ > asm volatile("mrs %0, cntkctl_el1" : "=r" (cntkctl)); > - cntkctl &= ~((3 << 8) | (1 << 0)); > > - /* Enable user access to the virtual counter and frequency. */ > - cntkctl |= (1 << 1); > + /* Disable user access to the timers and the physical counter. */ > + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN > + | ARCH_TIMER_USR_VT_ACCESS_EN > + | ARCH_TIMER_USR_PCT_ACCESS_EN); > + > + /* Enable user access to the virtual counter. */ > + cntkctl |= ARCH_TIMER_USR_VCT_ACCESS_EN; > + > asm volatile("msr cntkctl_el1, %0" : : "r" (cntkctl)); > } Before doing those changes, I suggest you kill include/asm/arch_timer.h header file and move the functions directly in the arm_arch_timer.c as they are only used by it. > diff --git a/include/clocksource/arm_arch_timer.h b/include/clocksource/arm_arch_timer.h > index c463ce9..551f7e9 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 */ > + > #ifdef CONFIG_ARM_ARCH_TIMER > > extern u32 arch_timer_get_rate(void); > -- <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
WARNING: multiple messages have this Message-ID (diff)
From: daniel.lezcano@linaro.org (Daniel Lezcano) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 1/6] ARM/ARM64: arch_timer: add macros for bits in control register Date: Wed, 14 Aug 2013 10:50:23 +0200 [thread overview] Message-ID: <520B44CF.6040403@linaro.org> (raw) In-Reply-To: <1376414984-14182-2-git-send-email-Sudeep.KarkadaNagesha@arm.com> On 08/13/2013 07:29 PM, Sudeep KarkadaNagesha wrote: > 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. > > Cc: Catalin Marinas <catalin.marinas@arm.com> > 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 | 12 ++++++++---- > include/clocksource/arm_arch_timer.h | 8 ++++++++ > 3 files changed, 23 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/include/asm/arch_timer.h b/arch/arm/include/asm/arch_timer.h > index e406d57..9ef74da 100644 > --- a/arch/arm/include/asm/arch_timer.h > +++ b/arch/arm/include/asm/arch_timer.h > @@ -95,8 +95,13 @@ static inline void arch_counter_set_user_access(void) > > asm volatile("mrc p15, 0, %0, c14, c1, 0" : "=r" (cntkctl)); > > - /* disable user access to everything */ > - cntkctl &= ~((3 << 8) | (7 << 0)); > + /* Disable user access to both physical/virtual counters/timers. */ > + /* Also disable virtual event stream */ > + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN > + | ARCH_TIMER_USR_VT_ACCESS_EN > + | ARCH_TIMER_VIRT_EVT_EN > + | ARCH_TIMER_USR_VCT_ACCESS_EN > + | ARCH_TIMER_USR_PCT_ACCESS_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 98abd47..00b09d0 100644 > --- a/arch/arm64/include/asm/arch_timer.h > +++ b/arch/arm64/include/asm/arch_timer.h > @@ -101,12 +101,16 @@ static inline void arch_counter_set_user_access(void) > { > u32 cntkctl; > > - /* Disable user access to the timers and the physical counter. */ > asm volatile("mrs %0, cntkctl_el1" : "=r" (cntkctl)); > - cntkctl &= ~((3 << 8) | (1 << 0)); > > - /* Enable user access to the virtual counter and frequency. */ > - cntkctl |= (1 << 1); > + /* Disable user access to the timers and the physical counter. */ > + cntkctl &= ~(ARCH_TIMER_USR_PT_ACCESS_EN > + | ARCH_TIMER_USR_VT_ACCESS_EN > + | ARCH_TIMER_USR_PCT_ACCESS_EN); > + > + /* Enable user access to the virtual counter. */ > + cntkctl |= ARCH_TIMER_USR_VCT_ACCESS_EN; > + > asm volatile("msr cntkctl_el1, %0" : : "r" (cntkctl)); > } Before doing those changes, I suggest you kill include/asm/arch_timer.h header file and move the functions directly in the arm_arch_timer.c as they are only used by it. > diff --git a/include/clocksource/arm_arch_timer.h b/include/clocksource/arm_arch_timer.h > index c463ce9..551f7e9 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 */ > + > #ifdef CONFIG_ARM_ARCH_TIMER > > extern u32 arch_timer_get_rate(void); > -- <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
next prev parent reply other threads:[~2013-08-14 8:50 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 ` [PATCH v2 2/3] ARM: arch_timer: add macros for bits in control register Sudeep KarkadaNagesha 2013-07-22 11:21 ` 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 [this message] 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=520B44CF.6040403@linaro.org \ --to=daniel.lezcano@linaro.org \ --cc=Sudeep.KarkadaNagesha@arm.com \ --cc=catalin.marinas@arm.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lorenzo.pieralisi@arm.com \ --cc=tglx@linutronix.de \ --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.