[dpdk-dev] [PATCH 3/7] ethdev: separate driver APIs
hemant.agrawal at nxp.com
Fri Dec 1 09:59:01 CET 2017
On 12/1/2017 7:59 AM, Ferruh Yigit wrote:
> diff --git a/lib/librte_ether/rte_ethdev_driver.h b/lib/librte_ether/rte_ethdev_driver.h
> new file mode 100644
> index 000000000..3e77d1439
> --- /dev/null
> +++ b/lib/librte_ether/rte_ethdev_driver.h
> @@ -0,0 +1,163 @@
> + * BSD LICENSE
> + *
> + * Copyright(c) 2017 Intel Corporation. All rights reserved.
> + * All rights reserved.
You can remove one of the all rights reserved.
This is also an issue in your next patch for rte_ethdev_core.h
Also, as Shreyansh mentioned, Why not start with SPDX tags instead of
full license text?
> + * @internal Executes all the user application registered callbacks for
> + * the specific device. It is for DPDK internal user only. User
> + * application should not call it directly.
> + *
> + * @param dev
> + * Pointer to struct rte_eth_dev.
> + * @param event
> + * Eth device interrupt event type.
> + * @param cb_arg
> + * callback parameter.
> + * @param ret_param
> + * To pass data back to user application.
> + * This allows the user application to decide if a particular function
> + * is permitted or not.
> + *
> + * @return
> + * int
> + */
> +int _rte_eth_dev_callback_process(struct rte_eth_dev *dev,
> + enum rte_eth_event_type event, void *cb_arg, void *ret_param);
> + * Create memzone for HW rings.
Like all other functions, you can also add "@internal" for this as well.
Acked-by: Hemant Agrawal <hemant.agrawal at nxp.com>
More information about the dev