[PATCH net] net: ynl: allow u8 sub-type for indexed array

Hilal Rahmatullah posted 1 patch 1 month, 1 week ago
tools/net/ynl/pyynl/ynl_gen_c.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH net] net: ynl: allow u8 sub-type for indexed array
Posted by Hilal Rahmatullah 1 month, 1 week ago
add u8 as supported subtype for indexed array, this is required
for attribute NL80211_STA_INFO_CHAIN_SIGNAL serialization.

Signed-off-by: Hilal Rahmatullah <hilalrahmatullah11@gmail.com>
---
 tools/net/ynl/pyynl/ynl_gen_c.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/net/ynl/pyynl/ynl_gen_c.py b/tools/net/ynl/pyynl/ynl_gen_c.py
index cdc3646f2642..7373a48d5516 100755
--- a/tools/net/ynl/pyynl/ynl_gen_c.py
+++ b/tools/net/ynl/pyynl/ynl_gen_c.py
@@ -1158,7 +1158,7 @@ class AttrSet(SpecAttrSet):
         elif elem['type'] == 'nest':
             t = TypeNest(self.family, self, elem, value)
         elif elem['type'] == 'indexed-array' and 'sub-type' in elem:
-            if elem["sub-type"] in ['binary', 'nest', 'u32']:
+            if elem["sub-type"] in ['binary', 'nest', 'u32', 'u8']:
                 t = TypeIndexedArray(self.family, self, elem, value)
             else:
                 raise Exception(f'new_attr: unsupported sub-type {elem["sub-type"]}')
-- 
2.53.0
Re: [PATCH net] net: ynl: allow u8 sub-type for indexed array
Posted by Jakub Kicinski 1 month, 1 week ago
On Thu, 20 Aug 2026 18:09:03 +0700 Hilal Rahmatullah wrote:
> add u8 as supported subtype for indexed array, this is required
> for attribute NL80211_STA_INFO_CHAIN_SIGNAL serialization.

Could you please send this as part of a series which adds that field?
Otherwise it's a bit hard to review
Re: [PATCH net] net: ynl: allow u8 sub-type for indexed array
Posted by Hilal Rahmatullah 1 month, 1 week ago
On Thu, Aug 20, 2026 at 11:00:42AM -0700, Jakub Kicinski wrote:
> On Thu, 20 Aug 2026 18:09:03 +0700 Hilal Rahmatullah wrote:
> > add u8 as supported subtype for indexed array, this is required
> > for attribute NL80211_STA_INFO_CHAIN_SIGNAL serialization.
> 
> Could you please send this as part of a series which adds that field?
> Otherwise it's a bit hard to review

Surely! I'll work on it