From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH v2 4/4] checkpatch: apply barebox-specific modifications
Date: Wed, 15 May 2019 11:36:44 +0200 [thread overview]
Message-ID: <20190515093644.9691-5-a.fatoum@pengutronix.de> (raw)
In-Reply-To: <20190515093644.9691-1-a.fatoum@pengutronix.de>
For future reference, following barebox changes were applied
on top of the scripts/checkpatch.pl copied from upstream:
4a7f56056d ("scripts: Adapt checkpatch.pl for barebox.")
2671c30c25 ("scripts/checkpatch.pl: don't search for Doxyfile when checking top_of_kernel_tree()")
ad3c55fbf2 ("scripts/checkpatch.pl: don't search for CREDITS when checking top_of_kernel_tree()")
13b0f4666d ("checkpatch: don't check TODO file presence")
These adjust sub top_of_kernel_tree's @tree_check to eventually contain
(
"arch", "commands", "common", "COPYING", "defaultenv",
"Documentation", "drivers", "fs", "include", "lib",
"MAKEALL", "Makefile", "net", "README", "scripts"
);
9e809ef431 ("scripts: allow lines longer than 80 cols with printf() in checkpatch")
adds an exception for printf, same as printk.
9496896fbf ("checkpatch: add DT compatible string documentation checks")
searches dts/Bindings as well as Documentation/devicetree/bindings
for compatibles.
Cc: Antony Pavlov <antonynpavlov@gmail.com>
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
scripts/checkpatch.pl | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index a09333fd7cef..65c2cfad45f7 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -457,7 +457,7 @@ our $typeTypedefs = qr{(?x:
our $zero_initializer = qr{(?:(?:0[xX])?0+$Int_type?|NULL|false)\b};
our $logFunctions = qr{(?x:
- printk(?:_ratelimited|_once|_deferred_once|_deferred|)|
+ printf|printk(?:_ratelimited|_once|_deferred_once|_deferred|)|
(?:[a-z0-9]+_){1,2}(?:printk|emerg|alert|crit|err|warning|warn|notice|info|debug|dbg|vdbg|devel|cont|WARN)(?:_ratelimited|_once|)|
TP_printk|
WARN(?:_RATELIMIT|_ONCE|)|
@@ -1095,9 +1095,9 @@ sub top_of_kernel_tree {
my ($root) = @_;
my @tree_check = (
- "COPYING", "CREDITS", "Kbuild", "MAINTAINERS", "Makefile",
- "README", "Documentation", "arch", "include", "drivers",
- "fs", "init", "ipc", "kernel", "lib", "scripts",
+ "arch", "commands", "common", "COPYING", "defaultenv",
+ "Documentation", "drivers", "fs", "include", "lib",
+ "MAKEALL", "Makefile", "net", "README", "scripts"
);
foreach my $check (@tree_check) {
@@ -3008,8 +3008,12 @@ sub process {
my @compats = $rawline =~ /\"([a-zA-Z0-9\-\,\.\+_]+)\"/g;
- my $dt_path = $root . "/Documentation/devicetree/bindings/";
- my $vp_file = $dt_path . "vendor-prefixes.txt";
+ # linux device tree files
+ my $dt_path = $root . "/dts/Bindings/";
+ my $vp_file = $dt_path . "vendor-prefixes.txt";
+
+ # barebox-specific bindings
+ $dt_path = $dt_path . " " . $root . "/Documentation/devicetree/bindings/";
foreach my $compat (@compats) {
my $compat2 = $compat;
--
2.20.1
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2019-05-15 9:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-15 9:36 [PATCH v2 0/4] scripts/checkpatch.pl: rebase on top of upstream v5.1 Ahmad Fatoum
2019-05-15 9:36 ` [PATCH v2 1/4] LICENSES: remove empty other, exceptions directories Ahmad Fatoum
2019-05-15 9:36 ` [PATCH v2 2/4] checkpatch: add upstream checkpatch v5.1 dependencies Ahmad Fatoum
2019-05-15 9:36 ` [PATCH v2 3/4] checkpatch: import as-is from Linux v5.1 Ahmad Fatoum
2019-05-15 9:36 ` Ahmad Fatoum [this message]
2019-05-21 7:19 ` [PATCH v2 0/4] scripts/checkpatch.pl: rebase on top of upstream v5.1 Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190515093644.9691-5-a.fatoum@pengutronix.de \
--to=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox