checkpatch: Remove links to outdated documentation
authorAnna-Maria Behnsen <anna-maria@linutronix.de>
Mon, 14 Oct 2024 08:22:27 +0000 (10:22 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 15 Oct 2024 22:36:47 +0000 (00:36 +0200)
checkpatch.pl checks for several things related to sleep and delay
functions. In all warnings the outdated documentation is referenced. Also
in checkpatch kernel documentation the outdated documentation is
referenced.

Replace the links to the outdated documentation with links to the function
description.

Note: Update of the outdated checkpatch checks is done in a second step.

Signed-off-by: Anna-Maria Behnsen <anna-maria@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Frederic Weisbecker <frederic@kernel.org>
Link: https://lore.kernel.org/all/20241014-devel-anna-maria-b4-timers-flseep-v3-10-dc8b907cb62f@linutronix.de
Documentation/dev-tools/checkpatch.rst
scripts/checkpatch.pl

index a9fac978a52514ba9197faf217c470b398b184bf..abb3ff6820766ee0c29112b256bcc44ce41fffba 100644 (file)
@@ -470,8 +470,6 @@ API usage
     usleep_range() should be preferred over udelay(). The proper way of
     using usleep_range() is mentioned in the kernel docs.
 
-    See: https://www.kernel.org/doc/html/latest/timers/timers-howto.html#delays-information-on-the-various-kernel-delay-sleep-mechanisms
-
 
 Comments
 --------
index 4427572b24771ddb1f2bf3de3176f9437f643951..98790fe5115d82e8fdb7e2c8b92951780ae1828e 100755 (executable)
@@ -6597,11 +6597,11 @@ sub process {
                        # ignore udelay's < 10, however
                        if (! ($delay < 10) ) {
                                CHK("USLEEP_RANGE",
-                                   "usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst\n" . $herecurr);
+                                   "usleep_range is preferred over udelay; see function description of usleep_range() and udelay().\n" . $herecurr);
                        }
                        if ($delay > 2000) {
                                WARN("LONG_UDELAY",
-                                    "long udelay - prefer mdelay; see arch/arm/include/asm/delay.h\n" . $herecurr);
+                                    "long udelay - prefer mdelay; see function description of mdelay().\n" . $herecurr);
                        }
                }
 
@@ -6609,7 +6609,7 @@ sub process {
                if ($line =~ /\bmsleep\s*\((\d+)\);/) {
                        if ($1 < 20) {
                                WARN("MSLEEP",
-                                    "msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.rst\n" . $herecurr);
+                                    "msleep < 20ms can sleep for up to 20ms; see function description of msleep().\n" . $herecurr);
                        }
                }
 
@@ -7077,11 +7077,11 @@ sub process {
                        my $max = $7;
                        if ($min eq $max) {
                                WARN("USLEEP_RANGE",
-                                    "usleep_range should not use min == max args; see Documentation/timers/timers-howto.rst\n" . "$here\n$stat\n");
+                                    "usleep_range should not use min == max args;  see function description of usleep_range().\n" . "$here\n$stat\n");
                        } elsif ($min =~ /^\d+$/ && $max =~ /^\d+$/ &&
                                 $min > $max) {
                                WARN("USLEEP_RANGE",
-                                    "usleep_range args reversed, use min then max; see Documentation/timers/timers-howto.rst\n" . "$here\n$stat\n");
+                                    "usleep_range args reversed, use min then max;  see function description of usleep_range().\n" . "$here\n$stat\n");
                        }
                }