From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from wolverine02.qualcomm.com ([199.106.114.251]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YGPki-0004Ho-J3 for ath10k@lists.infradead.org; Wed, 28 Jan 2015 10:21:40 +0000 Date: Wed, 28 Jan 2015 15:51:01 +0530 From: SenthilKumar Jegadeesan Subject: Re: [PATCH] ath10k:Prevent setting wrong key idx for station Message-ID: <20150128102059.GA22295@senthilj-ThinkPad-L430> References: <1422369059-21274-1-git-send-email-senthilb@qti.qualcomm.com> <87d25z9uuw.fsf@kamboji.qca.qualcomm.com> <20150128095205.GA22052@senthilj-ThinkPad-L430> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20150128095205.GA22052@senthilj-ThinkPad-L430> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Kalle Valo Cc: senthilb@qti.qualcomm.com, ath10k@lists.infradead.org On Wed, Jan 28, 2015 at 03:22:07PM +0530, SenthilKumar Jegadeesan wrote: > On Wed, Jan 28, 2015 at 10:23:35AM +0200, Kalle Valo wrote: > > writes: > > > > > From: SenthilKumar Jegadeesan > > > > > > Ath10k driver sets wrong default key idx that results in > > > sending unicast frames with multicast key. > > > > > > The reason for this behavior is that cached broadcast key > > > is installed for station MAC address on association. After > > > dot1x completes, unicast key is installed for station > > > MAC address. Default key idx is set to broadcast key id when > > > driver tries to send broadcast frame. This causes firmware > > > to use broadcast key id to transmit unicast frames to stations. > > > > > > Used TX_USAGE flag to set default key for stations. > > > > > > Added callback for setting unicast default idx which will be > > > invoked on every default key idx configuration. > > > > > > Signed-off-by: SenthilKumar Jegadeesan > > > > I had a trivial conflict in ath10k_add_interface(), please double check > > my conflict resolution in the pending branch: > > > > https://github.com/kvalo/ath/commit/34e5197d250d7be04bba5ad2b2d33cd9a7c41b0f > > > > And in the future please also CC linux-wireless when submitting ath10k > > patches. > > > Kalle, > > Confilcts aren't resolved properly. I will re-send the patch from top of the > branch. > The conflicts are resolved correclty. Sorry for the confusion. Thanks, Senthil J _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k