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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html