29/10/2026 17:11, Thomas Monjalon: > 26/10/2026 21:09, Roman Khromenok: > > 1. The library name: "transceiver" or "xcvr". > > "transceiver" reads better, but "xcvr" is shorter. > I cannot decide.
Then I would keep "transceiver": it needs no explanation in the documentation, and the function prefix rte_transceiver_ is not too long. If Stephen or Andrew prefer "xcvr", I will rename it. > > 2. Whether rte_eth_module_eeprom_parse() stays as a wrapper > > or is deprecated in favor of rte_transceiver_decode(). > > rte_eth_module_eeprom_parse() is experimental so it can be removed. OK, the v1 for 27.03 will remove it: - testpmd "show port <id> module_eeprom decode" will call rte_transceiver_decode(), - the ethdev telemetry command keeps using the library internally, - the removal will be listed in the 27.03 release notes. The patch 2/2 of the dpdk-proc-info series 39575 uses this function, so I will send a v2 of it based on the library. The fix 1/2 does not depend on it. Thanks, Roman
