[dpdk-dev] [PATCH] bnx2x: tx_start_bd->vlan_or_ethertype is le16

[Thomas Monjalon]

2015-12-01 14:37, Stephen Hemminger:
> Harish Patil <harish.patil at qlogic.com> wrote:
> > >2015-11-03 12:26, Chas Williams:  
> > >> --- a/drivers/net/bnx2x/bnx2x.c
> > >> +++ b/drivers/net/bnx2x/bnx2x.c
> > >> -                            tx_start_bd->vlan_or_ethertype = eh->ether_type;
> > >> +                            tx_start_bd->vlan_or_ethertype
> > >> +                                = rte_cpu_to_le_16(rte_be_to_cpu_16(eh->ether_type));
> > 
> > Minor question - any specific reason to use rte_be_to_cpu_16() on
> > ether_type alone before converting from native order to le16?
> 
> ether_type is in network byte order (big endian)
> and hardware wants little endian. On x86 the second step is a nop.

Doesn't it deserve a macro rte_ntole16()?
It may be in lib/librte_eal/common/include/generic/rte_byteorder.h.