|WARNING| pw104435 [PATCH v2 1/2] common/sfc_efx/base: support NIC DMA memory regions API

automatic DPDK test reports
 help / color / mirror / Atom feed

From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Subject: |WARNING| pw104435 [PATCH v2 1/2] common/sfc_efx/base: support NIC DMA memory regions API
Date: Wed, 17 Nov 2021 08:07:22 +0100 (CET)	[thread overview]
Message-ID: <20211117070722.1B4D3120735@dpdk.org> (raw)
In-Reply-To: <20211117070545.4004374-2-andrew.rybchenko@oktetlabs.ru>

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/104435

_coding style issues_


ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#134: FILE: drivers/common/sfc_efx/base/ef10_nic.c:1899:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#303: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3270:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#310: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3277:
+	return (rc);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#349: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3316:
+	EFX_STATIC_ASSERT(sizeof (*regions) == DESC_ADDR_REGION_LEN);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#354: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3321:
+	if (alloc_size / sizeof (endrip->endri_regions[0]) != nregions) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#377: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3344:
+^I^I        GET_DESC_ADDR_REGIONS_OUT_REGIONS, i,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#378: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3345:
+^I^I        DESC_ADDR_REGION_DESC_ADDR_BASE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#382: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3349:
+^I^I        GET_DESC_ADDR_REGIONS_OUT_REGIONS, i,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#383: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3350:
+^I^I        DESC_ADDR_REGION_TRGT_ADDR_BASE);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#387: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3354:
+^I^I        GET_DESC_ADDR_REGIONS_OUT_REGIONS, i,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#388: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3355:
+^I^I        DESC_ADDR_REGION_WINDOW_SIZE_LOG2);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#392: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3359:
+^I^I        GET_DESC_ADDR_REGIONS_OUT_REGIONS, i,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#393: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3360:
+^I^I        DESC_ADDR_REGION_TRGT_ADDR_ALIGN_LOG2);$

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#407: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3374:
+	return (rc);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#436: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3403:
+	EFX_STATIC_ASSERT(sizeof (*trgt_addr_base) ==

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#449: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3416:
+		EFX_STATIC_ASSERT(sizeof (1U) * 8 >=

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#473: FILE: drivers/common/sfc_efx/base/efx_mcdi.c:3440:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#572: FILE: drivers/common/sfc_efx/base/efx_nic.c:1341:
+		return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#583: FILE: drivers/common/sfc_efx/base/efx_nic.c:1352:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#633: FILE: drivers/common/sfc_efx/base/efx_nic.c:1402:
+		return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#645: FILE: drivers/common/sfc_efx/base/efx_nic.c:1414:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#689: FILE: drivers/common/sfc_efx/base/efx_nic.c:1458:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#698: FILE: drivers/common/sfc_efx/base/efx_nic.c:1467:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#742: FILE: drivers/common/sfc_efx/base/efx_nic.c:1511:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#756: FILE: drivers/common/sfc_efx/base/efx_nic.c:1525:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#761: FILE: drivers/common/sfc_efx/base/efx_nic.c:1530:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#794: FILE: drivers/common/sfc_efx/base/efx_nic.c:1563:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#845: FILE: drivers/common/sfc_efx/base/efx_nic.c:1614:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#864: FILE: drivers/common/sfc_efx/base/efx_nic.c:1633:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#934: FILE: drivers/common/sfc_efx/base/efx_nic.c:1703:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#992: FILE: drivers/common/sfc_efx/base/efx_nic.c:1761:
+	return (rc);

total: 27 errors, 4 warnings, 892 lines checked

          parent reply	other threads:[~2021-11-17  7:07 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20211117070545.4004374-2-andrew.rybchenko@oktetlabs.ru>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211117070722.1B4D3120735@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=andrew.rybchenko@oktetlabs.ru \
    --cc=test-report@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Be sure your reply has a Subject: header at the top and a blank line before the message body.

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).