Patchwork [U-Boot] checkpatch.pl: Add 'debug' to the list of logFunctions

login
register
mail settings
Submitter Tom Rini
Date March 14, 2013, 3:36 p.m.
Message ID <1363275373-8823-1-git-send-email-trini@ti.com>
Download mbox | patch
Permalink /patch/227717/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Tom Rini - March 14, 2013, 3:36 p.m.
While the kernel mainly uses pr_debug(...), etc, for debug messages, we
use debug(...).  Add this to the list of logFunctions so that they are
correctly checked (and not warned against) for long string literals.

Signed-off-by: Tom Rini <trini@ti.com>
---
 tools/checkpatch.pl |    1 +
 1 file changed, 1 insertion(+)
Tom Rini - March 27, 2013, 6:48 p.m.
On Thu, Mar 14, 2013 at 05:36:13AM -0000, Tom Rini wrote:

> While the kernel mainly uses pr_debug(...), etc, for debug messages, we
> use debug(...).  Add this to the list of logFunctions so that they are
> correctly checked (and not warned against) for long string literals.
> 
> Signed-off-by: Tom Rini <trini@ti.com>

Applied to u-boot/master, thanks!

Patch

diff --git a/tools/checkpatch.pl b/tools/checkpatch.pl
index 051ba0d..9f23901 100755
--- a/tools/checkpatch.pl
+++ b/tools/checkpatch.pl
@@ -272,6 +272,7 @@  our $logFunctions = qr{(?x:
 	[a-z0-9]+_(?:printk|emerg|alert|crit|err|warning|warn|notice|info|debug|dbg|vdbg|devel|cont|WARN)(?:_ratelimited|_once|)|
 	WARN(?:_RATELIMIT|_ONCE|)|
 	panic|
+	debug|
 	MODULE_[A-Z_]+
 )};