* [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
@ 2016-09-14 15:26 zyjzyj2000
2016-09-19 17:00 ` Tantilov, Emil S
0 siblings, 1 reply; 3+ messages in thread
From: zyjzyj2000 @ 2016-09-14 15:26 UTC (permalink / raw)
To: netdev, intel-wired-lan, jeffrey.t.kirsher, zyjzyj2000,
e1000-devel
From: Zhu Yanjun <zyjzyj2000@gmail.com>
Replace defined macro with IS_ENABLED in ixgbe.h file
Signed-off-by: Zhu Yanjun <zyjzyj2000@gmail.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe.h b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
index 9475ff9..f8bc1d0 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe.h
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
@@ -45,7 +45,7 @@
#include "ixgbe_type.h"
#include "ixgbe_common.h"
#include "ixgbe_dcb.h"
-#if defined(CONFIG_FCOE) || defined(CONFIG_FCOE_MODULE)
+#if IS_ENABLED(CONFIG_FCOE)
#define IXGBE_FCOE
#include "ixgbe_fcoe.h"
#endif /* CONFIG_FCOE or CONFIG_FCOE_MODULE */
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
2016-09-14 15:26 [PATCH 1/1] ixgbe: replace defined with IS_ENABLED zyjzyj2000
@ 2016-09-19 17:00 ` Tantilov, Emil S
2016-09-20 13:09 ` zhuyj
0 siblings, 1 reply; 3+ messages in thread
From: Tantilov, Emil S @ 2016-09-19 17:00 UTC (permalink / raw)
To: zyjzyj2000@gmail.com, netdev@vger.kernel.org,
intel-wired-lan@lists.osuosl.org, Kirsher, Jeffrey T,
e1000-devel@lists.sourceforge.net
>-----Original Message-----
>From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] On
>Behalf Of zyjzyj2000@gmail.com
>Sent: Wednesday, September 14, 2016 8:27 AM
>To: netdev@vger.kernel.org; intel-wired-lan@lists.osuosl.org; Kirsher,
>Jeffrey T <jeffrey.t.kirsher@intel.com>; zyjzyj2000@gmail.com; e1000-
>devel@lists.sourceforge.net
>Subject: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
>
>From: Zhu Yanjun <zyjzyj2000@gmail.com>
>
>Replace defined macro with IS_ENABLED in ixgbe.h file
>
>Signed-off-by: Zhu Yanjun <zyjzyj2000@gmail.com>
>---
> drivers/net/ethernet/intel/ixgbe/ixgbe.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>index 9475ff9..f8bc1d0 100644
>--- a/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>@@ -45,7 +45,7 @@
> #include "ixgbe_type.h"
> #include "ixgbe_common.h"
> #include "ixgbe_dcb.h"
>-#if defined(CONFIG_FCOE) || defined(CONFIG_FCOE_MODULE)
>+#if IS_ENABLED(CONFIG_FCOE)
> #define IXGBE_FCOE
> #include "ixgbe_fcoe.h"
> #endif /* CONFIG_FCOE or CONFIG_FCOE_MODULE */
>--
>2.7.4
Already taken care of by the following patch:
http://patchwork.ozlabs.org/patch/668894/
Thanks,
Emil
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
2016-09-19 17:00 ` Tantilov, Emil S
@ 2016-09-20 13:09 ` zhuyj
0 siblings, 0 replies; 3+ messages in thread
From: zhuyj @ 2016-09-20 13:09 UTC (permalink / raw)
To: Tantilov, Emil S
Cc: e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org,
intel-wired-lan@lists.osuosl.org
how about this patch "ixgbe: remove the useless header file ixgbe_type.h"?
On Tue, Sep 20, 2016 at 1:00 AM, Tantilov, Emil S
<emil.s.tantilov@intel.com> wrote:
>>-----Original Message-----
>>From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] On
>>Behalf Of zyjzyj2000@gmail.com
>>Sent: Wednesday, September 14, 2016 8:27 AM
>>To: netdev@vger.kernel.org; intel-wired-lan@lists.osuosl.org; Kirsher,
>>Jeffrey T <jeffrey.t.kirsher@intel.com>; zyjzyj2000@gmail.com; e1000-
>>devel@lists.sourceforge.net
>>Subject: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
>>
>>From: Zhu Yanjun <zyjzyj2000@gmail.com>
>>
>>Replace defined macro with IS_ENABLED in ixgbe.h file
>>
>>Signed-off-by: Zhu Yanjun <zyjzyj2000@gmail.com>
>>---
>> drivers/net/ethernet/intel/ixgbe/ixgbe.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>>diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>>b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>>index 9475ff9..f8bc1d0 100644
>>--- a/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>>+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe.h
>>@@ -45,7 +45,7 @@
>> #include "ixgbe_type.h"
>> #include "ixgbe_common.h"
>> #include "ixgbe_dcb.h"
>>-#if defined(CONFIG_FCOE) || defined(CONFIG_FCOE_MODULE)
>>+#if IS_ENABLED(CONFIG_FCOE)
>> #define IXGBE_FCOE
>> #include "ixgbe_fcoe.h"
>> #endif /* CONFIG_FCOE or CONFIG_FCOE_MODULE */
>>--
>>2.7.4
>
> Already taken care of by the following patch:
> http://patchwork.ozlabs.org/patch/668894/
>
> Thanks,
> Emil
>
------------------------------------------------------------------------------
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-20 13:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-14 15:26 [PATCH 1/1] ixgbe: replace defined with IS_ENABLED zyjzyj2000
2016-09-19 17:00 ` Tantilov, Emil S
2016-09-20 13:09 ` zhuyj
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).