Re: [U-Boot] [PATCH] ARM: phytec: pcm051: drop unused CONFIG_NET_MULTI
On Mon, Jun 01, 2015 at 01:34:12PM -0500, Joe Hershberger wrote: Hi Lars, On Mon, Jun 1, 2015 at 9:59 AM, poesc...@lemonage.de wrote: From: Lars Poeschel poesc...@lemonage.de Drop unused CONFIG_NET_MULTI. Also drop CONFIG_USE_IRQ, that is not used either. Signed-off-by: Lars Poeschel poesc...@lemonage.de --- include/configs/pcm051.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/configs/pcm051.h b/include/configs/pcm051.h index c0bb227..eef3269 100644 --- a/include/configs/pcm051.h +++ b/include/configs/pcm051.h @@ -163,10 +163,8 @@ #define CONFIG_USB_ETH_RNDIS #endif /* CONFIG_MUSB_GADGET */ -/* Unsupported features */ -#undef CONFIG_USE_IRQ These are unrelated changes. They should be two separate patches. Also, you say this is unused, but you are editing an ARM board and it seems that the arch/arm/lib/ source does refer to this. Are you sure it is unused? If so, Please make a separate patch with more explanation instead of adding it into another patch as a me-too. Well, I am quite shure this not needed anymore. This is an undef and CONFIG_USE_IRQ is not defined in the include path before. I just tested it and with or without the undef the same code is compiled. But your are right: These changes are not related. I will make two seperate patches. -#define CONFIG_NET_MULTI Since this is not used anywhere, can you remove it from all of the configs that still define it all in the same patch? Yes, I will do that. Lars ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] ARM: phytec: pcm051: drop unused CONFIG_NET_MULTI
From: Lars Poeschel poesc...@lemonage.de Drop unused CONFIG_NET_MULTI. Also drop CONFIG_USE_IRQ, that is not used either. Signed-off-by: Lars Poeschel poesc...@lemonage.de --- include/configs/pcm051.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/configs/pcm051.h b/include/configs/pcm051.h index c0bb227..eef3269 100644 --- a/include/configs/pcm051.h +++ b/include/configs/pcm051.h @@ -163,10 +163,8 @@ #define CONFIG_USB_ETH_RNDIS #endif /* CONFIG_MUSB_GADGET */ -/* Unsupported features */ -#undef CONFIG_USE_IRQ -#define CONFIG_NET_MULTI +/* Network. */ #define CONFIG_PHY_GIGE #define CONFIG_PHYLIB #define CONFIG_PHY_SMSC -- 2.1.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ARM: phytec: pcm051: drop unused CONFIG_NET_MULTI
Hi Lars, On Mon, Jun 1, 2015 at 9:59 AM, poesc...@lemonage.de wrote: From: Lars Poeschel poesc...@lemonage.de Drop unused CONFIG_NET_MULTI. Also drop CONFIG_USE_IRQ, that is not used either. Signed-off-by: Lars Poeschel poesc...@lemonage.de --- include/configs/pcm051.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/configs/pcm051.h b/include/configs/pcm051.h index c0bb227..eef3269 100644 --- a/include/configs/pcm051.h +++ b/include/configs/pcm051.h @@ -163,10 +163,8 @@ #define CONFIG_USB_ETH_RNDIS #endif /* CONFIG_MUSB_GADGET */ -/* Unsupported features */ -#undef CONFIG_USE_IRQ These are unrelated changes. They should be two separate patches. Also, you say this is unused, but you are editing an ARM board and it seems that the arch/arm/lib/ source does refer to this. Are you sure it is unused? If so, Please make a separate patch with more explanation instead of adding it into another patch as a me-too. -#define CONFIG_NET_MULTI Since this is not used anywhere, can you remove it from all of the configs that still define it all in the same patch? +/* Network. */ #define CONFIG_PHY_GIGE #define CONFIG_PHYLIB #define CONFIG_PHY_SMSC Thanks, -Joe ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot