Re: [lng-odp] [PATCH 3/6] validation: cosmetic fixes in odp_pktio.c

[Stuart Haslam]

On Thu, May 28, 2015 at 12:58:09PM +0200, Christophe Milard wrote:
> Preparing for the next patch where this file is moved and
> check-odp would yell if these things were still there.
> 
> Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>

Reviewed-by: Stuart Haslam <stuart.has...@linaro.org>

> ---
>  test/validation/odp_pktio.c | 13 ++++---------
>  1 file changed, 4 insertions(+), 9 deletions(-)
> 
> diff --git a/test/validation/odp_pktio.c b/test/validation/odp_pktio.c
> index 9806376..156285a 100644
> --- a/test/validation/odp_pktio.c
> +++ b/test/validation/odp_pktio.c
> @@ -14,7 +14,7 @@
>  #include <stdlib.h>
>  
>  #define PKT_BUF_NUM            32
> -#define PKT_BUF_SIZE           (9*1024)
> +#define PKT_BUF_SIZE           (9 * 1024)
>  #define PKT_LEN_NORMAL         64
>  #define PKT_LEN_JUMBO          (PKT_BUF_SIZE - ODPH_ETHHDR_LEN - \
>                               ODPH_IPV4HDR_LEN - ODPH_UDPHDR_LEN)
> @@ -212,7 +212,7 @@ static int default_pool_create(void)
>       params.type        = ODP_POOL_PACKET;
>  
>       default_pkt_pool = odp_pool_create("pkt_pool_default",
> -                                               ODP_SHM_NULL, &params);
> +                                        ODP_SHM_NULL, &params);
>       if (default_pkt_pool == ODP_POOL_INVALID)
>               return -1;
>  
> @@ -265,7 +265,8 @@ static int create_inq(odp_pktio_t pktio, odp_queue_type_t 
> qtype)
>                odp_pktio_to_u64(pktio));
>       inq_def = odp_queue_lookup(inq_name);
>       if (inq_def == ODP_QUEUE_INVALID)
> -             inq_def = odp_queue_create(inq_name,
> +             inq_def = odp_queue_create(
> +                             inq_name,
>                               ODP_QUEUE_TYPE_PKTIN,
>                               qtype == ODP_QUEUE_TYPE_POLL ? NULL : &qparam);
>  
> @@ -496,8 +497,6 @@ static void pktio_test_mtu(void)
>  
>       ret = odp_pktio_close(pktio);
>       CU_ASSERT(ret == 0);
> -
> -     return;
>  }
>  
>  static void pktio_test_promisc(void)
> @@ -521,8 +520,6 @@ static void pktio_test_promisc(void)
>  
>       ret = odp_pktio_close(pktio);
>       CU_ASSERT(ret == 0);
> -
> -     return;
>  }
>  
>  static void pktio_test_mac(void)
> @@ -547,8 +544,6 @@ static void pktio_test_mac(void)
>  
>       ret = odp_pktio_close(pktio);
>       CU_ASSERT(0 == ret);
> -
> -     return;
>  }
>  
>  static void pktio_test_inq_remdef(void)
> -- 
> 1.9.1
> 

-- 
Stuart.
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp