From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: Pavel Roskin <proski@gnu.org>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
"linville@tuxdriver.com" <linville@tuxdriver.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"ath9k-devel@lists.ath9k.org" <ath9k-devel@lists.ath9k.org>,
Jouni Malinen <Jouni.Malinen@Atheros.com>,
Sujith Manoharan <Sujith.Manoharan@Atheros.com>
Subject: Re: [PATCH 11/16] ath9k: Remove "OS_" prefixes in macros.
Date: Wed, 30 Jul 2008 19:06:17 -0700 [thread overview]
Message-ID: <20080731020617.GJ7625@tesla> (raw)
In-Reply-To: <1217468689.17029.7.camel@dv>
On Wed, Jul 30, 2008 at 06:44:49PM -0700, Pavel Roskin wrote:
> On Wed, 2008-07-30 at 18:23 -0700, Luis R. Rodriguez wrote:
> > On Wed, Jul 30, 2008 at 6:16 PM, Pavel Roskin <proski@gnu.org> wrote:
> > > On Mon, 2008-07-28 at 22:47 -0700, lrodriguez@atheros.com wrote:
> > >> From: Sujith Manoharan <smanoharan@atheros.com>
> > >>
> > >> diff --git a/drivers/net/wireless/ath9k/ath9k.h b/drivers/net/wireless/ath9k/ath9k.h
> > >> index 6fa1eb6..c3294c0 100644
> > >> --- a/drivers/net/wireless/ath9k/ath9k.h
> > >> +++ b/drivers/net/wireless/ath9k/ath9k.h
> > >> @@ -622,17 +622,15 @@ struct hal_country_entry {
> > >>
> > >> #define SM(_v, _f) (((_v) << _f##_S) & _f)
> > >> #define MS(_v, _f) (((_v) & _f) >> _f##_S)
> > >> -#define OS_REG_RMW(_a, _r, _set, _clr) \
> > >> +#define REG_RMW(_a, _r, _set, _clr) \
> > >> REG_WRITE(_a, _r, (REG_READ(_a, _r) & ~(_clr)) | (_set))
> > >
> > > I think these are pretty good candidates for inline functions. I'm not
> > > sure removing "OS_" is a useful intermediate step. It's better to keep
> > > such massive changes to the minimum to avoid potential conflicts with
> > > other patches.
> >
> > This was done in response to Johannes comments. This can be moved to
> > inline though.
>
> It would be really nice. Explicit return types tend to unmask some
> issues in the calling code, if there are any.
>
> I don't care if you are going to change the macros/functions themselves
> in many steps, but let's avoid the need to rename all callers again.
Agreed.
Luis
next prev parent reply other threads:[~2008-07-31 2:06 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-29 5:47 [PATCH 00/16] ath9k: some cleanups, and a few small fixes lrodriguez
2008-07-29 5:47 ` [PATCH 02/16] ath9k: Remove ineffective multirate retry stuff lrodriguez
2008-07-29 5:47 ` [PATCH 03/16] ath9k: Clean up in antenna diversity handling. Get rid of sc_needmib lrodriguez
2008-07-29 5:47 ` [PATCH 04/16] ath9k: Use struct ath_softc directly instead of a void pointer lrodriguez
2008-07-29 5:47 ` [PATCH 05/16] ath9k: Use 'ATH9K' as a prefix instead of 'HAL' for a lot of macros lrodriguez
2008-07-29 5:47 ` [PATCH 06/16] ath9k: Fix missing brace in branch on ath_chainmask_sel_logic() lrodriguez
2008-07-31 1:11 ` Pavel Roskin
2008-07-31 1:21 ` Luis R. Rodriguez
2008-07-31 1:41 ` Pavel Roskin
2008-07-29 5:47 ` [PATCH 07/16] ath9k: Replace enum hal_bool with bool lrodriguez
2008-07-29 5:47 ` [PATCH 08/16] ath9k: Revamp debugging lrodriguez
2008-07-29 5:47 ` [PATCH 09/16] ath9k: Use standard ISR return values, merge ath_intr and ath_isr lrodriguez
2008-07-29 5:47 ` [PATCH 10/16] ath9k: Don't pass opmode twice to ath_vap_attach, return standard error values from add_interface lrodriguez
2008-07-29 5:47 ` [PATCH 11/16] ath9k: Remove "OS_" prefixes in macros lrodriguez
2008-07-31 1:16 ` Pavel Roskin
2008-07-31 1:23 ` Luis R. Rodriguez
2008-07-31 1:44 ` Pavel Roskin
2008-07-31 2:06 ` Luis R. Rodriguez [this message]
2008-07-29 5:47 ` [PATCH 12/16] ath9k: Dont maintain some of h/w caps in ath_softc, access them directly from ah_caps instead lrodriguez
2008-07-29 5:47 ` [PATCH 13/16] ath9k: Break down ath9k_regd_init_channels and use helper functions lrodriguez
2008-07-29 5:47 ` [PATCH 14/16] ath9k: Remove AH_DEBUG_COUNTRY lrodriguez
2008-07-29 5:47 ` [PATCH 15/16] ath9k: Don't set the channel in ath_rate_init() lrodriguez
2008-07-29 5:47 ` [PATCH 16/16] ath9k: Remove some unused macros lrodriguez
2008-07-29 5:50 ` [PATCH 00/16] ath9k: some cleanups, and a few small fixes Luis R. Rodriguez
2008-07-29 5:52 ` Luis R. Rodriguez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080731020617.GJ7625@tesla \
--to=lrodriguez@atheros.com \
--cc=Jouni.Malinen@Atheros.com \
--cc=Luis.Rodriguez@Atheros.com \
--cc=Sujith.Manoharan@Atheros.com \
--cc=ath9k-devel@lists.ath9k.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=proski@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).