[U-Boot] [PATCH 1/8] tegra: usb: convert USB_PORTS_MAX to be a define

Marek Vasut marex at denx.de
Tue Oct 30 14:11:43 CET 2012


Dear Simon Glass,

> Hi Lucas,
> 
> On Tue, Oct 30, 2012 at 2:22 AM, Lucas Stach <dev at lynxeye.de> wrote:
> > No point in having this as an enum. Also while at it set it to the real
> > hardware maximum for both Tegra 2 and Tegra 3. If new Tegra hardware
> > includes more USB controllers we can always bump the limit then.
> > 
> > Signed-off-by: Lucas Stach <dev at lynxeye.de>
> > ---
> > 
> >  arch/arm/cpu/armv7/tegra20/usb.c | 4 +---
> >  1 Datei geändert, 1 Zeile hinzugefügt(+), 3 Zeilen entfernt(-)
> > 
> > diff --git a/arch/arm/cpu/armv7/tegra20/usb.c
> > b/arch/arm/cpu/armv7/tegra20/usb.c index 1bccf2b..9fd1edc 100644
> > --- a/arch/arm/cpu/armv7/tegra20/usb.c
> > +++ b/arch/arm/cpu/armv7/tegra20/usb.c
> > @@ -43,9 +43,7 @@
> > 
> >         #endif
> >  
> >  #endif
> > 
> > -enum {
> > -       USB_PORTS_MAX   = 4,                    /* Maximum ports we allow
> > */ -};
> > +#define USB_PORTS_MAX  3               /* Maximum ports we allow */
> 
> That's fine with me if you wan to change it.
> 
> I tend to use enums most of the time. It shows up as a symbol in the
> debugger, avoids bracketed expressions, side-effects and the like, and
> works well when numbering multiple things (automatic increment). It's
> also a welcome language feature IMO - use it or lose it :-) But in
> this case the benefit is small.

What about using static const int ?

> >  /* Parameters we need for USB */
> >  enum {
> > 
> > --
> > 1.7.11.7
> 
> Regards,
> Simon

Best regards,
Marek Vasut


More information about the U-Boot mailing list