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

Michal Kazior michal.kazior at tieto.com
Mon Mar 3 11:23:24 EST 2014

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


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

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 :-)


More information about the ath10k mailing list