Hi Jianfeng,
On Fri, Feb 26, 2016 at 08:09:28AM +0800, Jianfeng Tan wrote:
> Signed-off-by: Jianfeng Tan <jianfeng.tan at intel.com>
> ---
> drivers/net/mlx4/mlx4.c | 2 +-
> drivers/net/mlx5/mlx5.c | 1 +
> drivers/net/mlx5/mlx5.h | 1 +
> drivers/net/mlx5/mlx5_ethdev.c | 20 ++++++++++++++++++++
> drivers/net/mlx5/mlx5_rxtx.c | 2 ++
> 5 files changed, 25 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c
> index 85fdebf..58f4e1a 100644
> --- a/drivers/net/mlx4/mlx4.c
> +++ b/drivers/net/mlx4/mlx4.c
> @@ -4269,7 +4269,7 @@ mlx4_dev_infos_get(struct rte_eth_dev *dev, struct
> rte_eth_dev_info *info)
> priv_unlock(priv);
> }
>
> -static uint32_t *
> +static const uint32_t *
> mlx4_dev_ptype_info_get(struct rte_eth_dev *dev)
> {
> static const uint32_t ptypes[] = {
I'm probably nitpicking here but this change should be merged in the mlx4
patch. Otherwise both mlx4 and mlx5 patches look fine to me.
--
Adrien Mazarguil
6WIND
