[dpdk-dev] [PATCH v4 05/15] net/dpaa2: use rte_eth_linkstatus_set
Shreyansh Jain
shreyansh.jain at nxp.com
Tue Jan 16 10:57:20 CET 2018
On Tuesday 16 January 2018 03:14 PM, Shreyansh Jain wrote:
> On Thursday 11 January 2018 10:36 PM, Ferruh Yigit wrote:
>> From: Stephen Hemminger <stephen at networkplumber.org>
>>
>> Use new helper function to update the link status.
>> As a good side effect this fixes a but because this driver was not
> ^^^^^^
> needs rephrasing
>
>> returning correct status (should be -1 in link_status changed).
>>
>> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
>> ---
>> drivers/net/dpaa2/dpaa2_ethdev.c | 65
>> +++++-----------------------------------
>> 1 file changed, 7 insertions(+), 58 deletions(-)
>
> Other than the change in commit message highlighted above:
>
> Acked-by: Shreyansh Jain <shreyansh.jain at nxp.com>
>
> Thanks.
>
Also, I forgot to add in previous email, I am assuming
_rte_eth_linkstatus_get would be rte_eth_linkstatus_get eventually.
I concur with your comments in [1] - it certainly would be better to
have either @internal or _rte - and not both.
http://dpdk.org/ml/archives/dev/2018-January/086742.html
More information about the dev
mailing list