From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0728FA0C58 for ; Mon, 23 Aug 2021 21:41:20 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F0EA24122B; Mon, 23 Aug 2021 21:41:19 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 2C42E41227 for ; Mon, 23 Aug 2021 21:41:19 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 27EF81226B3; Mon, 23 Aug 2021 21:41:19 +0200 (CEST) In-Reply-To: <20210823194020.1229-5-pbhagavatula@marvell.com> References: <20210823194020.1229-5-pbhagavatula@marvell.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Message-Id: <20210823194119.27EF81226B3@dpdk.org> Date: Mon, 23 Aug 2021 21:41:19 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw97244 [RFC 05/15] eventdev: add helper functions for new driver API X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/97244 _coding style issues_ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #182: FILE: lib/eventdev/eventdev_pmd.h:1290: +#define _RTE_EVENT_ENQ_DEF(fn) \ + _RTE_EVENT_ENQ_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #182: FILE: lib/eventdev/eventdev_pmd.h:1290: +#define _RTE_EVENT_ENQ_DEF(fn) \ + _RTE_EVENT_ENQ_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #189: FILE: lib/eventdev/eventdev_pmd.h:1297: +#define _RTE_EVENT_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #189: FILE: lib/eventdev/eventdev_pmd.h:1297: +#define _RTE_EVENT_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #196: FILE: lib/eventdev/eventdev_pmd.h:1304: +#define _RTE_EVENT_DEQ_DEF(fn) \ + _RTE_EVENT_DEQ_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, timeout_ticks); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #196: FILE: lib/eventdev/eventdev_pmd.h:1304: +#define _RTE_EVENT_DEQ_DEF(fn) \ + _RTE_EVENT_DEQ_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, timeout_ticks); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #203: FILE: lib/eventdev/eventdev_pmd.h:1311: +#define _RTE_EVENT_DEQ_BURST_DEF(fn) \ + _RTE_EVENT_DEQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events, timeout_ticks); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #203: FILE: lib/eventdev/eventdev_pmd.h:1311: +#define _RTE_EVENT_DEQ_BURST_DEF(fn) \ + _RTE_EVENT_DEQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events, timeout_ticks); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #210: FILE: lib/eventdev/eventdev_pmd.h:1318: +#define _RTE_EVENT_TXA_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_TXA_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #210: FILE: lib/eventdev/eventdev_pmd.h:1318: +#define _RTE_EVENT_TXA_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_TXA_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #217: FILE: lib/eventdev/eventdev_pmd.h:1325: +#define _RTE_EVENT_CA_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_CA_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #217: FILE: lib/eventdev/eventdev_pmd.h:1325: +#define _RTE_EVENT_CA_ENQ_BURST_DEF(fn) \ + _RTE_EVENT_CA_ENQ_BURST_PROTO(fn) \ + { \ + void *port = _rte_event_dev_prolog(dev_id, port_id); \ + return fn(port, ev, nb_events); \ + } total: 6 errors, 6 warnings, 606 lines checked