[U-Boot] [PATCH 3/8] USB-CDC: Linux-like debug printout

Vitaly Kuzmichev vkuzmichev at mvista.com
Fri Aug 13 12:17:45 CEST 2010


Hi Sergei,

Sergei Shtylyov wrote:
>> -            printf ("enable %s --> %d\n",
>> +            DEBUG(dev, "enable %s --> %d\n",
> 
>   Well, I think the coding style shouldbe consistent -- you either leave
> the space before ( or remove it. And as U-Boot seems to follow the Linux
> coding style now, it seems better to remove the space.

Well, I would say: "either leave the space before ( or remove it from
EVERYWHERE".
I think that the last thing should be done as a separate patch.
So I would like to keep the coding style that is in this file.


Best regards,
Vitaly.


More information about the U-Boot mailing list