From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from sabertooth02.qualcomm.com ([65.197.215.38]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1ZPqkt-0007IG-PM for ath10k@lists.infradead.org; Thu, 13 Aug 2015 11:33:08 +0000 From: Kalle Valo Subject: Re: [PATCH 1/2] ath10k: refactor phyerr event handlers References: <1439373396-10828-1-git-send-email-rmani@qti.qualcomm.com> Date: Thu, 13 Aug 2015 14:32:19 +0300 In-Reply-To: <1439373396-10828-1-git-send-email-rmani@qti.qualcomm.com> (Raja Mani's message of "Wed, 12 Aug 2015 15:26:35 +0530") Message-ID: <87si7nmnjw.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 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: Raja Mani Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Raja Mani writes: > Existing phyerr event handlers directly uses phyerr header format > (ie, struct wmi_phyerr and struct wmi_phyerr_event) in the code > exactly on how firmware packs it. This is the problem in 10.4 fw > specific phyerr event handling where it uses different phyerror > header format. Before adding 10.4 specific handler, little bit of > refactor is done in existing phyerr handlers. > > Two new abstracted structures (struct wmi_phyerr_ev_hdr_arg and > struct wmi_phyerr_ev_arg) are introduced to remove dependency of using > firmware specific header format in the code. So that firmware specific > phyerror handlers can populate values to abstracted structures and > the following code can use abstracted struct for further operation. > > .pull_phyerr_hdr is added newly to pull common phyerr header info > like tsf, buf_len, number of phyerr packed. Existing .pull_phyerr > handler is changed and called to parse every sub phyerrs in the event. > > Validated these refactoring on qca988x hw2.0 using fw 10.2.4 version. > > Signed-off-by: Raja Mani Thanks, applied. -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:40646 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752289AbbHMLc1 (ORCPT ); Thu, 13 Aug 2015 07:32:27 -0400 From: Kalle Valo To: Raja Mani CC: , Subject: Re: [PATCH 1/2] ath10k: refactor phyerr event handlers References: <1439373396-10828-1-git-send-email-rmani@qti.qualcomm.com> Date: Thu, 13 Aug 2015 14:32:19 +0300 In-Reply-To: <1439373396-10828-1-git-send-email-rmani@qti.qualcomm.com> (Raja Mani's message of "Wed, 12 Aug 2015 15:26:35 +0530") Message-ID: <87si7nmnjw.fsf@kamboji.qca.qualcomm.com> (sfid-20150813_133230_687740_68A6030D) MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: linux-wireless-owner@vger.kernel.org List-ID: Raja Mani writes: > Existing phyerr event handlers directly uses phyerr header format > (ie, struct wmi_phyerr and struct wmi_phyerr_event) in the code > exactly on how firmware packs it. This is the problem in 10.4 fw > specific phyerr event handling where it uses different phyerror > header format. Before adding 10.4 specific handler, little bit of > refactor is done in existing phyerr handlers. > > Two new abstracted structures (struct wmi_phyerr_ev_hdr_arg and > struct wmi_phyerr_ev_arg) are introduced to remove dependency of using > firmware specific header format in the code. So that firmware specific > phyerror handlers can populate values to abstracted structures and > the following code can use abstracted struct for further operation. > > .pull_phyerr_hdr is added newly to pull common phyerr header info > like tsf, buf_len, number of phyerr packed. Existing .pull_phyerr > handler is changed and called to parse every sub phyerrs in the event. > > Validated these refactoring on qca988x hw2.0 using fw 10.2.4 version. > > Signed-off-by: Raja Mani Thanks, applied. -- Kalle Valo