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 71090A0A0C for ; Thu, 15 Jul 2021 17:45:46 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6C50E40686; Thu, 15 Jul 2021 17:45:46 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 4E0DA4014D for ; Thu, 15 Jul 2021 17:45:45 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 31919120F1F; Thu, 15 Jul 2021 17:45:45 +0200 (CEST) In-Reply-To: <1626363661-51103-1-git-send-email-fengchengwen@huawei.com> References: <1626363661-51103-1-git-send-email-fengchengwen@huawei.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Chengwen Feng Message-Id: <20210715154545.31919120F1F@dpdk.org> Date: Thu, 15 Jul 2021 17:45:45 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw95904 [PATCH v4] dmadev: introduce DMA device library 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/95904 _coding style issues_ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #205: FILE: lib/dmadev/rte_dmadev.c:40: +#define RTE_DMADEV_VALID_DEV_ID_OR_ERR_RET(dev_id, retval) do { \ + if (!rte_dmadev_is_valid_dev(dev_id)) { \ + RTE_DMADEV_LOG(ERR, "Invalid dev_id=%u ", dev_id); \ + return retval; \ + } \ +} while (0) WARNING:TYPO_SPELLING: 'fileds' may be misspelled - perhaps 'fields'? #1175: FILE: lib/dmadev/rte_dmadev.h:465: + * these fileds to zero. And also there are no WARNING:TYPO_SPELLING: 'ATTEMPED' may be misspelled - perhaps 'ATTEMPTED'? #1374: FILE: lib/dmadev/rte_dmadev.h:664: + RTE_DMA_STATUS_NOT_ATTEMPED, WARNING:TYPO_SPELLING: 'primay' may be misspelled - perhaps 'primary'? #1891: FILE: lib/dmadev/rte_dmadev_core.h:166: + * It's the primay process's responsibility to deinitialize this field WARNING:TYPO_SPELLING: 'process'' may be misspelled - perhaps 'process's'? #1891: FILE: lib/dmadev/rte_dmadev_core.h:166: + * It's the primay process's responsibility to deinitialize this field total: 0 errors, 5 warnings, 1870 lines checked Warning in lib/dmadev/rte_dmadev.c: Using RTE_LOG_REGISTER, prefer RTE_LOG_REGISTER_(DEFAULT|SUFFIX)