Re: [trivial PATCH 2/7] ixgbe: remove unused #define

4 messages, 2 authors, 2012-07-12 · open the first message on its own page

Re: [trivial PATCH 2/7] ixgbe: remove unused #define

From: Jeff Kirsher <hidden>
Date: 2012-07-10 22:43:32

On Tue, 2012-07-10 at 15:31 -0700, Jon Mason wrote:
Remove unused IXGBE_INTEL_VENDOR_ID #define

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Jeff Kirsher <redacted>
Cc: Jesse Brandeburg <redacted>
Cc: Bruce Allan <redacted>
Cc: Carolyn Wyborny <redacted>
Cc: Don Skidmore <redacted>
Cc: Greg Rose <redacted>
Cc: Peter P Waskiewicz Jr <redacted>
Cc: Alex Duyck <redacted>
Cc: John Ronciak <redacted>
---
 drivers/net/ethernet/intel/ixgbe/ixgbe_type.h |    3 ---
 1 file changed, 3 deletions(-)
This should also go through David Miller's networking tree's.  Adding
netdev mailing list.

Acked-by: Jeff Kirsher <redacted>
quoted hunk
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
index 204848d..c8d8040 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
@@ -32,9 +32,6 @@
 #include <linux/mdio.h>
 #include <linux/netdevice.h>
 
-/* Vendor ID */
-#define IXGBE_INTEL_VENDOR_ID   0x8086
-
 /* Device IDs */
 #define IXGBE_DEV_ID_82598               0x10B6
 #define IXGBE_DEV_ID_82598_BX            0x1508

Re: [trivial PATCH 2/7] ixgbe: remove unused #define

From: Jeff Kirsher <hidden>
Date: 2012-07-12 18:15:43

On Tue, 2012-07-10 at 15:43 -0700, Jeff Kirsher wrote:
On Tue, 2012-07-10 at 15:31 -0700, Jon Mason wrote:
quoted
Remove unused IXGBE_INTEL_VENDOR_ID #define

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Jeff Kirsher <redacted>
Cc: Jesse Brandeburg <redacted>
Cc: Bruce Allan <redacted>
Cc: Carolyn Wyborny <redacted>
Cc: Don Skidmore <redacted>
Cc: Greg Rose <redacted>
Cc: Peter P Waskiewicz Jr <redacted>
Cc: Alex Duyck <redacted>
Cc: John Ronciak <redacted>
---
 drivers/net/ethernet/intel/ixgbe/ixgbe_type.h |    3 ---
 1 file changed, 3 deletions(-)
This should also go through David Miller's networking tree's.  Adding
netdev mailing list.

Acked-by: Jeff Kirsher <redacted>
After looking at this further, I rescind my ACK.

NAK, this define is used in several places in ixgbe_main.c and the
driver will not compile with this patch.
quoted
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
index 204848d..c8d8040 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
@@ -32,9 +32,6 @@
 #include <linux/mdio.h>
 #include <linux/netdevice.h>
 
-/* Vendor ID */
-#define IXGBE_INTEL_VENDOR_ID   0x8086
-
 /* Device IDs */
 #define IXGBE_DEV_ID_82598               0x10B6
 #define IXGBE_DEV_ID_82598_BX            0x1508

Re: [trivial PATCH 2/7] ixgbe: remove unused #define

From: Jon Mason <jdmason@kudzu.us>
Date: 2012-07-12 19:49:49

On Thu, Jul 12, 2012 at 11:15 AM, Jeff Kirsher
[off-list ref] wrote:
On Tue, 2012-07-10 at 15:43 -0700, Jeff Kirsher wrote:
quoted
On Tue, 2012-07-10 at 15:31 -0700, Jon Mason wrote:
quoted
Remove unused IXGBE_INTEL_VENDOR_ID #define

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Jeff Kirsher <redacted>
Cc: Jesse Brandeburg <redacted>
Cc: Bruce Allan <redacted>
Cc: Carolyn Wyborny <redacted>
Cc: Don Skidmore <redacted>
Cc: Greg Rose <redacted>
Cc: Peter P Waskiewicz Jr <redacted>
Cc: Alex Duyck <redacted>
Cc: John Ronciak <redacted>
---
 drivers/net/ethernet/intel/ixgbe/ixgbe_type.h |    3 ---
 1 file changed, 3 deletions(-)
This should also go through David Miller's networking tree's.  Adding
netdev mailing list.

Acked-by: Jeff Kirsher <redacted>
After looking at this further, I rescind my ACK.

NAK, this define is used in several places in ixgbe_main.c and the
driver will not compile with this patch.
My apologies, I was a bit sloppy and simply rebased my old patches
without retesting them.  The code has changed since I originally
pushed the patch (http://www.spinics.net/lists/netdev/msg171523.html).
 I will fix and resubmit.

Thanks,
Jon

quoted
quoted
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
index 204848d..c8d8040 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_type.h
@@ -32,9 +32,6 @@
 #include <linux/mdio.h>
 #include <linux/netdevice.h>

-/* Vendor ID */
-#define IXGBE_INTEL_VENDOR_ID   0x8086
-
 /* Device IDs */
 #define IXGBE_DEV_ID_82598               0x10B6
 #define IXGBE_DEV_ID_82598_BX            0x1508

Re: [trivial PATCH 2/7] ixgbe: remove unused #define

From: Jeff Kirsher <hidden>
Date: 2012-07-12 19:58:36

On Thu, 2012-07-12 at 12:49 -0700, Jon Mason wrote:
On Thu, Jul 12, 2012 at 11:15 AM, Jeff Kirsher
[off-list ref] wrote:
quoted
On Tue, 2012-07-10 at 15:43 -0700, Jeff Kirsher wrote:
quoted
On Tue, 2012-07-10 at 15:31 -0700, Jon Mason wrote:
quoted
Remove unused IXGBE_INTEL_VENDOR_ID #define

Signed-off-by: Jon Mason <jdmason@kudzu.us>
Cc: Jeff Kirsher <redacted>
Cc: Jesse Brandeburg <redacted>
Cc: Bruce Allan <redacted>
Cc: Carolyn Wyborny <redacted>
Cc: Don Skidmore <redacted>
Cc: Greg Rose <redacted>
Cc: Peter P Waskiewicz Jr <redacted>
Cc: Alex Duyck <redacted>
Cc: John Ronciak <redacted>
---
 drivers/net/ethernet/intel/ixgbe/ixgbe_type.h |    3 ---
 1 file changed, 3 deletions(-)
This should also go through David Miller's networking tree's.
Adding
quoted
quoted
netdev mailing list.

Acked-by: Jeff Kirsher <redacted>
After looking at this further, I rescind my ACK.

NAK, this define is used in several places in ixgbe_main.c and the
driver will not compile with this patch.
My apologies, I was a bit sloppy and simply rebased my old patches
without retesting them.  The code has changed since I originally
pushed the patch (http://www.spinics.net/lists/netdev/msg171523.html).
 I will fix and resubmit.

Thanks,
Jon 
Also note, that when you fix up the patch and resend it, could you
resend patch 1 (ixgb) as well to netdev@vger.kernel.org.  That way
netdev patchwork will pickup the patches and I can get them into my
queue.

Thanks!
Jeff

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help