From: Kalle Valo <kvalo@codeaurora.org> To: Jouni Malinen <jouni@codeaurora.org> Cc: ath11k@lists.infradead.org, linux-wireless@vger.kernel.org, Rameshkumar Sundaram <ramess@codeaurora.org>, Sathishkumar Muruganandam <murugana@codeaurora.org> Subject: Re: [PATCH 10/12] ath11k: Send PPDU_STATS_CFG with proper pdev mask to firmware Date: Tue, 28 Sep 2021 13:46:58 +0300 [thread overview] Message-ID: <87a6jxq8nx.fsf@codeaurora.org> (raw) In-Reply-To: <20210721212029.142388-10-jouni@codeaurora.org> (Jouni Malinen's message of "Thu, 22 Jul 2021 00:20:27 +0300") Jouni Malinen <jouni@codeaurora.org> writes: > From: Rameshkumar Sundaram <ramess@codeaurora.org> > > HTT_PPDU_STATS_CFG_PDEV_ID bit mask for target FW PPDU stats request > message was set as bit 8 to 15. Bit 8 is reserved for soc stats and > pdev id starts from bit 9. Hence change the bitmask as bit 9 to 15 > and fill the proper pdev id in the request message. > > Tested on: IPQ8074 hw2.0 AHB WLAN.HK.2.5.0.1-01092-QCAHKSWPL_SILICONZ-1 > Tested on: IPQ6018 hw1.0 WLAN.HK.2.5.0.1-01067-QCAHKSWPL_SILICONZ-1 > > Fixes: 701e48a43e15 ("ath11k: add packet log support for QCA6390") > > Co-developed-by: Sathishkumar Muruganandam <murugana@codeaurora.org> > Signed-off-by: Sathishkumar Muruganandam <murugana@codeaurora.org> > Signed-off-by: Rameshkumar Sundaram <ramess@codeaurora.org> > Signed-off-by: Jouni Malinen <jouni@codeaurora.org> > --- > drivers/net/wireless/ath/ath11k/dp.h | 3 ++- > drivers/net/wireless/ath/ath11k/dp_tx.c | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath11k/dp.h b/drivers/net/wireless/ath/ath11k/dp.h > index b2e7621969c5..522d3a6ce253 100644 > --- a/drivers/net/wireless/ath/ath11k/dp.h > +++ b/drivers/net/wireless/ath/ath11k/dp.h > @@ -541,7 +541,8 @@ struct htt_ppdu_stats_cfg_cmd { > } __packed; > > #define HTT_PPDU_STATS_CFG_MSG_TYPE GENMASK(7, 0) > -#define HTT_PPDU_STATS_CFG_PDEV_ID GENMASK(15, 8) > +#define HTT_PPDU_STATS_CFG_SOC_STATS BIT(8) > +#define HTT_PPDU_STATS_CFG_PDEV_ID GENMASK(15, 9) This part is clear after reading the commit log. > --- a/drivers/net/wireless/ath/ath11k/dp_tx.c > +++ b/drivers/net/wireless/ath/ath11k/dp_tx.c > @@ -906,7 +906,7 @@ int ath11k_dp_tx_htt_h2t_ppdu_stats_req(struct ath11k *ar, u32 mask) > cmd->msg = FIELD_PREP(HTT_PPDU_STATS_CFG_MSG_TYPE, > HTT_H2T_MSG_TYPE_PPDU_STATS_CFG); > > - pdev_mask = 1 << (i + 1); > + pdev_mask = 1 << (ar->pdev_idx + i); > cmd->msg |= FIELD_PREP(HTT_PPDU_STATS_CFG_PDEV_ID, pdev_mask); > cmd->msg |= FIELD_PREP(HTT_PPDU_STATS_CFG_TLV_TYPE_BITMASK, mask); But this part isn't. All I can see in the commit log is a vague comment "and fill the proper pdev id in the request message" and nothing else. Please provide a proper explanation for this change and I can then add that to the commit log. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
WARNING: multiple messages have this Message-ID (diff)
From: Kalle Valo <kvalo@codeaurora.org> To: Jouni Malinen <jouni@codeaurora.org> Cc: ath11k@lists.infradead.org, linux-wireless@vger.kernel.org, Rameshkumar Sundaram <ramess@codeaurora.org>, Sathishkumar Muruganandam <murugana@codeaurora.org> Subject: Re: [PATCH 10/12] ath11k: Send PPDU_STATS_CFG with proper pdev mask to firmware Date: Tue, 28 Sep 2021 13:46:58 +0300 [thread overview] Message-ID: <87a6jxq8nx.fsf@codeaurora.org> (raw) In-Reply-To: <20210721212029.142388-10-jouni@codeaurora.org> (Jouni Malinen's message of "Thu, 22 Jul 2021 00:20:27 +0300") Jouni Malinen <jouni@codeaurora.org> writes: > From: Rameshkumar Sundaram <ramess@codeaurora.org> > > HTT_PPDU_STATS_CFG_PDEV_ID bit mask for target FW PPDU stats request > message was set as bit 8 to 15. Bit 8 is reserved for soc stats and > pdev id starts from bit 9. Hence change the bitmask as bit 9 to 15 > and fill the proper pdev id in the request message. > > Tested on: IPQ8074 hw2.0 AHB WLAN.HK.2.5.0.1-01092-QCAHKSWPL_SILICONZ-1 > Tested on: IPQ6018 hw1.0 WLAN.HK.2.5.0.1-01067-QCAHKSWPL_SILICONZ-1 > > Fixes: 701e48a43e15 ("ath11k: add packet log support for QCA6390") > > Co-developed-by: Sathishkumar Muruganandam <murugana@codeaurora.org> > Signed-off-by: Sathishkumar Muruganandam <murugana@codeaurora.org> > Signed-off-by: Rameshkumar Sundaram <ramess@codeaurora.org> > Signed-off-by: Jouni Malinen <jouni@codeaurora.org> > --- > drivers/net/wireless/ath/ath11k/dp.h | 3 ++- > drivers/net/wireless/ath/ath11k/dp_tx.c | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath11k/dp.h b/drivers/net/wireless/ath/ath11k/dp.h > index b2e7621969c5..522d3a6ce253 100644 > --- a/drivers/net/wireless/ath/ath11k/dp.h > +++ b/drivers/net/wireless/ath/ath11k/dp.h > @@ -541,7 +541,8 @@ struct htt_ppdu_stats_cfg_cmd { > } __packed; > > #define HTT_PPDU_STATS_CFG_MSG_TYPE GENMASK(7, 0) > -#define HTT_PPDU_STATS_CFG_PDEV_ID GENMASK(15, 8) > +#define HTT_PPDU_STATS_CFG_SOC_STATS BIT(8) > +#define HTT_PPDU_STATS_CFG_PDEV_ID GENMASK(15, 9) This part is clear after reading the commit log. > --- a/drivers/net/wireless/ath/ath11k/dp_tx.c > +++ b/drivers/net/wireless/ath/ath11k/dp_tx.c > @@ -906,7 +906,7 @@ int ath11k_dp_tx_htt_h2t_ppdu_stats_req(struct ath11k *ar, u32 mask) > cmd->msg = FIELD_PREP(HTT_PPDU_STATS_CFG_MSG_TYPE, > HTT_H2T_MSG_TYPE_PPDU_STATS_CFG); > > - pdev_mask = 1 << (i + 1); > + pdev_mask = 1 << (ar->pdev_idx + i); > cmd->msg |= FIELD_PREP(HTT_PPDU_STATS_CFG_PDEV_ID, pdev_mask); > cmd->msg |= FIELD_PREP(HTT_PPDU_STATS_CFG_TLV_TYPE_BITMASK, mask); But this part isn't. All I can see in the commit log is a vague comment "and fill the proper pdev id in the request message" and nothing else. Please provide a proper explanation for this change and I can then add that to the commit log. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k
next prev parent reply other threads:[~2021-09-28 10:47 UTC|newest] Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-07-21 21:20 [PATCH 01/12] ath11k: Fix pktlog lite rx events Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 02/12] ath11k: Update pdev tx and rx firmware stats Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 03/12] ath11k: Avoid reg rules update during firmware recovery Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 04/12] ath11k: Avoid race during regd updates Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 05/12] ath11k: Add vdev start flag to disable hardware encryption Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 06/12] ath11k: Assign free_vdev_map value before ieee80211_register_hw Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 07/12] ath11k: Fix crash during firmware recovery on reo cmd ring access Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 08/12] ath11k: Avoid "No VIF found" warning message Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 09/12] ath11k: Add wmi peer create conf event in wmi_tlv_event_id Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 10/12] ath11k: Send PPDU_STATS_CFG with proper pdev mask to firmware Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-09-28 10:46 ` Kalle Valo [this message] 2021-09-28 10:46 ` Kalle Valo 2021-09-28 12:01 ` Rameshkumar Sundaram 2021-09-28 12:01 ` Rameshkumar Sundaram 2021-11-12 8:01 ` Kalle Valo 2021-11-12 8:01 ` Kalle Valo 2021-07-21 21:20 ` [PATCH 11/12] ath11k: Clear auth flag only for actual association in security mode Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-07-21 21:20 ` [PATCH 12/12] ath11k: Change QCN9074 firmware to operate in mode-2 Jouni Malinen 2021-07-21 21:20 ` Jouni Malinen 2021-11-12 8:19 ` Kalle Valo 2021-11-12 8:19 ` Kalle Valo 2021-12-08 12:39 ` Anilkumar Kolli 2021-12-08 12:39 ` Anilkumar Kolli 2021-09-28 10:58 ` [PATCH 01/12] ath11k: Fix pktlog lite rx events Kalle Valo 2021-09-28 10:58 ` Kalle Valo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=87a6jxq8nx.fsf@codeaurora.org \ --to=kvalo@codeaurora.org \ --cc=ath11k@lists.infradead.org \ --cc=jouni@codeaurora.org \ --cc=linux-wireless@vger.kernel.org \ --cc=murugana@codeaurora.org \ --cc=ramess@codeaurora.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.