On Mon, 7 Apr 2025 14:51:12 -0700 Tony Nguyen wrote: > + err = devlink_info_version_running_put(req, "fw.mgmt.srev", ctx->buf); > + if (err) > + return err; > + > + ixgbe_info_orom_srev(adapter, ctx); > + err = devlink_info_version_running_put(req, "fw.undi.srev", ctx->buf); Please add a note in the documentation (the iproute2 man page, the main devlink info one, and in the driver doc) stating that any security versions reported via devlink are purely informational. Devlink does not use a secure channel to communicate with the device.