[dpdk-dev] app/testpmd: remove unnecessary cast

Message ID 20171011065950.59867-1-zhiyong.yang@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Yang, Zhiyong Oct. 11, 2017, 6:59 a.m. UTC
  Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
---
 app/test-pmd/testpmd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Ferruh Yigit Oct. 11, 2017, 10:56 p.m. UTC | #1
On 10/11/2017 7:59 AM, Zhiyong Yang wrote:
> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
  
Ferruh Yigit Oct. 11, 2017, 10:59 p.m. UTC | #2
On 10/11/2017 11:56 PM, Ferruh Yigit wrote:
> On 10/11/2017 7:59 AM, Zhiyong Yang wrote:
>> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> 
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
  
Ferruh Yigit Oct. 11, 2017, 10:59 p.m. UTC | #3
On 10/11/2017 11:56 PM, Ferruh Yigit wrote:
> On 10/11/2017 7:59 AM, Zhiyong Yang wrote:
>> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> 
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied to dpdk-next-net/master, thanks.
  

Patch

diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index 69d64026c..4595e664c 100644
--- a/app/test-pmd/testpmd.c
+++ b/app/test-pmd/testpmd.c
@@ -1321,14 +1321,14 @@  stop_packet_forwarding(void)
 void
 dev_set_link_up(portid_t pid)
 {
-	if (rte_eth_dev_set_link_up((uint8_t)pid) < 0)
+	if (rte_eth_dev_set_link_up(pid) < 0)
 		printf("\nSet link up fail.\n");
 }
 
 void
 dev_set_link_down(portid_t pid)
 {
-	if (rte_eth_dev_set_link_down((uint8_t)pid) < 0)
+	if (rte_eth_dev_set_link_down(pid) < 0)
 		printf("\nSet link down fail.\n");
 }