On 07/22/2015 01:23 PM, Maxim Uvarov wrote: > On 07/22/15 13:57, Nicolas Morey-Chaisemartin wrote: >> >> On 07/22/2015 12:00 PM, Maxim Uvarov wrote: >>> On 07/17/15 12:15, Nicolas Morey-Chaisemartin wrote: >>>> .open = sock_mmsg_open, >>>> - .close = sock_close_pkt, >>>> + .close = sock_close, >>>> .recv = sock_mmsg_recv_pkt, >>>> .send = sock_mmsg_send_pkt, >>> Nicolas, I'm going to apply first patch. But in that patch it's not clear >>> why you also did not remove _pkt from recv and send functions. >> Because all the interfaces used the exact same name so I wasn't sure it was >> worth it. >> If we want to have a standar <pktio>_<action> name for all methods, I might >> aswell rename the <pktio>_mac_addr_get to <pktio>_mac_get too >> >> Nicolas > > All this functions are static. And having prefix on some of them and do not > have on others is strange. > > For mac addr, I think, you should not drop addr word to better match with api > call odp_pktio_mac_addr() > > Maxim. Maybe drop the get so we have <pktio type>_mac_addr() then ?
_______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp