From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751763AbcFERkj (ORCPT ); Sun, 5 Jun 2016 13:40:39 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:33053 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751348AbcFERkh (ORCPT ); Sun, 5 Jun 2016 13:40:37 -0400 Message-ID: <57546409.70506@gmail.com> Date: Sun, 05 Jun 2016 23:10:25 +0530 From: Sudip Mukherjee User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Kalle Valo , "Pan, Miaoqing" CC: Stephen Rothwell , ath9k-devel , "linux-next@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-wireless@vger.kernel.org" , "ath9k-devel@lists.ath9k.org" , "netdev@vger.kernel.org" , Miaoqing Pan Subject: Re: ath9k gpio request References: <20160530132225.4d297678@canb.auug.org.au> <574C4851.9090201@gmail.com> <87fusy3cqx.fsf_-_@kamboji.qca.qualcomm.com> <574D3EAA.3060102@gmail.com> <1464764036580.70506@qti.qualcomm.com> <574EC331.1080901@gmail.com> <574ED297.4080004@gmail.com> <1464854569880.79341@qti.qualcomm.com> <57501AE4.7080104@gmail.com> <8760tqn8fc.fsf@purkki.adurom.net> <1464932944074.85131@qti.qualcomm.com> <87d1nxt3y9.fsf@kamboji.qca.qualcomm.com> In-Reply-To: <87d1nxt3y9.fsf@kamboji.qca.qualcomm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Saturday 04 June 2016 08:07 PM, Kalle Valo wrote: > (Fixing top posting) > > "Pan, Miaoqing" writes: > >>>> --- a/drivers/net/wireless/ath/ath9k/reg.h >>>> +++ b/drivers/net/wireless/ath/ath9k/reg.h >>>> @@ -1122,8 +1122,8 @@ enum { >>>> #define AR9300_NUM_GPIO 16 >>>> #define AR9330_NUM_GPIO 16 >>>> #define AR9340_NUM_GPIO 23 >>>> -#define AR9462_NUM_GPIO 10 >>>> -#define AR9485_NUM_GPIO 12 >>>> +#define AR9462_NUM_GPIO 14 >>>> +#define AR9485_NUM_GPIO 11 >>>> #define AR9531_NUM_GPIO 18 >>>> #define AR9550_NUM_GPIO 24 >>>> #define AR9561_NUM_GPIO 23 >>>> @@ -1139,8 +1139,8 @@ enum { >>>> #define AR9300_GPIO_MASK 0x0000F4FF >>>> #define AR9330_GPIO_MASK 0x0000F4FF >>>> #define AR9340_GPIO_MASK 0x0000000F >>>> -#define AR9462_GPIO_MASK 0x000003FF >>>> -#define AR9485_GPIO_MASK 0x00000FFF >>>> +#define AR9462_GPIO_MASK 0x00003FFF >>>> +#define AR9485_GPIO_MASK 0x000007FF >>>> #define AR9531_GPIO_MASK 0x0000000F >>>> #define AR9550_GPIO_MASK 0x0000000F >>>> #define AR9561_GPIO_MASK 0x0000000F >>> >>> solves the problem. >>> >>> Tested-by: Sudip Mukherjee >> >> Done, https://patchwork.kernel.org/patch/9151847/ > > But the patch 9151847 is different from what Sudip tested above? Why? > > And if you modify something _after_ the reporter has tested the patch > clearly document what you changed and why. I do not want find hidden > changes like this, even more so when the patch is going to a 4.7-rc > release. > > Sudip, could you also test patch 9151847, please? You can download the > patch from the patchwork link above. This is also ok. Please add my Tested-by: Sudip Mukherjee and maybe a Reported-by tag is also appropriate in this case. Regards Sudip