From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-pa0-x22c.google.com ([2607:f8b0:400e:c03::22c]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Zqekw-00064T-K1 for ath10k@lists.infradead.org; Mon, 26 Oct 2015 10:11:59 +0000 Received: by padhk11 with SMTP id hk11so184722175pad.1 for ; Mon, 26 Oct 2015 03:11:37 -0700 (PDT) Message-ID: <562DFC61.7080108@gmail.com> Date: Mon, 26 Oct 2015 15:41:45 +0530 From: Manikanta MIME-Version: 1.0 Subject: Re: [PATCH] ath10k: update antenna configuration after setting in fw References: <1445851330-18736-1-git-send-email-manikanta.pubbisetty@gmail.com> In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Albino B Neto Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org On Monday 26 October 2015 03:35 PM, Albino B Neto wrote: > 2015-10-26 7:22 GMT-02:00 : >> Signed-off-by: Manikanta Pubbisetty >> --- >> drivers/net/wireless/ath/ath10k/mac.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >> index 484c1a1..8d0ce37 100644 >> --- a/drivers/net/wireless/ath/ath10k/mac.c >> +++ b/drivers/net/wireless/ath/ath10k/mac.c >> @@ -3771,9 +3771,6 @@ static int __ath10k_set_antenna(struct ath10k *ar, u32 tx_ant, u32 rx_ant) >> ath10k_check_chain_mask(ar, tx_ant, "tx"); >> ath10k_check_chain_mask(ar, rx_ant, "rx"); >> >> - ar->cfg_tx_chainmask = tx_ant; >> - ar->cfg_rx_chainmask = rx_ant; >> - >> if ((ar->state != ATH10K_STATE_ON) && >> (ar->state != ATH10K_STATE_RESTARTED)) >> return 0; >> @@ -3794,6 +3791,9 @@ static int __ath10k_set_antenna(struct ath10k *ar, u32 tx_ant, u32 rx_ant) >> return ret; >> } >> >> + ar->cfg_tx_chainmask = tx_ant; >> + ar->cfg_rx_chainmask = rx_ant; >> + >> return 0; >> } > Sorry, I see no change in the code. script checkpatch.pl ? I have moved these two lines of code from the start of the function to the end, in between there is code for setting the antenna config in firmware. You can find in commit log why I have moved the code down in the function. -Manikanta _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k