On Mon, Sep 19, 2011 at 03:38:02PM +0200, Thomas Gleixner wrote: > On Wed, 14 Sep 2011, Richard Kuo wrote: > > + > > +void debug_adsp_timers(void) > > +{ > > + printk(KERN_INFO "rtos_timer->match=0x%08x\n", rtos_timer->match); > > + printk(KERN_INFO "rtos_timer->count=%d\n", rtos_timer->count); > > + printk(KERN_INFO "rtos_timer->enable=%d\n", rtos_timer->enable); > > + printk(KERN_INFO "rtos_timer->clear=%d\n", rtos_timer->clear); > > +} > > Does this have a caller and is it useful ? > > Otherwise this looks good. > > Reviewed-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Mostly just me being paranoid. It can be removed. Thanks, Richard Kuo -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html