From patchwork Fri May 7 10:12:47 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shijith Thotton X-Patchwork-Id: 93046 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4DA7FA034F; Fri, 7 May 2021 12:12:56 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C3E544013F; Fri, 7 May 2021 12:12:55 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id 81C9A40040 for ; Fri, 7 May 2021 12:12:54 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 147A6BgV025091; Fri, 7 May 2021 03:12:53 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : mime-version : content-transfer-encoding : content-type; s=pfpt0220; bh=vDzAJq+z63Jqj32uXfehHCIi9+GBIo86A096l6TkJbc=; b=TVFqilQah6g7KUh7vt5tSpCyYymJ9SwU64MHfx5axvjPToJ7bGMYbRjDUbDBGqpq5w6a GqNz8+Z5BhHWHO//4jXg9nvqTp5CxutFqt/d438PVP4JKLggpWr+Q7SCGNKYwYMs4fs5 l5ziIeP32zvKdLxH9L3clu/g+QJuSM2/HrW7vyAGCKqAjkHzN+aScIAq1ceHph7zlHyh mtbGPYQfM92yN/4GGfg39eGDqF7uvGcqn7zoQgkTOlc3bc1Gne2bWZ2gikp3DJ1NdSbO ipbcZz0vqiHp0SCn7jap4IbqL5r1RfkJAJkW5fojgbmzzupiLxVkVGdkOYHW2xeYyXUp Dg== Received: from dc5-exch01.marvell.com ([199.233.59.181]) by mx0a-0016f401.pphosted.com with ESMTP id 38cspj1vja-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Fri, 07 May 2021 03:12:53 -0700 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH01.marvell.com (10.69.176.38) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Fri, 7 May 2021 03:12:51 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Fri, 7 May 2021 03:12:52 -0700 Received: from localhost.localdomain (unknown [10.28.34.29]) by maili.marvell.com (Postfix) with ESMTP id B30FD5B6942; Fri, 7 May 2021 03:12:50 -0700 (PDT) From: Shijith Thotton To: CC: Shijith Thotton , Erik Gabriel Carrillo , Jerin Jacob , "Pavan Nikhilesh" Date: Fri, 7 May 2021 15:42:47 +0530 Message-ID: <7d28b0e4a5e0eb715eaced93b98af833da34b63b.1620382076.git.sthotton@marvell.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-Proofpoint-GUID: xM7pl_ALXKyKIGCC7K42W8l_-RmoKCpx X-Proofpoint-ORIG-GUID: xM7pl_ALXKyKIGCC7K42W8l_-RmoKCpx X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391, 18.0.761 definitions=2021-05-07_03:2021-05-06, 2021-05-07 signatures=0 Subject: [dpdk-dev] [PATCH v1] test/event: fix return value of timer test setup X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Test case setup should return -ENOTSUP, if it is not supported. Fixes: 7d761b07fcf6 ("test/event: add unit tests for periodic timer") Signed-off-by: Shijith Thotton Acked-by: Jerin Jacob --- app/test/test_event_timer_adapter.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/app/test/test_event_timer_adapter.c b/app/test/test_event_timer_adapter.c index c36ab32ac..121c2d93e 100644 --- a/app/test/test_event_timer_adapter.c +++ b/app/test/test_event_timer_adapter.c @@ -303,10 +303,8 @@ _timdev_setup(uint64_t max_tmo_ns, uint64_t bkt_tck_ns, uint64_t flags) "failed to get adapter capabilities"); if (flags & RTE_EVENT_TIMER_ADAPTER_F_PERIODIC && - !(caps & RTE_EVENT_TIMER_ADAPTER_CAP_PERIODIC)) { - printf("Adapter does not support periodic timers\n"); - return TEST_SKIPPED; - } + !(caps & RTE_EVENT_TIMER_ADAPTER_CAP_PERIODIC)) + return -ENOTSUP; if (!(caps & RTE_EVENT_TIMER_ADAPTER_CAP_INTERNAL_PORT)) { timdev = rte_event_timer_adapter_create_ext(&config,