From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id 4B569AAC8; Thu, 29 Mar 2018 23:28:59 +0200 (CEST) In-Reply-To: <1522358852-3630-2-git-send-email-erik.g.carrillo@intel.com> References: <1522358852-3630-2-git-send-email-erik.g.carrillo@intel.com> To: test-report@dpdk.org Cc: Erik Carrillo Message-Id: <20180329212859.4B569AAC8@dpdk.org> Date: Thu, 29 Mar 2018 23:28:59 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw36754 [PATCH v8 1/9] eventtimer: introduce event timer adapter X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 29 Mar 2018 21:28:59 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/36754 _coding style issues_ ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar" #288: FILE: lib/librte_eventdev/rte_event_timer_adapter.h:238: +struct rte_event_timer_adapter * __rte_experimental ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar" #318: FILE: lib/librte_eventdev/rte_event_timer_adapter.h:268: +struct rte_event_timer_adapter * __rte_experimental WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #618: FILE: lib/librte_eventdev/rte_event_timer_adapter.h:568: +#define ADAPTER_VALID_OR_ERR_RET(adapter, retval) do { \ + if (adapter == NULL || !adapter->allocated) \ + return retval; \ +} while (0) WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #622: FILE: lib/librte_eventdev/rte_event_timer_adapter.h:572: +#define FUNC_PTR_OR_ERR_RET(func, errval) do { \ + if ((func) == NULL) \ + return errval; \ +} while (0) WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #627: FILE: lib/librte_eventdev/rte_event_timer_adapter.h:577: +#define FUNC_PTR_OR_NULL_RET_WITH_ERRNO(func, errval) do { \ + if ((func) == NULL) { \ + rte_errno = errval; \ + return NULL; \ + } \ +} while (0) total: 2 errors, 3 warnings, 732 lines checked