> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Friday, January 3, 2020 01:37
> To: Wang, Haiyue <haiyue.w...@intel.com>
> Cc: dev@dpdk.org; Ye, Xiaolong <xiaolong...@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v1] net/ice: align with MAC address length 
> definition
> 
> On Thu,  2 Jan 2020 15:14:57 +0800
> Haiyue Wang <haiyue.w...@intel.com> wrote:
> 
> > Use the standand MAC address length definition in PMD's scope.
> >
> > Signed-off-by: Haiyue Wang <haiyue.w...@intel.com>
> > ---
> >  drivers/net/ice/ice_ethdev.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
> > index 2cbd82c94..80b422a27 100644
> > --- a/drivers/net/ice/ice_ethdev.c
> > +++ b/drivers/net/ice/ice_ethdev.c
> > @@ -873,7 +873,7 @@ ice_add_mac_filter(struct ice_vsi *vsi, struct 
> > rte_ether_addr *mac_addr)
> >             ret = -ENOMEM;
> >             goto DONE;
> >     }
> > -   rte_memcpy(&f->mac_info.mac_addr, mac_addr, ETH_ADDR_LEN);
> > +   rte_memcpy(&f->mac_info.mac_addr, mac_addr, RTE_ETHER_ADDR_LEN);
> 
> Please use rte_ether_addr_copy instead.
Will be update patch v2.

Reply via email to