diff mbox

[3/6] validation: cosmetic fixes in odp_pktio.c

Message ID 1432810692-25231-4-git-send-email-christophe.milard@linaro.org
State Superseded
Headers show

Commit Message

Christophe Milard May 28, 2015, 10:58 a.m. UTC
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.milard@linaro.org>
---
 test/validation/odp_pktio.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

Comments

Stuart Haslam June 2, 2015, 9:29 a.m. UTC | #1
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.milard@linaro.org>

Reviewed-by: Stuart Haslam <stuart.haslam@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
>
diff mbox

Patch

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)