| Message ID | 1462835475-11079-14-git-send-email-greearb@candelatech.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Kalle Valo |
| Headers | show |
On 10 May 2016 at 01:11, <greearb@candelatech.com> wrote: > From: Ben Greear <greearb@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 <greearb@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?
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 */ u32 phy_err_count; u32 chan_tx_power; u32 ack_rx_bad;