[U-Boot] [PATCH] ARM: phytec: pcm051: drop unused CONFIG_NET_MULTI

[Lars Poeschel]

On Mon, Jun 01, 2015 at 01:34:12PM -0500, Joe Hershberger wrote:
> Hi Lars,
> 
> On Mon, Jun 1, 2015 at 9:59 AM,  <poeschel at lemonage.de> wrote:
> > From: Lars Poeschel <poeschel at lemonage.de>
> >
> > Drop unused CONFIG_NET_MULTI.
> > Also drop CONFIG_USE_IRQ, that is not used either.
> >
> > Signed-off-by: Lars Poeschel <poeschel at 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