[U-Boot] [PATCH v2 1/5] usb:gadget:s5p USB Device Controller (UDC) implementation
Scott Wood
scottwood at freescale.com
Mon Oct 10 22:55:56 CEST 2011
On 10/10/2011 03:38 PM, Wolfgang Denk wrote:
> Dear Scott Wood,
>
> In message <4E934EF5.9030302 at freescale.com> you wrote:
>>
>>>> This code is a debug code. As fair as I remember debug print code shall
>>>> not be break in Linux kernel for easier debugging.
>>>
>>> Can you please point me to the respective entry in the CodingStyle
>>> file?
>>
>> Note the last sentence:
> ...
>>> with a long argument list. However, never break user-visible strings such as
>>> printk messages, because that breaks the ability to grep for them.
>
> Well, "user-visible strings" is definitely not the same thing as
> "debug print code", at least not for me.
They're visible to a user that has #defined DEBUG. They're something
one might want to grep on. checkpatch.pl explicitly considers
pr_debug/dev_dbg to be part of the set of functions that can go over the
limit (it just doesn't know about U-Boot's plain old "debug()"). But
whatever...
At some point you might want to consider actually writing down your
interpretations of these things into a U-Boot coding style document, so
at least we don't have to guess.
-Scott
More information about the U-Boot
mailing list