On 10 May 2016 at 01:11, <gree...@candelatech.com> wrote:
> From: Ben Greear <gree...@candelatech.com>
>
> They are not necessarily named in an intuitive manner,
> so at least add some comments to help the next person.
>
> Signed-off-by: Ben Greear <gree...@candelatech.com>
> ---
> drivers/net/wireless/ath/ath10k/core.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath10k/core.h
> b/drivers/net/wireless/ath/ath10k/core.h
> index e7c228a..c4f649f 100644
> --- a/drivers/net/wireless/ath/ath10k/core.h
> +++ b/drivers/net/wireless/ath/ath10k/core.h
> @@ -189,10 +189,10 @@ struct ath10k_fw_stats_pdev {
>
> /* PDEV stats */
> s32 ch_noise_floor;
> - u32 tx_frame_count;
> - u32 rx_frame_count;
> - u32 rx_clear_count;
> - u32 cycle_count;
> + u32 tx_frame_count; /* cycles spent transmitting frames */
> + u32 rx_frame_count; /* cycles spent receiving frames */
> + u32 rx_clear_count; /* Total channel busy time, evidently */
> + u32 cycle_count; /* Total on-channel time */
Hmm, there are also other instances of these vars in wmi.h. Although
redundant maybe it's worth to comment them as well (or first and
foremost they should be the ones that need a comment as they are the
"source"). Just an idea.
(oh, and my silly OCD is irritated at the big/small first letters in
the comments not being consistent)
Michał
_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k
