Hi All, I fixed only part of coding style violation. Not all of them. But you are right KERN_<level> is necessary. For me is especially important to move these files from arch/powerpc to drivers/of. M >> /* Debug utility */ >> #ifdef DEBUG >> static void of_dump_addr(const char *s, const u32 *addr, int na) >> { >> - printk("%s", s); >> - while(na--) >> - printk(" %08x", *(addr++)); >> - printk("\n"); >> + printk(KERN_INFO "%s", s); >> + while (na--) >> + printk(KERN_INFO " %08x", *(addr++)); >> + printk(KERN_INFO "\n"); >> } > > You've prefixed KERN_INFO before every address block. > > How about: > > print_hex_dump(KERN_<level>, s, DUMP_PREFIX_NONE, 4, addr, na, false); > > -- 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