dt: dt-extract-compatibles: Extract compatibles from function parameters
Various DT and fwnode functions take a compatible string as a parameter. These are often used in cases which don't have a driver, so they've been missed. The additional checks add about 400 more undocumented compatible strings. Link: https://lore.kernel.org/all/20240903200753.2097911-1-robh@kernel.org/ Acked-by: Saravana Kannan <saravanak@google.com> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
This commit is contained in:
parent
000f6d588a
commit
4c727150a6
@ -46,6 +46,15 @@ def parse_of_match_table(data):
|
|||||||
return match_table_list
|
return match_table_list
|
||||||
|
|
||||||
|
|
||||||
|
def parse_of_functions(data, func_name):
|
||||||
|
""" Find all compatibles in the last argument of a given function """
|
||||||
|
compat_list = []
|
||||||
|
for m in re.finditer(rf'{func_name}\(([a-zA-Z0-9_>\(\)"\-]+,\s)*"([a-zA-Z0-9_,-]+)"\)', data):
|
||||||
|
compat_list.append(m[2])
|
||||||
|
|
||||||
|
return compat_list
|
||||||
|
|
||||||
|
|
||||||
def parse_compatibles(file, compat_ignore_list):
|
def parse_compatibles(file, compat_ignore_list):
|
||||||
with open(file, 'r', encoding='utf-8') as f:
|
with open(file, 'r', encoding='utf-8') as f:
|
||||||
data = f.read().replace('\n', '')
|
data = f.read().replace('\n', '')
|
||||||
@ -60,6 +69,10 @@ def parse_compatibles(file, compat_ignore_list):
|
|||||||
else:
|
else:
|
||||||
compat_list = parse_of_declare_macros(data)
|
compat_list = parse_of_declare_macros(data)
|
||||||
compat_list += parse_of_device_id(data)
|
compat_list += parse_of_device_id(data)
|
||||||
|
compat_list += parse_of_functions(data, "_is_compatible")
|
||||||
|
compat_list += parse_of_functions(data, "of_find_compatible_node")
|
||||||
|
compat_list += parse_of_functions(data, "for_each_compatible_node")
|
||||||
|
compat_list += parse_of_functions(data, "of_get_compatible_child")
|
||||||
|
|
||||||
return compat_list
|
return compat_list
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user