Return-path: Received: from wolverine02.qualcomm.com ([199.106.114.251]:25591 "EHLO wolverine02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751098AbcJCHw5 (ORCPT ); Mon, 3 Oct 2016 03:52:57 -0400 From: "Valo, Kalle" To: "bartosz.markowski@tieto.com" CC: "linux-wireless@vger.kernel.org" , "ath10k@lists.infradead.org" Subject: Re: [PATCH 2/2] ath10k: add platform regulatory domain support Date: Mon, 3 Oct 2016 07:52:51 +0000 Message-ID: <877f9pswbx.fsf@kamboji.qca.qualcomm.com> (sfid-20161003_095300_555462_1C15D756) References: <1470140454-3899-1-git-send-email-bartosz.markowski@tieto.com> <1470140454-3899-3-git-send-email-bartosz.markowski@tieto.com> <87bmztm8qm.fsf@kamboji.qca.qualcomm.com> <871t0nhsdu.fsf@kamboji.qca.qualcomm.com> In-Reply-To: (Bartosz Markowski's message of "Mon, 3 Oct 2016 09:43:17 +0200") Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: Bartosz Markowski writes: > On 14 September 2016 at 09:06, Valo, Kalle wrote= : >> >> Bartosz Markowski writes: >> >> > On 12 September 2016 at 17:35, Valo, Kalle wr= ote: >> > >> >> > +#ifdef CONFIG_ACPI >> >> > +#define WRD_METHOD "WRDD" >> >> > +#define WRDD_WIFI (0x07) >> >> > + >> >> > +static u32 ath10k_mac_wrdd_get_mcc(struct ath10k *ar, union acpi_o= bject *wrdd) >> >> > +{ >> >> >> >> I don't think the ifdef is really necessary, acpi.h should handle tha= t >> >> (hopefully). Also I changed the error handling to use standard error >> >> values and changed the info messages to dbg, they are too spammy in m= y >> >> opinion. Please check carefully my changes in the pending branch: >> >> >> >> https://git.kernel.org/cgit/linux/kernel/git/kvalo/ath.git/commit/?h= =3Dpending&id=3Dfe91745381ec3999d8de6dedb07b396c82539717 >> > >> > I'm OK with the changes, I have not tried that though, except of >> > reviewing and compiling it (do not have access to the chromebook for >> > next few days). If you want to wait with it until I test it, it's fine >> > too. >> >> Ok, I'll wait for few days in case you have time to test it. > > > Sorry, it took so long. I've final check this and can confirm the patch. Thanks, I'll apply the patch soon. Do note that I changed more of the warnings messages to debug level. --=20 Kalle Valo=