[dpdk-dev] [PATCH v2 2/5] net/i40e: add ppp processing
ferruh.yigit at intel.com
Wed Mar 8 13:07:16 CET 2017
On 3/3/2017 7:39 AM, Beilei Xing wrote:
> Add loading profile function.
> Signed-off-by: Beilei Xing <beilei.xing at intel.com>
> drivers/net/i40e/i40e_ethdev.c | 59 +++++++++++++++++++++++++++++++++++++++++
> drivers/net/i40e/rte_pmd_i40e.h | 7 +++++
Also you need to update *version.map file for new API.
> +i40e_process_package(uint8_t port, uint8_t *buff)
Function name is so generic, please pick another one that associated
Also please use defined name_space for public API: rte_pmd_i40e_<...>
> + struct rte_eth_dev *dev = &rte_eth_devices[port];
Now this is public API to user applications, no more driver function, so
requires more attention. Input values needs to be verified before using
Also you need to check if provided port_id if i40e port id.
> + * i40e_process_package - Load package
> + * @port: port id
> + * @buff: buffer of package
> + **/
Please provide proper doxygen tags, these are causing error.
> +int i40e_process_package(uint8_t port, uint8_t *buff);
> #endif /* _PMD_I40E_H_ */
More information about the dev