From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Dwaipayan Ray <dwaipayanray1@gmail.com>,
Lukas Bulwahn <lukas.bulwahn@gmail.com>,
Joe Perches <joe@perches.com>,
Andy Whitcroft <apw@canonical.com>
Cc: Jonathan Corbet <corbet@lwn.net>,
Shuah Khan <skhan@linuxfoundation.org>,
workflows@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: [PATCH] checkpatch: allow correctly handle full files on stdin
Date: Wed, 25 Mar 2026 23:20:34 -0700 [thread overview]
Message-ID: <acTPXMJfkHLeItrK@google.com> (raw)
checkpatch does not handle full files well when they are passed on
stdin, because it does not know how to treat the text, and whether it is
a C file, or a DTS file, or something else, and so it assumes that when
it works with stdin it should be a unified diff. For full files it
expects to have a file name as an argument and read the contents from
disk. Unfortunately this does not well when trying to use checkpatch as
an online linter and feed it contents of an editor buffer that have not
made it to the disk yet.
To solve the above introduce a new optional argument --stdin-filename=FILE
that allows tell checkpatch the kind of file it is dealing with and
apply appropriate set of checks and rules to it.
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
Documentation/dev-tools/checkpatch.rst | 4 ++++
scripts/checkpatch.pl | 11 +++++++++++
2 files changed, 15 insertions(+)
diff --git a/Documentation/dev-tools/checkpatch.rst b/Documentation/dev-tools/checkpatch.rst
index dccede68698c..b521e3ca6ebf 100644
--- a/Documentation/dev-tools/checkpatch.rst
+++ b/Documentation/dev-tools/checkpatch.rst
@@ -68,6 +68,10 @@ Available options:
Show the diffed file position instead of the input file position.
+ - --stdin-filename
+
+ When using stdin, identify the file as FILE.
+
- -g, --git
Treat FILE as a single commit or a git revision range.
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index e56374662ff7..e26951ceb36b 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -57,6 +57,7 @@ my %ignore_type = ();
my @ignore = ();
my $help = 0;
my $configuration_file = ".checkpatch.conf";
+my $stdin_filename;
my $max_line_length = 100;
my $ignore_perl_version = 0;
my $minimum_perl_version = 5.10.0;
@@ -94,6 +95,7 @@ Options:
--emacs emacs compile window format
--terse one line per report
--showfile emit diffed file position, not input file position
+ --stdin-filename=FILE when using stdin, identify the file as FILE
-g, --git treat FILE as a single commit or git revision range
single git commit with:
<rev>
@@ -323,6 +325,7 @@ GetOptions(
'showfile!' => \$showfile,
'f|file!' => \$file,
'g|git!' => \$git,
+ 'stdin-filename=s' => \$stdin_filename,
'subjective!' => \$check,
'strict!' => \$check,
'ignore=s' => \@ignore,
@@ -2652,6 +2655,10 @@ sub is_userspace {
sub process {
my $filename = shift;
+ if ($filename eq '-' && defined($stdin_filename)) {
+ $filename = $stdin_filename;
+ }
+
my $linenr=0;
my $prevline="";
my $prevrawline="";
@@ -2891,6 +2898,10 @@ sub process {
$realfile =~ s@^([^/]*)/@@ if (!$file);
$in_commit_log = 0;
+ if ($realfile eq "-" && defined($stdin_filename)) {
+ $realfile = $stdin_filename;
+ }
+
$p1_prefix = $1;
if (!$file && $tree && $p1_prefix ne '' &&
-e "$root/$p1_prefix") {
--
2.53.0.1018.g2bb0e51243-goog
--
Dmitry
next reply other threads:[~2026-03-26 6:20 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-26 6:20 Dmitry Torokhov [this message]
2026-03-26 8:46 ` [PATCH] checkpatch: allow correctly handle full files on stdin Joe Perches
2026-03-26 14:53 ` Dmitry Torokhov
2026-03-26 21:36 ` Joe Perches
2026-03-26 21:42 ` Dmitry Torokhov
2026-03-26 22:56 ` Joe Perches
2026-03-26 23:04 ` Dmitry Torokhov
2026-03-26 23:10 ` Joe Perches
2026-03-26 23:19 ` Dmitry Torokhov
2026-03-26 23:27 ` Joe Perches
2026-03-26 23:19 ` Joe Perches
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=acTPXMJfkHLeItrK@google.com \
--to=dmitry.torokhov@gmail.com \
--cc=apw@canonical.com \
--cc=corbet@lwn.net \
--cc=dwaipayanray1@gmail.com \
--cc=joe@perches.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lukas.bulwahn@gmail.com \
--cc=skhan@linuxfoundation.org \
--cc=workflows@vger.kernel.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