"Reizer, Eyal" <eyalr@xxxxxx> writes: >> > + if (wl->fuse_oui_addr == 0 && wl->fuse_nic_addr == 0) { >> > + wl1271_warning("Fuse mac address is zero. using " >> > + "random mac\n"); >> >> This one should also go into one line. > > This will still exceed 80 characters. Is this still ok? Yes, it's ok that the _format string_ goes above 80 chars, but if there are paremeters you should split them to a separate line. Example: pr_debug("This is an example of a very long message which also prints few variables foo %d bar %d", foo, bar); -- Kalle Valo