Thread (20 messages) 20 messages, 4 authors, 2023-03-23

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

On Wed, 22 Mar 2023 11:38:01 +0000 Donald Hunter wrote:
quoted
quoted
+    Attributes:
+        type    string, kernel type of the member attribute  
We 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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help