[U-Boot] [RFC PATCH] USB style patch

Remy Bohmer linux at bohmer.net
Wed Nov 26 12:15:00 CET 2008


Hello Michael,

Nice work, but I have a few small comments:

> +                       case USB_PROT_HID_MOUSE:
> +                               printf("Mouse");
> +                               break;
> +                       default:
> +                               printf("reserved");
>                        }
>                        break;

A default case without a break...

> +               case US_SC_SCSI:
> +                       printf("Transp. SCSI");
>                        break;
>                default:
> -                       printf("%s",usb_get_class_desc(dclass));
> +                       printf("reserved");
> +                       break;

And a default case with a break...
I see the 'break' keyword is not being used very consequent across all
these files/switch-statements.

> -               printf("  NOTE: this command is obsolete and will be phased out\n");
> -               printf("  please use 'usb storage' for USB storage devices information\n\n");
> +               printf("  NOTE: this command is obsolete and will be"
> +                       " phased out\n");
> +               printf("  please use 'usb storage' for USB storage devices"
> +                       " information\n\n");

I believe it is good to keep line lengths below 80 characters in
general, but code should not become less readable by this.
In case of printf lines, it becomes more difficult to 'grep' for the
strings printed in the terminal to find them back in the code.
So, I am very curious about the point of view Wolfgang about this
rule. (How strict does this rule need to be applied?)

Kind Regards,

Remy


More information about the U-Boot mailing list