Messages in this thread | | | Date | Tue, 09 Apr 2013 07:49:20 -0700 | From | Stephen Boyd <> | Subject | Re: [PATCH 3/4] clocksource: arch_timer: Push the read/write wrappers deeper |
| |
On 04/09/13 02:38, Mark Rutland wrote: > On Tue, Apr 09, 2013 at 03:30:22AM +0100, Stephen Boyd wrote: >> -static inline u32 arch_timer_reg_read(const int access, const int reg) >> +static inline u32 __arch_timer_reg_read(const int access, const int reg) > Rather than prefixing all of these with "__", why not add "cp15" to the names? > > I think that'd make the end result more consistent and clearer, as we'd have a > _mem version and a _cp15 version of each function.
Sounds fair. I'll make the change.
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
| |