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

[Ben Greear <greearb@candelatech.com>]

On 03/11/2014 04:16 AM, Kalle Valo wrote:
> Kalle Valo <kvalo@qca.qualcomm.com> writes:
> 
>>> 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)).
>>
>> That's true. I would vote for option (a).
> 
> I didn't get any comments about Michal's proposal. Can I conclude from
> this that (a) above is ok for everyone?


I like (a), but I would also suggest giving user a clue as to the
severity.  Something starting with 'failed' or 'warning' is pretty obvious.

A timeout is less so..ie, how much of a problem is the timeout?

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k