From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by dpdk.org (Postfix, from userid 1017) id F1B5F1B3D3; Mon, 2 Apr 2018 21:41:09 +0200 (CEST) In-Reply-To: <1522697994-22515-2-git-send-email-erik.g.carrillo@intel.com> References: <1522697994-22515-2-git-send-email-erik.g.carrillo@intel.com> To: test-report@dpdk.org Cc: Erik Carrillo Message-Id: <20180402194109.F1B5F1B3D3@dpdk.org> Date: Mon, 2 Apr 2018 21:41:09 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw36892 [PATCH v9 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: Mon, 02 Apr 2018 19:41:10 -0000 Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/36892 _coding style issues_ ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar" #289: 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" #319: 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 #619: 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 #623: 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 #628: 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