Re: [PATCH net-next v2 2/6] tools: ynl: Add struct parsing to nlspec
From: Jakub Kicinski <kuba@kernel.org>
Date: 2023-03-22 18:25:32
From: Jakub Kicinski <kuba@kernel.org>
Date: 2023-03-22 18:25:32
On Wed, 22 Mar 2023 11:38:01 +0000 Donald Hunter wrote:
quoted
quoted
+ Attributes: + type string, kernel type of the member attributeWe can have structs inside structs in theory, or "binary blobs" so this is really a subset of what attr can be rather than necessarily a kernel type?Okay, so the schema currently defines the member types as u*, s* and string. Does it make sense to add 'binary' and 'struct'?
We don't have to until it's needed.
To be clear, do you want me to drop the word 'kernel' from the docstring, or something more?
Removing kernel should be good enough