Patchwork [6/7] fcoe: remove ETH_P_FCOE from fc_fcoe.h

login
register
mail settings
Submitter Yi Zou
Date Feb. 19, 2009, 7:50 p.m.
Message ID <20090219195022.22270.8543.stgit.yi.zou@intel.com>
Download mbox | patch
Permalink /patch/23455/
State Changes Requested
Delegated to: David Miller
Headers show

Comments

Yi Zou - Feb. 19, 2009, 7:50 p.m.
No need to have ETH_P_FCOE in fc_fcoe.h as it is now defined in if_ether.h

Signed-off-by: Yi Zou <yi.zou@intel.com>
---

 include/scsi/fc/fc_fcoe.h |    7 -------
 1 files changed, 0 insertions(+), 7 deletions(-)


--
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
stephen hemminger - Feb. 19, 2009, 9:25 p.m.
On Thu, 19 Feb 2009 12:50:23 -0700
Yi Zou <yi.zou@intel.com> wrote:

> No need to have ETH_P_FCOE in fc_fcoe.h as it is now defined in if_ether.h
> 
> Signed-off-by: Yi Zou <yi.zou@intel.com>
> ---
> 
>  include/scsi/fc/fc_fcoe.h |    7 -------
>  1 files changed, 0 insertions(+), 7 deletions(-)
> 
> diff --git a/include/scsi/fc/fc_fcoe.h b/include/scsi/fc/fc_fcoe.h
> index f271d9c..ccb3dbe 100644
> --- a/include/scsi/fc/fc_fcoe.h
> +++ b/include/scsi/fc/fc_fcoe.h
> @@ -25,13 +25,6 @@
>   */
>  
>  /*
> - * The FCoE ethertype eventually goes in net/if_ether.h.
> - */
> -#ifndef ETH_P_FCOE
> -#define	ETH_P_FCOE	0x8906		/* FCOE ether type */
> -#endif
> -
>
Just do the move from fc_fcoe.h to if_ether.h in one patch.
--
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
Yi Zou - Feb. 19, 2009, 9:49 p.m.
>On Thu, 19 Feb 2009 12:50:23 -0700
>Yi Zou <yi.zou@intel.com> wrote:
>
>> No need to have ETH_P_FCOE in fc_fcoe.h as it is now defined in
>if_ether.h
>>
>> Signed-off-by: Yi Zou <yi.zou@intel.com>
>> ---
>>
>>  include/scsi/fc/fc_fcoe.h |    7 -------
>>  1 files changed, 0 insertions(+), 7 deletions(-)
>>
>> diff --git a/include/scsi/fc/fc_fcoe.h b/include/scsi/fc/fc_fcoe.h
>> index f271d9c..ccb3dbe 100644
>> --- a/include/scsi/fc/fc_fcoe.h
>> +++ b/include/scsi/fc/fc_fcoe.h
>> @@ -25,13 +25,6 @@
>>   */
>>
>>  /*
>> - * The FCoE ethertype eventually goes in net/if_ether.h.
>> - */
>> -#ifndef ETH_P_FCOE
>> -#define     ETH_P_FCOE      0x8906          /* FCOE ether type */
>> -#endif
>> -
>>
>Just do the move from fc_fcoe.h to if_ether.h in one patch.
Good point, I'll fix that.

Thanks.
yi
--
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

Patch

diff --git a/include/scsi/fc/fc_fcoe.h b/include/scsi/fc/fc_fcoe.h
index f271d9c..ccb3dbe 100644
--- a/include/scsi/fc/fc_fcoe.h
+++ b/include/scsi/fc/fc_fcoe.h
@@ -25,13 +25,6 @@ 
  */
 
 /*
- * The FCoE ethertype eventually goes in net/if_ether.h.
- */
-#ifndef ETH_P_FCOE
-#define	ETH_P_FCOE	0x8906		/* FCOE ether type */
-#endif
-
-/*
  * FC_FCOE_OUI hasn't been standardized yet.   XXX TBD.
  */
 #ifndef FC_FCOE_OUI