Is this something that it might be better to expose via ethtool in the
firmware-version field?  (You can have a string with multiple versions
of different components there, see eg get_drvinfo() in
drivers/net/cxgb3/cxgb3_main.c)

As far as I can see, in the current patch, you have eeprom_version as a
pure write-only field.  So instead of dumping to the kernel log, where
it might get lost and is in any case hard to find, make it easier to
read out when someone needs it.

 - R.
-- 
Roland Dreier <[email protected]> || For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to