Unrelated (I think?) this is a relatively big series so I don't want to race with it, but I think we should rename the defines.
you are right that this is unrelated but conflicting :)
DEVLINK_INFO_VERSION_GENERIC_x -> DEVLINK_VER_x ? You did some major devlink refactors, maybe you want to take this on? :) The 40 char defines lead to pretty ugly wrapping, and make constructs like: if (something) devlink_info_version_running_put(... impossible. We could also rename the helpers to s/_version// ..
sure, I have also one more refactor idea around this family of functions so it would fit well :)
