[RFC PATCH 3/3] ethdev: rename parameter in API to get FEC
Ferruh Yigit
ferruh.yigit at amd.com
Tue May 2 17:02:46 CEST 2023
On 4/28/2023 11:27 AM, Denis Pryazhennikov wrote:
> Only one valid FEC mode can be get by rte_eth_fec_get().
> The previous name implied that more than one FEC mode
> can be obtained.
+1 and patch looks good.
But isn't this valid for 'rte_eth_fec_set()', it gets 'fec_mode'. FEC
capability has its own type "struct rte_eth_fec_capa".
Independent from being single FEC mode or not, I think both
'rte_eth_fec_get()' & 'rte_eth_fec_set()' should get 'fec_mode' as
param, what do you think?
> Documentation was updated accordingly.
>
> Signed-off-by: Denis Pryazhennikov <denis.pryazhennikov at arknetworks.am>
> Acked-by: Ivan Malov <ivan.malov at arknetworks.am>
> Acked-by: Viacheslav Galaktionov <viacheslav.galaktionov at arknetworks.am>
<...>
More information about the dev
mailing list