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