* [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4
@ 2016-03-22 22:44 Peter Oh
2016-03-22 23:14 ` kbuild test robot
2016-04-04 14:57 ` Valo, Kalle
0 siblings, 2 replies; 5+ messages in thread
From: Peter Oh @ 2016-03-22 22:44 UTC (permalink / raw)
To: ath10k; +Cc: linux-wireless
Check and parse Rx MAC timestamp when firmware sets its flag
to status variable.
10.4 firmware adds it in Rx beacon frame only at this moment.
Drivers and mac80211 may utilize it to detect such clockdrift
or beacon collision and use the result for beacon collision
avoidance.
Signed-off-by: Peter Oh <poh@qca.qualcomm.com>
---
v2:
* Rebased. no code changes.
drivers/net/wireless/ath/ath10k/wmi.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c
index 9137566..9977876 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.c
+++ b/drivers/net/wireless/ath/ath10k/wmi.c
@@ -2191,6 +2191,8 @@ static int ath10k_wmi_10_4_op_pull_mgmt_rx_ev(struct ath10k *ar,
struct wmi_10_4_mgmt_rx_hdr *ev_hdr;
size_t pull_len;
u32 msdu_len;
+ struct wmi_mgmt_rx_ext_info *ext_info;
+ u32 len;
ev = (struct wmi_10_4_mgmt_rx_event *)skb->data;
ev_hdr = &ev->hdr;
@@ -2211,6 +2213,13 @@ static int ath10k_wmi_10_4_op_pull_mgmt_rx_ev(struct ath10k *ar,
if (skb->len < msdu_len)
return -EPROTO;
+ if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
+ len = ALIGN(le32_to_cpu(arg->buf_len), 4);
+ ext_info = (struct wmi_mgmt_rx_ext_info *)(skb->data + len);
+ memcpy(&arg->ext_info, ext_info,
+ sizeof(struct wmi_mgmt_rx_ext_info));
+ }
+
/* Make sure bytes added for padding are removed. */
skb_trim(skb, msdu_len);
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4
2016-03-22 22:44 [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4 Peter Oh
@ 2016-03-22 23:14 ` kbuild test robot
2016-03-23 0:14 ` Peter Oh
2016-04-04 14:57 ` Valo, Kalle
1 sibling, 1 reply; 5+ messages in thread
From: kbuild test robot @ 2016-03-22 23:14 UTC (permalink / raw)
To: Peter Oh; +Cc: kbuild-all, ath10k, linux-wireless
[-- Attachment #1: Type: text/plain, Size: 3507 bytes --]
Hi Peter,
[auto build test WARNING on wireless-drivers-next/master]
[also build test WARNING on v4.5 next-20160322]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/Peter-Oh/ath10k-parse-Rx-MAC-timestamp-in-mgmt-frame-for-FW-10-4/20160323-064843
base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: x86_64-randconfig-x000-201612 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
In file included from include/linux/linkage.h:4:0,
from include/linux/kernel.h:6,
from include/linux/skbuff.h:17,
from drivers/net/wireless/ath/ath10k/wmi.c:18:
drivers/net/wireless/ath/ath10k/wmi.c: In function 'ath10k_wmi_10_4_op_pull_mgmt_rx_ev':
drivers/net/wireless/ath/ath10k/wmi.c:2236:33: error: 'WMI_RX_STATUS_EXT_INFO' undeclared (first use in this function)
if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
^
include/linux/compiler.h:147:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^
>> drivers/net/wireless/ath/ath10k/wmi.c:2236:2: note: in expansion of macro 'if'
if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
^
drivers/net/wireless/ath/ath10k/wmi.c:2236:33: note: each undeclared identifier is reported only once for each function it appears in
if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
^
include/linux/compiler.h:147:30: note: in definition of macro '__trace_if'
if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
^
>> drivers/net/wireless/ath/ath10k/wmi.c:2236:2: note: in expansion of macro 'if'
if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
^
drivers/net/wireless/ath/ath10k/wmi.c:2239:14: error: 'struct wmi_mgmt_rx_ev_arg' has no member named 'ext_info'
memcpy(&arg->ext_info, ext_info,
^
drivers/net/wireless/ath/ath10k/wmi.c:2240:17: error: invalid application of 'sizeof' to incomplete type 'struct wmi_mgmt_rx_ext_info'
sizeof(struct wmi_mgmt_rx_ext_info));
^
vim +/if +2236 drivers/net/wireless/ath/ath10k/wmi.c
2220
2221 if (skb->len < pull_len)
2222 return -EPROTO;
2223
2224 skb_pull(skb, pull_len);
2225 arg->channel = ev_hdr->channel;
2226 arg->buf_len = ev_hdr->buf_len;
2227 arg->status = ev_hdr->status;
2228 arg->snr = ev_hdr->snr;
2229 arg->phy_mode = ev_hdr->phy_mode;
2230 arg->rate = ev_hdr->rate;
2231
2232 msdu_len = __le32_to_cpu(arg->buf_len);
2233 if (skb->len < msdu_len)
2234 return -EPROTO;
2235
> 2236 if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
2237 len = ALIGN(le32_to_cpu(arg->buf_len), 4);
2238 ext_info = (struct wmi_mgmt_rx_ext_info *)(skb->data + len);
2239 memcpy(&arg->ext_info, ext_info,
2240 sizeof(struct wmi_mgmt_rx_ext_info));
2241 }
2242
2243 /* Make sure bytes added for padding are removed. */
2244 skb_trim(skb, msdu_len);
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 33947 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4
2016-03-22 23:14 ` kbuild test robot
@ 2016-03-23 0:14 ` Peter Oh
2016-03-23 6:27 ` Michal Kazior
0 siblings, 1 reply; 5+ messages in thread
From: Peter Oh @ 2016-03-23 0:14 UTC (permalink / raw)
To: kbuild test robot, Peter Oh; +Cc: kbuild-all, ath10k, linux-wireless
On 03/22/2016 04:14 PM, kbuild test robot wrote:
> Hi Peter,
>
> [auto build test WARNING on wireless-drivers-next/master]
> [also build test WARNING on v4.5 next-20160322]
> [if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
>
> url: https://github.com/0day-ci/linux/commits/Peter-Oh/ath10k-parse-Rx-MAC-timestamp-in-mgmt-frame-for-FW-10-4/20160323-064843
> base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
> config: x86_64-randconfig-x000-201612 (attached as .config)
> reproduce:
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> All warnings (new ones prefixed by >>):
>
> In file included from include/linux/linkage.h:4:0,
> from include/linux/kernel.h:6,
> from include/linux/skbuff.h:17,
> from drivers/net/wireless/ath/ath10k/wmi.c:18:
> drivers/net/wireless/ath/ath10k/wmi.c: In function 'ath10k_wmi_10_4_op_pull_mgmt_rx_ev':
> drivers/net/wireless/ath/ath10k/wmi.c:2236:33: error: 'WMI_RX_STATUS_EXT_INFO' undeclared (first use in this function)
it seems the warning is false report. I could see WMI_RX_STATUS_EXT_INFO
is defined in wmi.h.
Moreover this check command doesn't claim any warning/error make
M=drivers/net/wireless/ath/ath10k C=2 CF="-D__CHECK_ENDIAN__"
> if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
> ^
> include/linux/compiler.h:147:30: note: in definition of macro '__trace_if'
> if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
> ^
>>> drivers/net/wireless/ath/ath10k/wmi.c:2236:2: note: in expansion of macro 'if'
> if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
> ^
> drivers/net/wireless/ath/ath10k/wmi.c:2236:33: note: each undeclared identifier is reported only once for each function it appears in
> if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
> ^
> include/linux/compiler.h:147:30: note: in definition of macro '__trace_if'
> if (__builtin_constant_p(!!(cond)) ? !!(cond) : \
> ^
>>> drivers/net/wireless/ath/ath10k/wmi.c:2236:2: note: in expansion of macro 'if'
> if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
> ^
> drivers/net/wireless/ath/ath10k/wmi.c:2239:14: error: 'struct wmi_mgmt_rx_ev_arg' has no member named 'ext_info'
> memcpy(&arg->ext_info, ext_info,
> ^
> drivers/net/wireless/ath/ath10k/wmi.c:2240:17: error: invalid application of 'sizeof' to incomplete type 'struct wmi_mgmt_rx_ext_info'
> sizeof(struct wmi_mgmt_rx_ext_info));
> ^
>
> vim +/if +2236 drivers/net/wireless/ath/ath10k/wmi.c
>
> 2220
> 2221 if (skb->len < pull_len)
> 2222 return -EPROTO;
> 2223
> 2224 skb_pull(skb, pull_len);
> 2225 arg->channel = ev_hdr->channel;
> 2226 arg->buf_len = ev_hdr->buf_len;
> 2227 arg->status = ev_hdr->status;
> 2228 arg->snr = ev_hdr->snr;
> 2229 arg->phy_mode = ev_hdr->phy_mode;
> 2230 arg->rate = ev_hdr->rate;
> 2231
> 2232 msdu_len = __le32_to_cpu(arg->buf_len);
> 2233 if (skb->len < msdu_len)
> 2234 return -EPROTO;
> 2235
>> 2236 if (le32_to_cpu(arg->status) & WMI_RX_STATUS_EXT_INFO) {
> 2237 len = ALIGN(le32_to_cpu(arg->buf_len), 4);
> 2238 ext_info = (struct wmi_mgmt_rx_ext_info *)(skb->data + len);
> 2239 memcpy(&arg->ext_info, ext_info,
> 2240 sizeof(struct wmi_mgmt_rx_ext_info));
> 2241 }
> 2242
> 2243 /* Make sure bytes added for padding are removed. */
> 2244 skb_trim(skb, msdu_len);
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4
2016-03-23 0:14 ` Peter Oh
@ 2016-03-23 6:27 ` Michal Kazior
0 siblings, 0 replies; 5+ messages in thread
From: Michal Kazior @ 2016-03-23 6:27 UTC (permalink / raw)
To: Peter Oh
Cc: kbuild test robot, Peter Oh, kbuild-all,
ath10k@lists.infradead.org, linux-wireless
On 23 March 2016 at 01:14, Peter Oh <poh@codeaurora.org> wrote:
>
> On 03/22/2016 04:14 PM, kbuild test robot wrote:
>>
>> Hi Peter,
>>
>> [auto build test WARNING on wireless-drivers-next/master]
>> [also build test WARNING on v4.5 next-20160322]
>> [if your patch is applied to the wrong git tree, please drop us a note to
>> help improving the system]
>>
>> url:
>> https://github.com/0day-ci/linux/commits/Peter-Oh/ath10k-parse-Rx-MAC-timestamp-in-mgmt-frame-for-FW-10-4/20160323-064843
>> base:
>> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git
>> master
>> config: x86_64-randconfig-x000-201612 (attached as .config)
>> reproduce:
>> # save the attached .config to linux build tree
>> make ARCH=x86_64
>>
>> All warnings (new ones prefixed by >>):
>>
>> In file included from include/linux/linkage.h:4:0,
>> from include/linux/kernel.h:6,
>> from include/linux/skbuff.h:17,
>> from drivers/net/wireless/ath/ath10k/wmi.c:18:
>> drivers/net/wireless/ath/ath10k/wmi.c: In function
>> 'ath10k_wmi_10_4_op_pull_mgmt_rx_ev':
>> drivers/net/wireless/ath/ath10k/wmi.c:2236:33: error:
>> 'WMI_RX_STATUS_EXT_INFO' undeclared (first use in this function)
>
> it seems the warning is false report. I could see WMI_RX_STATUS_EXT_INFO is
> defined in wmi.h.
> Moreover this check command doesn't claim any warning/error make
> M=drivers/net/wireless/ath/ath10k C=2 CF="-D__CHECK_ENDIAN__"
I think the problem here is buildbot tested your patch against
wireless-drivers-next tree which doesn't include the macro yet. The
macro is in ath.git though.
Michał
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4
2016-03-22 22:44 [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4 Peter Oh
2016-03-22 23:14 ` kbuild test robot
@ 2016-04-04 14:57 ` Valo, Kalle
1 sibling, 0 replies; 5+ messages in thread
From: Valo, Kalle @ 2016-04-04 14:57 UTC (permalink / raw)
To: Oh, Peter; +Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org
Peter Oh <poh@qca.qualcomm.com> writes:
> Check and parse Rx MAC timestamp when firmware sets its flag
> to status variable.
> 10.4 firmware adds it in Rx beacon frame only at this moment.
> Drivers and mac80211 may utilize it to detect such clockdrift
> or beacon collision and use the result for beacon collision
> avoidance.
>
> Signed-off-by: Peter Oh <poh@qca.qualcomm.com>
Applied, thanks.
--
Kalle Valo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-04-04 14:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-22 22:44 [PATCH v2] ath10k: parse Rx MAC timestamp in mgmt frame for FW 10.4 Peter Oh
2016-03-22 23:14 ` kbuild test robot
2016-03-23 0:14 ` Peter Oh
2016-03-23 6:27 ` Michal Kazior
2016-04-04 14:57 ` Valo, Kalle
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).