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> -- 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