Re: [v7 wlcore: add missing nvs file name info for wilink8

From: Kalle Valo
Date: Wed Aug 16 2017 - 09:57:59 EST


"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