2010-03-30 15:44:42

by Luis de Bethencourt

Subject: [PATCH 2/2] ath: fix code readability in regd.c

This is a patch to the ath/regd.c file that fixes two code
readability issues. A space between to separate two defines
and the indentation inside the ath_redg_is_eeprom_valid
function.

Signed-off-by: Luis de Bethencourt <[email protected]>
---
drivers/net/wireless/ath/regd.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/net/wireless/ath/regd.c b/drivers/net/wireless/ath/regd.c
index 04abd1f..d5c2332 100644
--- a/drivers/net/wireless/ath/regd.c
+++ b/drivers/net/wireless/ath/regd.c
@@ -51,6 +51,7 @@

#define ATH9K_5GHZ_ALL ATH9K_5GHZ_5150_5350, \
ATH9K_5GHZ_5470_5850
+
/* This one skips what we call "mid band" */
#define ATH9K_5GHZ_NO_MIDBAND ATH9K_5GHZ_5150_5350, \
ATH9K_5GHZ_5725_5850
@@ -361,7 +362,7 @@ EXPORT_SYMBOL(ath_reg_notifier_apply);

static bool ath_regd_is_eeprom_valid(struct ath_regulatory *reg)
{
- u16 rd = ath_regd_get_eepromRD(reg);
+ u16 rd = ath_regd_get_eepromRD(reg);
int i;

if (rd & COUNTRY_ERD_FLAG) {
--
1.6.3.3

2010-03-30 16:56:47

by Luis R. Rodriguez

Subject: Re: [PATCH 2/2] ath: fix code readability in regd.c

On Tue, Mar 30, 2010 at 08:44:33AM -0700, Luis de Bethencourt wrote:
> This is a patch to the ath/regd.c file that fixes two code
> readability issues. A space between to separate two defines
> and the indentation inside the ath_redg_is_eeprom_valid
> function.

Acked-by: Luis R. Rodriguez <[email protected]>

You can drop netdev and linux-kernel from future linux-wireless e-mails
like this one.

Luis

2010-03-30 17:08:29

by Luis de Bethencourt

Subject: Re: [PATCH 2/2] ath: fix code readability in regd.c

On Tue, Mar 30, 2010 at 5:56 PM, Luis R. Rodriguez
<[email protected]> wrote:
> On Tue, Mar 30, 2010 at 08:44:33AM -0700, Luis de Bethencourt wrote:
>> This is a patch to the ath/regd.c file that fixes two code
>> readability issues. A space between to separate two defines
>> and the indentation inside the ath_redg_is_eeprom_valid
>> function.
>
> Acked-by: Luis R. Rodriguez <[email protected]>
>
> You can drop netdev and linux-kernel from future linux-wireless e-mails
> like this one.
>
> ?Luis
>

OK. Will do.

Luis