On Wed, 2011-07-13 at 04:34 +0300, Kalle Valo wrote: > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> [] > +int ath6kl_printk(const char *level, const char *fmt, ...) Looks familiar. > +#ifdef CONFIG_ATH6KL_DEBUG > +void ath6kl_dump_registers(struct ath6kl_device *dev, > + struct ath6kl_irq_proc_registers *irq_proc_reg, > + struct ath6kl_irqenable_reg *irq_enable_reg) > +{ > + > + ath6kl_dbg(ATH6KL_DBG_ANY, ("<------- Register Table -------->\n")); > + > + if (irq_proc_reg != NULL) { > + ath6kl_dbg(ATH6KL_DBG_ANY, > + "Host Int status: 0x%x\n", > + irq_proc_reg->host_int_status); > + ath6kl_dbg(ATH6KL_DBG_ANY, > + "CPU Int status: 0x%x\n", > + irq_proc_reg->cpu_int_status); Some text could be saved by using: ath6kl_dbg(ATH6KL_DBG_ANY, "%-26x 0x%x\n", "Host Int status:", inq_proc_reg->host_int_status); ath6kl_dbg(ATH6KL_DBG_ANY, "%-26x 0x%x\n" "CPU Int status:", irq_proc_reg->cpu_int_status); etc. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html