Skip to content

Commit 77e7720

Browse files
committed
pinvalidate.py script: add SPI and I2C alias check
Goal is to follow hal/include/hal/PinNameAliases.h
1 parent 616a84f commit 77e7720

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

hal/tests/pinvalidate/pinvalidate.py

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -440,6 +440,22 @@ def legacy_assignment_check(pin_name_content):
440440
invalid_items.append({"key": key, "val": val, "message": message})
441441
return invalid_items
442442

443+
444+
def legacy_alias_check(pin_name_content):
445+
invalid_items = []
446+
legacy_assignments = dict(
447+
re.findall(
448+
r"^\s*((?:SPI|I2C)_\w*)\s*=\s*([a-zA-Z0-9_]+)",
449+
pin_name_content,
450+
re.MULTILINE,
451+
)
452+
)
453+
for key, val in legacy_assignments.items():
454+
message = "legacy assignment; SPI_xxx and I2C_xxx must be #define'd"
455+
invalid_items.append({"key": key, "val": val, "message": message})
456+
return invalid_items
457+
458+
443459
def legacy_uart_check(pin_name_dict):
444460
invalid_items = []
445461
if "CONSOLE_TX" not in pin_name_dict or "CONSOLE_RX" not in pin_name_dict:
@@ -662,6 +678,12 @@ def has_passed_all_test_cases(report):
662678
"case_function": legacy_assignment_check,
663679
"case_input": "content",
664680
},
681+
{
682+
"suite_name": "generic",
683+
"case_name": "alias",
684+
"case_function": legacy_alias_check,
685+
"case_input": "content",
686+
},
665687
{
666688
"suite_name": "generic",
667689
"case_name": "uart",

0 commit comments

Comments
 (0)