Commit b49c34e2 authored by Jakub Kicinski's avatar Jakub Kicinski

tools: ynl: rename ops_list -> msg_list

ops_list contains all the operations, but the main iteration use
case is to walk only ops which define attrs. Rename ops_list to
msg_list, because now it looks like the contents are the same,
just the format is different. While at it convert from tuple
to just keys, none of the users care about the name of the op.
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 66fa34b9
...@@ -790,8 +790,10 @@ class Family: ...@@ -790,8 +790,10 @@ class Family:
self.mcgrps = self.yaml.get('mcast-groups', {'list': []}) self.mcgrps = self.yaml.get('mcast-groups', {'list': []})
self.consts = dict() self.consts = dict()
# list of all operations
self.msg_list = []
# dict of operations which have their own message type (have attributes)
self.ops = dict() self.ops = dict()
self.ops_list = []
self.attr_sets = dict() self.attr_sets = dict()
self.attr_sets_list = [] self.attr_sets_list = []
...@@ -858,7 +860,7 @@ class Family: ...@@ -858,7 +860,7 @@ class Family:
op = Operation(self, elem, val) op = Operation(self, elem, val)
val += 1 val += 1
self.ops_list.append((elem['name'], op),) self.msg_list.append(op)
if 'notify' in elem: if 'notify' in elem:
ntf.append(op) ntf.append(op)
continue continue
...@@ -2063,7 +2065,7 @@ def render_uapi(family, cw): ...@@ -2063,7 +2065,7 @@ def render_uapi(family, cw):
max_value = f"({cnt_name} - 1)" max_value = f"({cnt_name} - 1)"
uapi_enum_start(family, cw, family['operations'], 'enum-name') uapi_enum_start(family, cw, family['operations'], 'enum-name')
for _, op in family.ops_list: for op in family.msg_list:
if separate_ntf and ('notify' in op or 'event' in op): if separate_ntf and ('notify' in op or 'event' in op):
continue continue
...@@ -2082,7 +2084,7 @@ def render_uapi(family, cw): ...@@ -2082,7 +2084,7 @@ def render_uapi(family, cw):
if separate_ntf: if separate_ntf:
uapi_enum_start(family, cw, family['operations'], enum_name='async-enum') uapi_enum_start(family, cw, family['operations'], enum_name='async-enum')
for _, op in family.ops_list: for op in family.msg_list:
if separate_ntf and not ('notify' in op or 'event' in op): if separate_ntf and not ('notify' in op or 'event' in op):
continue continue
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment