Hi Kalle, I will make sure, I will run sparse before sending it for review http://linuxwireless.org/en/users/Drivers/ath10k/CodingStyle/#Linux_style
make drivers/net/wireless/ath/ath10k/ C=2 CF="-D__CHECK_ENDIAN__" regret the inconvenience so caused (including the compilation error) regards, shafi -----Original Message----- From: Mohammed Shafi Shajakhan [mailto:moham...@codeaurora.org] Sent: Wednesday, March 23, 2016 6:34 PM To: Valo, Kalle <kv...@qca.qualcomm.com> Cc: Shajakhan, Mohammed Shafi (Mohammed Shafi) <moham...@qti.qualcomm.com>; ath...@lists.infradead.org; linux-wireless@vger.kernel.org Subject: Re: [PATCH v1 3/3] ath10k: Enable parsing per station rx duration for 10.4 Hi Kalle, On Wed, Mar 23, 2016 at 01:00:01PM +0000, Valo, Kalle wrote: > Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> writes: > > > From: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > > > Rx duration support for per station is part of extended peer stats, > > enable provision to parse the same and provide backward > > compatibility based on the 'stats_id' event > > > > Signed-off-by: Mohammed Shafi Shajakhan <moham...@qti.qualcomm.com> > > There was a new sparse warning: > > drivers/net/wireless/ath/ath10k/wmi.c:2978:42: warning: incorrect type in > assignment (different base types) > drivers/net/wireless/ath/ath10k/wmi.c:2978:42: expected unsigned int > [unsigned] [usertype] rx_duration > drivers/net/wireless/ath/ath10k/wmi.c:2978:42: got restricted __le32 const > [usertype] rx_duration > > I fixed it like this in the pending branch, please double check: [shafi] thanks for fixing this, sorry i missed this. > > --- a/drivers/net/wireless/ath/ath10k/wmi.c > +++ b/drivers/net/wireless/ath/ath10k/wmi.c > @@ -2975,7 +2975,7 @@ static int ath10k_wmi_10_4_op_pull_fw_stats(struct > ath10k *ar, > ath10k_wmi_10_4_pull_peer_stats(&src->common, dst); > /* FIXME: expose 10.4 specific values */ > if (extd_peer_stats) > - dst->rx_duration = src->rx_duration; > + dst->rx_duration = > + __le32_to_cpu(src->rx_duration); > > list_add_tail(&dst->list, &stats->peers); > } > regards, shafi -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
