From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wi0-f182.google.com ([209.85.212.182]:35391 "EHLO mail-wi0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751240AbbCXHEd convert rfc822-to-8bit (ORCPT ); Tue, 24 Mar 2015 03:04:33 -0400 Received: by wibdy8 with SMTP id dy8so66728171wib.0 for ; Tue, 24 Mar 2015 00:04:31 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <87lhind7p7.fsf@kamboji.qca.qualcomm.com> References: <1427114301-18886-1-git-send-email-michal.kazior@tieto.com> <1427114301-18886-2-git-send-email-michal.kazior@tieto.com> <87lhind7p7.fsf@kamboji.qca.qualcomm.com> Date: Tue, 24 Mar 2015 08:04:31 +0100 Message-ID: (sfid-20150324_080437_240650_2CB0E2E3) Subject: Re: [PATCH 1/2] ath10k: share board file loading code across FW APIs From: Michal Kazior To: Kalle Valo Cc: "ath10k@lists.infradead.org" , linux-wireless Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 23 March 2015 at 17:01, Kalle Valo wrote: > Michal Kazior writes: > >> There's no need to implement the same thing twice. >> Reduce code duplication. >> >> Signed-off-by: Michal Kazior > > [...] > >> @@ -730,6 +716,12 @@ static int ath10k_core_fetch_firmware_files(struct ath10k *ar) >> /* calibration file is optional, don't check for any errors */ >> ath10k_fetch_cal_file(ar); >> >> + ret = ath10k_core_fetch_board_file(ar); >> + if (ret) { >> + ath10k_err(ar, "failed to fetch board file: %d\n", ret); >> + return ret; >> + } >> + >> ar->fw_api = 4; >> ath10k_dbg(ar, ATH10K_DBG_BOOT, "trying fw api %d\n", ar->fw_api); > > There was a conflict here, please check the resolution in the pending > branch. Looks good, thanks! Michał From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-we0-x236.google.com ([2a00:1450:400c:c03::236]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YaItY-00073X-Bg for ath10k@lists.infradead.org; Tue, 24 Mar 2015 07:05:01 +0000 Received: by wetk59 with SMTP id k59so155864601wet.3 for ; Tue, 24 Mar 2015 00:04:31 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <87lhind7p7.fsf@kamboji.qca.qualcomm.com> References: <1427114301-18886-1-git-send-email-michal.kazior@tieto.com> <1427114301-18886-2-git-send-email-michal.kazior@tieto.com> <87lhind7p7.fsf@kamboji.qca.qualcomm.com> Date: Tue, 24 Mar 2015 08:04:31 +0100 Message-ID: Subject: Re: [PATCH 1/2] ath10k: share board file loading code across FW APIs From: Michal Kazior List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Kalle Valo Cc: linux-wireless , "ath10k@lists.infradead.org" T24gMjMgTWFyY2ggMjAxNSBhdCAxNzowMSwgS2FsbGUgVmFsbyA8a3ZhbG9AcWNhLnF1YWxjb21t LmNvbT4gd3JvdGU6Cj4gTWljaGFsIEthemlvciA8bWljaGFsLmthemlvckB0aWV0by5jb20+IHdy aXRlczoKPgo+PiBUaGVyZSdzIG5vIG5lZWQgdG8gaW1wbGVtZW50IHRoZSBzYW1lIHRoaW5nIHR3 aWNlLgo+PiBSZWR1Y2UgY29kZSBkdXBsaWNhdGlvbi4KPj4KPj4gU2lnbmVkLW9mZi1ieTogTWlj aGFsIEthemlvciA8bWljaGFsLmthemlvckB0aWV0by5jb20+Cj4KPiBbLi4uXQo+Cj4+IEBAIC03 MzAsNiArNzE2LDEyIEBAIHN0YXRpYyBpbnQgYXRoMTBrX2NvcmVfZmV0Y2hfZmlybXdhcmVfZmls ZXMoc3RydWN0IGF0aDEwayAqYXIpCj4+ICAgICAgIC8qIGNhbGlicmF0aW9uIGZpbGUgaXMgb3B0 aW9uYWwsIGRvbid0IGNoZWNrIGZvciBhbnkgZXJyb3JzICovCj4+ICAgICAgIGF0aDEwa19mZXRj aF9jYWxfZmlsZShhcik7Cj4+Cj4+ICsgICAgIHJldCA9IGF0aDEwa19jb3JlX2ZldGNoX2JvYXJk X2ZpbGUoYXIpOwo+PiArICAgICBpZiAocmV0KSB7Cj4+ICsgICAgICAgICAgICAgYXRoMTBrX2Vy cihhciwgImZhaWxlZCB0byBmZXRjaCBib2FyZCBmaWxlOiAlZFxuIiwgcmV0KTsKPj4gKyAgICAg ICAgICAgICByZXR1cm4gcmV0Owo+PiArICAgICB9Cj4+ICsKPj4gICAgICAgYXItPmZ3X2FwaSA9 IDQ7Cj4+ICAgICAgIGF0aDEwa19kYmcoYXIsIEFUSDEwS19EQkdfQk9PVCwgInRyeWluZyBmdyBh cGkgJWRcbiIsIGFyLT5md19hcGkpOwo+Cj4gVGhlcmUgd2FzIGEgY29uZmxpY3QgaGVyZSwgcGxl YXNlIGNoZWNrIHRoZSByZXNvbHV0aW9uIGluIHRoZSBwZW5kaW5nCj4gYnJhbmNoLgoKTG9va3Mg Z29vZCwgdGhhbmtzIQoKCk1pY2hhxYIKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCmF0aDEwayBtYWlsaW5nIGxpc3QKYXRoMTBrQGxpc3RzLmluZnJhZGVh ZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9hdGgxMGsK