Return-path: Received: from mail.atheros.com ([12.19.149.2]:14324 "EHLO mail.atheros.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755093Ab1BNOCf (ORCPT ); Mon, 14 Feb 2011 09:02:35 -0500 Received: from mail.atheros.com ([10.10.20.105]) by sidewinder.atheros.com for ; Mon, 14 Feb 2011 06:02:14 -0800 Message-ID: <4D5935EF.1060302@atheros.com> Date: Mon, 14 Feb 2011 19:32:23 +0530 From: Mohammed Shafi MIME-Version: 1.0 To: Felix Fietkau CC: Mohammed Shajakhan , "linux-wireless@vger.kernel.org" , Luis Rodriguez Subject: Re: [PATCH] ath9k: make use of slot time macros References: <1297407704-4029-1-git-send-email-mshajakhan@atheros.com> <4D553107.3020402@openwrt.org> <20110211161551.GA2224@tuxdriver.com> <4D5561F2.3050404@openwrt.org> <20110211162944.GD2224@tuxdriver.com> <4D58B25D.4040507@atheros.com> <4D58B39B.9040207@atheros.com> <4D593295.6060800@openwrt.org> In-Reply-To: <4D593295.6060800@openwrt.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On Monday 14 February 2011 07:18 PM, Felix Fietkau wrote: > On 2011-02-14 5:46 AM, Mohammed Shafi wrote: > >> On Monday 14 February 2011 10:11 AM, Mohammed Shafi wrote: >> >>> On Friday 11 February 2011 09:59 PM, John W. Linville wrote: >>> >>>> On Fri, Feb 11, 2011 at 05:21:06PM +0100, Felix Fietkau wrote: >>>> >>>>> On 2011-02-11 5:15 PM, John W. Linville wrote: >>>>> >>>>>> On Fri, Feb 11, 2011 at 01:52:23PM +0100, Felix Fietkau wrote: >>>>>> >>>>>>> On 2011-02-11 8:01 AM, Mohammed Shafi Shajakhan wrote: >>>>>>> >>>>>>>> From: Mohammed Shafi Shajakhan >>>>>>>> >>>>>>>> Instead of using raw numbers to assign slot time it would be >>>>>>>> better to >>>>>>>> make use of predefined slot time macros >>>>>>>> >>>>>>> How does this make it better? >>>>>>> >>>>>> Maybe if it was ATH9K_SHORT_SLOT_TIME it would make more sense? >>>>>> >>>>> Well, neither the unit of this variable, nor the values that can be >>>>> used >>>>> are ath9k specific. >>>>> >> Felix then I don't know why these macros are used here and I followed >> the same thing: >> >> htc_drv_beacon.c 242 if (ah->slottime == ATH9K_SLOT_TIME_20) >> init.c 517 sc->beacon.slottime = ATH9K_SLOT_TIME_9; >> > Just because the macros are there doesn't mean that it was a good idea > to use them. As far as I know, these were simply inherited from the > Atheros codebase that ath9k was based on. > I actually consider the code more readable without the redundant > "ATH9K_SLOT_TIME_" part. > Felix I agree the first part, but I could still see no harm in using these macros. Initially we using these values 6,9,20(no other values) for the slot time and there are macros defined for them. If we are using some other values I would agree that its wrong. Why not make use of it ? IMHO if we use these macros it will at least people who are reading the code there are three standard values 6,9 and 20 I am sure it would help us to debug issues easily(just like Fair beacon distribution thing). > - Felix > . > >