[dpdk-test-report] |WARNING| [PATCH 06/56] net/sfc: import libefx MCDI implementation

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Nov 21 16:03:06 CET 2016
Previous message: [dpdk-test-report] |SUCCESS| [PATCH 33/56] net/sfc: add device configure and close stubs
Next message: [dpdk-test-report] |WARNING| [PATCH 21/56] net/sfc: import libefx RSS support
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/17123

_coding style issues_


WARNING:TYPO_SPELLING: 'managment' may be misspelled - perhaps 'management'?
#61: 
Implement interface to talk to NIC managment CPU. Provide

CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#114: FILE: drivers/net/sfc/efx/base/efx.h:202:
+extern	__checkReturn	efx_rc_t

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around ev[index]
#204: FILE: drivers/net/sfc/efx/base/efx_ev.c:328:
+				    &(ev[index]), eecp, arg);

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#363: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:69:
+	switch (enp->en_family) {
+

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#380: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:86:
+		if ((rc = emcop->emco_init(enp, emtp)) != 0)

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#387: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:93:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#400: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:106:
+	return (rc);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#407: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:113:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#427: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:133:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#457: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:163:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#468: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:174:
+	return (available);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#489: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:195:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#537: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:243:
+		hdr_len = sizeof (hdr);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#553: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:259:
+		hdr_len = sizeof (hdr[0]);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#575: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:281:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#586: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:292:
+	efx_mcdi_read_response(enp, &hdr[0], 0, sizeof (hdr[0]));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#587: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:293:
+	hdr_len = sizeof (hdr[0]);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#596: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:302:
+		efx_mcdi_read_response(enp, &hdr[1], hdr_len, sizeof (hdr[1]));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#597: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:303:
+		hdr_len += sizeof (hdr[1]);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#619: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:325:
+		unsigned int err_len = MIN(data_len, sizeof (err));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#626: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:332:
+		if (err_len >= (MC_CMD_ERR_CODE_OFST + sizeof (efx_dword_t)))

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#629: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:335:
+		if (err_len >= (MC_CMD_ERR_ARG_OFST + sizeof (efx_dword_t)))

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#668: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:374:
+	hdr_len = sizeof (hdr[0]);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#675: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:381:
+		efx_mcdi_read_response(enp, &hdr[1], hdr_len, sizeof (hdr[1]));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#676: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:382:
+		hdr_len += sizeof (hdr[1]);

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#686: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:392:
+
+}

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#693: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:399:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#711: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:417:
+		if ((rc = efx_mcdi_poll_reboot(enp)) != 0) {

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#726: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:432:
+		return (B_FALSE);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#740: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:446:
+	if ((rc = emrp->emr_rc) != 0)

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#744: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:450:
+	return (B_TRUE);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#753: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:459:
+	return (B_TRUE);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#760: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:466:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#797: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:503:
+	return (aborted);

CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#815: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:521:
+{
+

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#819: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:525:
+		return (EACCES);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EACCES)
#819: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:525:
+		return (EACCES);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#821: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:527:
+		return (ENOENT);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOENT)
#821: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:527:
+		return (ENOENT);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#823: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:529:
+		return (EINTR);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINTR)
#823: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:529:
+		return (EINTR);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#825: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:531:
+		return (EACCES);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EACCES)
#825: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:531:
+		return (EACCES);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#827: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:533:
+		return (EBUSY);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EBUSY)
#827: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:533:
+		return (EBUSY);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#829: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:535:
+		return (EINVAL);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#829: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:535:
+		return (EINVAL);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#831: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:537:
+		return (EDEADLK);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EDEADLK)
#831: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:537:
+		return (EDEADLK);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#833: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:539:
+		return (ENOTSUP);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#833: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:539:
+		return (ENOTSUP);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#835: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:541:
+		return (ETIMEDOUT);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ETIMEDOUT)
#835: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:541:
+		return (ETIMEDOUT);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#837: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:543:
+		return (ENOTSUP);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#837: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:543:
+		return (ENOTSUP);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#839: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:545:
+		return (EALREADY);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EALREADY)
#839: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:545:
+		return (EALREADY);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#843: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:549:
+		return (EEXIST);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EEXIST)
#843: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:549:
+		return (EEXIST);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#846: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:552:
+		return (EAGAIN);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EAGAIN)
#846: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:552:
+		return (EAGAIN);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#850: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:556:
+		return (ENOSPC);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSPC)
#850: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:556:
+		return (ENOSPC);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#853: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:559:
+		return (ERANGE);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ERANGE)
#853: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:559:
+		return (ERANGE);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#856: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:562:
+		return (ENOMEM);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOMEM)
#856: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:562:
+		return (ENOMEM);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#858: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:564:
+		return (ENOENT);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOENT)
#858: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:564:
+		return (ENOENT);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#860: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:566:
+		return (ENOENT);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOENT)
#860: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:566:
+		return (ENOENT);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#862: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:568:
+		return (ENODEV);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENODEV)
#862: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:568:
+		return (ENODEV);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#864: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:570:
+		return (EINVAL);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#864: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:570:
+		return (EINVAL);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#866: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:572:
+		return (ENODEV);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENODEV)
#866: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:572:
+		return (ENODEV);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#868: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:574:
+		return (EINVAL);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#868: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:574:
+		return (EINVAL);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#870: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:576:
+		return (EINVAL);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#870: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:576:
+		return (EINVAL);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#872: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:578:
+		return (EINVAL);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EINVAL)
#872: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:578:
+		return (EINVAL);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#874: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:580:
+		return (EEXIST);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EEXIST)
#874: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:580:
+		return (EEXIST);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#877: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:583:
+		return (EAGAIN);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EAGAIN)
#877: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:583:
+		return (EAGAIN);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#881: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:587:
+		return (EIO);

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -EIO)
#881: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:587:
+		return (EIO);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#946: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:652:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

WARNING:BRACES: braces {} are not necessary for single statement blocks
#988: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:694:
+	if (errcode == 0) {
+		efx_mcdi_finish_response(enp, emrp);
+	}

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#1000: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:706:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

CHECK:SPACING: No space is necessary after a cast
#1036: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:742:
+		(void) efx_mcdi_poll_reboot(enp);

CHECK:SPACING: No space is necessary after a cast
#1068: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:774:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1068: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:774:
+	(void) memset(payload, 0, sizeof (payload));

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#1084: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:790:
+		version[0] = version[1] = version[2] = version[3] = 0;

CHECK:SPACING: No space is necessary after a cast
#1109: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:815:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1109: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:815:
+	(void) memset(payload, 0, sizeof (payload));

CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#1121: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:827:
+		version[0] = version[1] = version[2] = version[3] = 0;

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1144: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:850:
+		memcpy(versionp, version, sizeof (version));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1150: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:856:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1161: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:867:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1182: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:888:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1182: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:888:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1206: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:912:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1211: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:917:
+	return (rc);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1218: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:924:
+	return (efx_mcdi_do_reboot(enp, B_FALSE));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1225: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:931:
+	return (efx_mcdi_do_reboot(enp, B_TRUE));

CHECK:SPACING: No space is necessary after a cast
#1255: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:961:
+		(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1255: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:961:
+		(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1284: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:990:
+		return (0);

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1309: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1015:
+		ofst += sizeof (efx_dword_t);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1314: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1020:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1321: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1027:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1339: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1045:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1339: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1045:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1367: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1073:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1374: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1080:
+	return (rc);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#1384: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1090:
+	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);

CHECK:SPACING: No space is necessary after a cast
#1390: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1096:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1390: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1096:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1444: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1150:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1455: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1161:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1470: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1176:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1470: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1176:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1496: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1202:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1503: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1209:
+	return (rc);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#1510: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1216:
+	efx_port_t *epp = &(enp->en_port);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#1511: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1217:
+	efx_nic_cfg_t *encp = &(enp->en_nic_cfg);

CHECK:SPACING: No space is necessary after a cast
#1517: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1223:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1517: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1223:
+	(void) memset(payload, 0, sizeof (payload));

CHECK:SPACING: No space is necessary after a cast
#1538: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1244:
+	(void) strncpy(encp->enc_phy_name,

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1540: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1246:
+		MIN(sizeof (encp->enc_phy_name) - 1,

CHECK:SPACING: No space is necessary after a cast
#1543: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1249:
+	(void) memset(encp->enc_phy_revision, 0,

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1544: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1250:
+	    sizeof (encp->enc_phy_revision));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1547: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1253:
+		MIN(sizeof (encp->enc_phy_revision) - 1,

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1572: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1278:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1579: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1285:
+	return (rc);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1591: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1297:
+		if ((rc = emcop->emco_feature_supported(enp,

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1599: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1305:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1604: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1310:
+	return (rc);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1616: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1322:
+		if ((rc = emcop->emco_feature_supported(enp,

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1624: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1330:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1629: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1335:
+	return (rc);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1641: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1347:
+		if ((rc = emcop->emco_feature_supported(enp,

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1649: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1355:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1654: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1360:
+	return (rc);

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1666: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1372:
+		if ((rc = emcop->emco_feature_supported(enp,

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1674: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1380:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1679: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1385:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1693: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1399:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1693: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1399:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1711: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1417:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1716: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1422:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1732: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1438:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1732: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1438:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1756: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1462:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1761: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1467:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1775: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1481:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1775: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1481:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1794: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1500:
+	if (enabledp != NULL) {
+		*enabledp = MCDI_OUT_DWORD(req, GET_WORKAROUNDS_OUT_ENABLED);
+	}

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1798: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1504:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1803: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1509:
+	return (rc);

CHECK:SPACING: No space is necessary after a cast
#1828: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1534:
+	(void) memset(payload, 0, sizeof (payload));

WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1828: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1534:
+	(void) memset(payload, 0, sizeof (payload));

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1861: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1567:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1870: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1576:
+	return (rc);

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#1902: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1608:
+	efx_port_t *epp = &(enp->en_port);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2007: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1713:
+	return (0);

ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2016: FILE: drivers/net/sfc/efx/base/efx_mcdi.c:1722:
+	return (rc);

CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_type' may be better as '(_type)' to avoid precedence issues
#2227: FILE: drivers/net/sfc/efx/base/efx_mcdi.h:202:
+#define	MCDI_IN(_emr, _type, _ofst)					\
+	((_type *)((_emr).emr_in_buf + (_ofst)))

CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_type' may be better as '(_type)' to avoid precedence issues
#2352: FILE: drivers/net/sfc/efx/base/efx_mcdi.h:327:
+#define	MCDI_OUT(_emr, _type, _ofst)					\
+	((_type *)((_emr).emr_out_buf + (_ofst)))

CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_eqp' may be better as '(_eqp)' to avoid precedence issues
#2374: FILE: drivers/net/sfc/efx/base/efx_mcdi.h:349:
+#define	MCDI_EV_FIELD(_eqp, _field)					\
+	EFX_QWORD_FIELD(*_eqp, MCDI_EVENT_ ## _field)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_edp' may be better as '(_edp)' to avoid precedence issues
#2377: FILE: drivers/net/sfc/efx/base/efx_mcdi.h:352:
+#define	MCDI_CMD_DWORD_FIELD(_edp, _field)				\
+	EFX_DWORD_FIELD(*_edp, MC_CMD_ ## _field)

total: 77 errors, 59 warnings, 38 checks, 2289 lines checked
Previous message: [dpdk-test-report] |SUCCESS| [PATCH 33/56] net/sfc: add device configure and close stubs
Next message: [dpdk-test-report] |WARNING| [PATCH 21/56] net/sfc: import libefx RSS support
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
More information about the test-report mailing list