michaelh/zephyr
1
0
Fork 0
Code Releases Activity

scripts: compliance: use get_files filter for dts checks

Browse source 
Drop the custom logic to skip deleted files, use get_files filter
instead.

Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
This commit is contained in:
Fabio Baltieri 2022-12-26 14:26:01 +00:00 • committed by Carles Cufí
commit 5bf79ed9cd
1 changed files with 10 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

24
scripts/ci/check_compliance.py
View file

@ -236,26 +236,22 @@ class DevicetreeBindingsCheck(ComplianceTest):
""" """
dt_bindings = [] dt_bindings = []
for file_name in get_files(): for file_name in get_files(filter="d"):
if file_name.startswith('dts/bindings/') and file_name.endswith('.yaml'): if file_name.startswith('dts/bindings/') and file_name.endswith('.yaml'):
dt_bindings.append(file_name) dt_bindings.append(file_name)
return dt_bindings return dt_bindings
def required_false_check(self, dts_binding): def required_false_check(self, dts_binding):
try: with open(dts_binding) as file:
with open(dts_binding) as file: line_number = 0
line_number = 0 for line in file:
for line in file: line_number += 1
line_number += 1 if 'required: false' in line:
if 'required: false' in line: self.fmtd_failure(
self.fmtd_failure( 'warning', 'Devicetree Bindings', dts_binding,
'warning', 'Devicetree Bindings', dts_binding, line_number, col=None,
line_number, col=None, desc="'required: false' is redundant, please remove")
desc="'required: false' is redundant, please remove")
except Exception:
# error opening file (it was likely deleted by the commit)
return
class KconfigCheck(ComplianceTest): class KconfigCheck(ComplianceTest):