Home » Mailing list archives » linux-netdev » 2008 » June » 16

[PATCH] 8139too: add missing brackets

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Németh Márton
Subject: [PATCH] 8139too: add missing brackets
Date: Sunday, June 15, 2008 - 10:09 pm

From: Márton Németh <nm127@freemail.hu>

Clean up the following error reported by checkpatch.pl:
 - Macros with complex values should be enclosed in parenthesis

The macro RTL8139_CAPS currently only used as a whole expression
to initialize board_info[] so this change won't change the resulting code.

Signed-off-by: Márton Németh <nm127@freemail.hu>
---
--- a/drivers/net/8139too.c	2008-06-14 00:20:47.000000000 +0200
+++ b/drivers/net/8139too.c	2008-06-16 07:03:20.000000000 +0200
@@ -219,7 +219,7 @@ enum {
 #define RTL8139B_IO_SIZE 256

 #define RTL8129_CAPS	HAS_MII_XCVR
-#define RTL8139_CAPS	HAS_CHIP_XCVR|HAS_LNK_CHNG
+#define RTL8139_CAPS	(HAS_CHIP_XCVR|HAS_LNK_CHNG)

 typedef enum {
 	RTL8139 = 0,
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 2/2] 8139too: different style cleanups, Németh Márton, (Sat Jun 14, 5:37 am)
Re: [PATCH 2/2] 8139too: different style cleanups, Németh Márton, (Sat Jun 14, 11:44 am)
Re: [PATCH 2/2] 8139too: different style cleanups, David Newall, (Sat Jun 14, 11:26 pm)
Re: [PATCH 2/2] 8139too: different style cleanups, Harvey Harrison, (Sat Jun 14, 11:50 pm)
[PATCH] 8139too: add missing brackets, Németh Márton, (Sun Jun 15, 10:09 pm)