On Thursday 23 February 2017 07:59 PM, Ferruh Yigit wrote: > On 2/21/2017 9:26 AM, Shijith Thotton wrote: >> Register LiquidIO PMD (net_liovf) and define APIs to init and >> uninit. >> >> Signed-off-by: Shijith Thotton <shijith.thot...@caviumnetworks.com> >> Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> >> Signed-off-by: Derek Chickles <derek.chick...@caviumnetworks.com> >> Signed-off-by: Venkat Koppula <venkat.kopp...@caviumnetworks.com> >> Signed-off-by: Mallesham Jatharakonda <mjatharako...@oneconvergence.com> > <...> > >> + >> + mac_addr_size = ETHER_ADDR_LEN; >> + >> + eth_dev->data->mac_addrs = rte_zmalloc("lio", mac_addr_size, 0); > > "mac_addr_size" only used here, your call but it is possible to remove > variable and use ETHER_ADDR_LEN here directly. >
I agree. Will remove variable "mac_addr_size".