Search Linux Wireless

Re: [PATCH] ath10k: unify warning messages in mac.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 3 March 2014 16:43, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote:
> Currently there are different styles used for warning messages, unify them to
> look similar.
>
> Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxxxx>
> ---

[...]

This still seems inconsistent. I also don't like the capital letters
(even for abbreviations) in these kind of messages, but that's just my
OCD..

I think we should agree on one of the two approaches:

a) start with a verb:

"failed to add peer %pM on vdev %i: %d"
"failed to initialize dfs pattern detector"
"timed out while waiting for scan completion"

b) start with a noun:

"peer %pM on vdev %i could not be added: %d"
"dfs pattern detector could not initialize"
"scan timed out"

These are still mixed.

We could probably also limit the set of verbs, e.g. replace "could
not" with "failed to" as it's practically the same thing (assuming we
pick (a)).

But then again, feel free to ignore my OCD :-)


Michał
--
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




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux