u-boot.git/tools/patman/test_checkpatch.py, branch v2024.01

patman: Add a 'keep_change_id' setting

2023-11-02T15:38:54Z

A Change-Id can be useful for traceability purposes, and some projects may wish to have them preserved. This change makes it configurable via a new 'keep_change_id' setting. Signed-off-by: Maxim Cournoyer Reviewed-by: Simon Glass

patman: Add a little documentation on the checkpatch tests

2023-10-24T20:34:45Z

These texts lack comments. Add some so that it is clearer what is going on. Signed-off-by: Simon Glass

checkpatch.pl: Make common.h check boarder

2023-10-24T20:34:45Z

At this point in time we should not add common.h to any new files, so make checkpatch.pl complain. Signed-off-by: Tom Rini Signed-off-by: Simon Glass

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

2023-02-14T16:43:27Z

Help ensure that these don't creep into development by adding a check in checkpatch for them. Signed-off-by: Simon Glass

checkpatch.pl: Update CONFIG logic in U-Boot section

2022-12-23T15:15:13Z

Now that all CONFIG symbols are in Kconfig, checkpatch.pl should check for and error on any case of define/undef CONFIG_*. Signed-off-by: Tom Rini

patman: Convert camel case in test_checkpatch.py

2022-02-09T19:26:12Z

Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass

patman: Convert camel case in checkpatch.py

2022-02-09T19:26:12Z

Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass

checkpatch: Add warnings for using strn(cat|cpy)

2021-04-12T21:44:55Z

strn(cat|cpy) has a bad habit of not nul-terminating the destination, resulting in constructions like strncpy(foo, bar, sizeof(foo) - 1); foo[sizeof(foo) - 1] = '\0'; However, it is very easy to forget about this behavior and accidentally leave a string unterminated. This has shown up in some recent coverity scans [1, 2] (including code recently touched by yours truly). Fortunately, the guys at OpenBSD came up with strl(cat|cpy), which always nul-terminate strings. These functions are already in U-Boot, so we should encourage new code to use them instead of strn(cat|cpy). [1] https://lists.denx.de/pipermail/u-boot/2021-March/442888.html [2] https://lists.denx.de/pipermail/u-boot/2021-January/438073.html Signed-off-by: Sean Anderson Reviewed-by: Simon Glass

checkpatch: Add warnings for unexpected struct names

2020-12-13T23:51:09Z

As a way of keeping the driver declarations more consistent, add a warning if the struct used does not end with _priv or _plat. Signed-off-by: Simon Glass

patman: Rename functions in patchstream

2020-11-05T16:11:31Z

Rename these functions to lower case as per PEP8. Signed-off-by: Simon Glass