Difuse/u-boot
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

checkpatch: Add a warning for pre-schema driver model tags

Browse source 
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>
This commit is contained in:
Simon Glass 2023-02-13 08:56:38 -07:00
parent 0a06d71061
commit 48be546b70
2 changed files with 12 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
scripts/checkpatch.pl
View file

@ -2680,6 +2680,12 @@ sub u_boot_line {
"DEVICE_PRIV_AUTO", $herecurr); "DEVICE_PRIV_AUTO", $herecurr);
u_boot_struct_name($line, "per_device_plat_auto", "_plat", u_boot_struct_name($line, "per_device_plat_auto", "_plat",
"DEVICE_PLAT_AUTO", $herecurr); "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 { sub exclude_global_initialisers {

6
tools/patman/test_checkpatch.py
View file

@ -452,6 +452,12 @@ index 0000000..2234c87
self.check_strl("cat"); self.check_strl("cat");
self.check_strl("cpy"); 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__": if __name__ == "__main__":
unittest.main() unittest.main()
gitutil.RunTests() gitutil.RunTests()