"Reizer, Eyal" <[email protected]> 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

Reply via email to