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

[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.

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

Regards,
Simon