From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 418F8A04B1 for ; Thu, 24 Sep 2020 14:21:45 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 649301DE55; Thu, 24 Sep 2020 14:21:41 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 4CD8E1DFEC; Thu, 24 Sep 2020 14:21:36 +0200 (CEST) In-Reply-To: <1600949555-28043-43-git-send-email-arybchenko@solarflare.com> References: <1600949555-28043-43-git-send-email-arybchenko@solarflare.com> To: test-report@dpdk.org Cc: Andrew Rybchenko Message-Id: <20200924122136.4CD8E1DFEC@dpdk.org> Date: Thu, 24 Sep 2020 14:21:36 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw78718 [PATCH v3 42/60] common/sfc_efx/base: choose smallest Rx prefix on Riverhead 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: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/78718 _coding style issues_ WARNING:SPACING: space prohibited between function name and open parenthesis '(' #220: FILE: drivers/common/sfc_efx/base/rhead_rx.c:317: + EFX_STATIC_ASSERT(sizeof (idsp[0]) == WARNING:SPACING: space prohibited between function name and open parenthesis '(' #224: FILE: drivers/common/sfc_efx/base/rhead_rx.c:321: + *nids * sizeof (idsp[0])); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #226: FILE: drivers/common/sfc_efx/base/rhead_rx.c:323: + return (0); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #235: FILE: drivers/common/sfc_efx/base/rhead_rx.c:332: + return (rc); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #262: FILE: drivers/common/sfc_efx/base/rhead_rx.c:359: + return (efx_mcdi_to_rx_prefix_field[mcdi_idx]); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #298: FILE: drivers/common/sfc_efx/base/rhead_rx.c:395: + return (efx_rx_prefix_field_to_mcdi[field]); WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL) #314: FILE: drivers/common/sfc_efx/base/rhead_rx.c:411: + return (EINVAL); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #321: FILE: drivers/common/sfc_efx/base/rhead_rx.c:418: + return (0); WARNING:SPACING: space prohibited between function name and open parenthesis '(' #380: FILE: drivers/common/sfc_efx/base/rhead_rx.c:477: + memset(erplp, 0, sizeof (*erplp)); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #421: FILE: drivers/common/sfc_efx/base/rhead_rx.c:518: + return (rc); WARNING:SPACING: space prohibited between function name and open parenthesis '(' #441: FILE: drivers/common/sfc_efx/base/rhead_rx.c:538: + memset(erplp, 0, sizeof (*erplp)); ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required #480: FILE: drivers/common/sfc_efx/base/rhead_rx.c:577: + return (rc); ERROR:ASSIGN_IN_IF: do not use assignment in if condition #506: FILE: drivers/common/sfc_efx/base/rhead_rx.c:634: + if ((rc = rhead_rx_choose_prefix_id(enp, total: 8 errors, 5 warnings, 406 lines checked