Re: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
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 wrote: >>-Original Message- >>From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org] On >>Behalf Of zyjzyj2...@gmail.com >>Sent: Wednesday, September 14, 2016 8:27 AM >>To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org; Kirsher, >>Jeffrey T ; zyjzyj2...@gmail.com; e1000- >>de...@lists.sourceforge.net >>Subject: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED >> >>From: Zhu Yanjun >> >>Replace defined macro with IS_ENABLED in ixgbe.h file >> >>Signed-off-by: Zhu Yanjun >>--- >> 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 >
RE: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED
>-Original Message- >From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org] On >Behalf Of zyjzyj2...@gmail.com >Sent: Wednesday, September 14, 2016 8:27 AM >To: netdev@vger.kernel.org; intel-wired-...@lists.osuosl.org; Kirsher, >Jeffrey T ; zyjzyj2...@gmail.com; e1000- >de...@lists.sourceforge.net >Subject: [PATCH 1/1] ixgbe: replace defined with IS_ENABLED > >From: Zhu Yanjun > >Replace defined macro with IS_ENABLED in ixgbe.h file > >Signed-off-by: Zhu Yanjun >--- > 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
[PATCH 1/1] ixgbe: replace defined with IS_ENABLED
From: Zhu Yanjun Replace defined macro with IS_ENABLED in ixgbe.h file Signed-off-by: Zhu Yanjun --- 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