Re: [PATCH net-next v12 04/14] dpll: zl3073x: Add support for devlink device info
flat view
From: Jiri Pirko <jiri@resnulli.us>
Date: 2025-07-02 10:26:11
Also in:
linux-devicetree, linux-doc, lkml
Sun, Jun 29, 2025 at 09:10:39PM +0200, ivecera@redhat.com wrote: [...]
quoted hunk ↗ jump to hunk
+ snprintf(buf, sizeof(buf), "%lu.%lu.%lu.%lu", + FIELD_GET(GENMASK(31, 24), cfg_ver), + FIELD_GET(GENMASK(23, 16), cfg_ver), + FIELD_GET(GENMASK(15, 8), cfg_ver), + FIELD_GET(GENMASK(7, 0), cfg_ver)); + + return devlink_info_version_running_put(req, "cfg.custom_ver", buf);
Nit: It's redundant to put "ver" string into version name. Also, isn't it rather "custom_config" or "custom_cfg"?