diff options
| author | Simon Glass <sjg@chromium.org> | 2023-02-13 08:56:38 -0700 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2023-02-14 09:43:27 -0700 |
| commit | 48be546b70f509d8ac02288fd5069913ec9196f3 (patch) | |
| tree | 689a434be90fb42e32db7332f154cd39c29ecea9 | |
| parent | 0a06d7106195bbbe3126d205c1c173d6eb9e48a8 (diff) | |
| download | u-boot-48be546b70f509d8ac02288fd5069913ec9196f3.tar.gz u-boot-48be546b70f509d8ac02288fd5069913ec9196f3.tar.bz2 u-boot-48be546b70f509d8ac02288fd5069913ec9196f3.zip | |
checkpatch: Add a warning for pre-schema driver model tags
Help ensure that these don't creep into development by adding a check in
checkpatch for them.
Signed-off-by: Simon Glass <sjg@chromium.org>
| -rwxr-xr-x | scripts/checkpatch.pl | 6 | ||||
| -rw-r--r-- | tools/patman/test_checkpatch.py | 6 |
2 files changed, 12 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index ccfcbb3e12..62b764f6c3 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2680,6 +2680,12 @@ sub u_boot_line { "DEVICE_PRIV_AUTO", $herecurr); u_boot_struct_name($line, "per_device_plat_auto", "_plat", "DEVICE_PLAT_AUTO", $herecurr); + + # Avoid using the pre-schema driver model tags + if ($line =~ /^\+.*u-boot,dm-.*/) { + ERROR("PRE_SCHEMA", + "Driver model schema uses 'bootph-...' tags now\n" . $herecurr); + } } sub exclude_global_initialisers { diff --git a/tools/patman/test_checkpatch.py b/tools/patman/test_checkpatch.py index 4c2ab6e590..a8bb364e42 100644 --- a/tools/patman/test_checkpatch.py +++ b/tools/patman/test_checkpatch.py @@ -452,6 +452,12 @@ index 0000000..2234c87 self.check_strl("cat"); self.check_strl("cpy"); + def test_schema(self): + """Check for uses of strn(cat|cpy)""" + pm = PatchMaker() + pm.add_line('arch/sandbox/dts/sandbox.dtsi', '\tu-boot,dm-pre-proper;') + self.check_single_message(pm, 'PRE_SCHEMA', 'error') + if __name__ == "__main__": unittest.main() gitutil.RunTests() |