Message ID | 20200909235827.3335881-2-andrew@lunn.ch |
---|---|
State | Changes Requested |
Delegated to: | David Miller |
Headers | show |
Series | mv88e6xxx: Add devlink regions support | expand |
On 9/9/2020 4:58 PM, Andrew Lunn wrote: > The driver may have multiple regions which can be dumped using one > function. However, for this to work, additional information is > needed. Add a priv member to the ops structure for the driver to use > however it likes. > > Signed-off-by: Andrew Lunn <andrew@lunn.ch> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
diff --git a/include/net/devlink.h b/include/net/devlink.h index eaec0a8cc5ef..86ce644260b3 100644 --- a/include/net/devlink.h +++ b/include/net/devlink.h @@ -542,12 +542,14 @@ struct devlink_info_req; * the data variable must be updated to point to the snapshot data. * The function will be called while the devlink instance lock is * held. + * @priv: Pointer to driver private data for the region operation */ struct devlink_region_ops { const char *name; void (*destructor)(const void *data); int (*snapshot)(struct devlink *devlink, struct netlink_ext_ack *extack, u8 **data); + void *priv; }; struct devlink_fmsg;
The driver may have multiple regions which can be dumped using one function. However, for this to work, additional information is needed. Add a priv member to the ops structure for the driver to use however it likes. Signed-off-by: Andrew Lunn <andrew@lunn.ch> --- include/net/devlink.h | 2 ++ 1 file changed, 2 insertions(+)