From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Beilei Xing <beilei.xing@intel.com>
Subject: [dpdk-test-report] |WARNING| pw21231 [PATCH] ppp implemantation
Date: Fri, 3 Mar 2017 08:28:03 +0100 (CET) [thread overview]
Message-ID: <20170303072803.E6AC1F955@dpdk.org> (raw)
In-Reply-To: <1488525977-15321-6-git-send-email-beilei.xing@intel.com>
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/21231
_coding style issues_
WARNING:TYPO_SPELLING: 'implemantation' may be misspelled - perhaps 'implementation'?
#27:
Subject: [dpdk-dev] [PATCH] ppp implemantation
ERROR:CODE_INDENT: code indent should use tabs where possible
#171: FILE: app/test-pmd/config.c:3281:
+ printf("%s: File operations failed
", __func__);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#171: FILE: app/test-pmd/config.c:3281:
+ printf("%s: File operations failed
", __func__);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#174: FILE: app/test-pmd/config.c:3284:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#174: FILE: app/test-pmd/config.c:3284:
+ }$
ERROR:SPACING: space required before the open parenthesis '('
#177: FILE: app/test-pmd/config.c:3287:
+ for(uint32_t i = 0; i < pkg_size; i++) {
ERROR:CODE_INDENT: code indent should use tabs where possible
#223: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:202:
+ i40e_aqc_opc_write_personalization_profile^I= 0x0270,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#223: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:202:
+ i40e_aqc_opc_write_personalization_profile^I= 0x0270,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#224: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:203:
+ i40e_aqc_opc_read_personalization_profile_list^I= 0x0271,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#224: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:203:
+ i40e_aqc_opc_read_personalization_profile_list^I= 0x0271,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#235: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1450:
+ u8 flags;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#235: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1450:
+ u8 flags;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#238: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1453:
+ u8 reserved[3];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#238: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1453:
+ u8 reserved[3];$
ERROR:CODE_INDENT: code indent should use tabs where possible
#239: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1454:
+ __le32 profile_track_id;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#239: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1454:
+ __le32 profile_track_id;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#240: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1455:
+ __le32 addr_high;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#240: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1455:
+ __le32 addr_high;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#241: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1456:
+ __le32 addr_low;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#241: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1456:
+ __le32 addr_low;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#247: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1462:
+ __le32 error_offset;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#247: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1462:
+ __le32 error_offset;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#248: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1463:
+ __le32 error_info;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#248: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1463:
+ __le32 error_info;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#249: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1464:
+ __le32 addr_high;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#249: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1464:
+ __le32 addr_high;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#250: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1465:
+ __le32 addr_low;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#250: FILE: drivers/net/i40e/base/i40e_adminq_cmd.h:1465:
+ __le32 addr_low;$
WARNING:LONG_LINE: line over 90 characters
#329: FILE: drivers/net/i40e/base/i40e_common.c:7054:
+ i40e_fill_default_direct_cmd_desc(&desc, i40e_aqc_opc_read_personalization_profile_list);
ERROR:CODE_INDENT: code indent should use tabs where possible
#367: FILE: drivers/net/i40e/base/i40e_common.c:7092:
+^I^I pkg_hdr->segment_offset[i]);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#561: FILE: drivers/net/i40e/i40e_ethdev.c:11228:
+ struct rte_eth_dev *dev = &rte_eth_devices[port];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#561: FILE: drivers/net/i40e/i40e_ethdev.c:11228:
+ struct rte_eth_dev *dev = &rte_eth_devices[port];$
ERROR:CODE_INDENT: code indent should use tabs where possible
#563: FILE: drivers/net/i40e/i40e_ethdev.c:11230:
+ struct i40e_package_header *pkg_hdr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#563: FILE: drivers/net/i40e/i40e_ethdev.c:11230:
+ struct i40e_package_header *pkg_hdr;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#564: FILE: drivers/net/i40e/i40e_ethdev.c:11231:
+ struct i40e_generic_seg_header *profile_seg_hdr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#564: FILE: drivers/net/i40e/i40e_ethdev.c:11231:
+ struct i40e_generic_seg_header *profile_seg_hdr;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#565: FILE: drivers/net/i40e/i40e_ethdev.c:11232:
+ struct i40e_generic_seg_header *metadata_seg_hdr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#565: FILE: drivers/net/i40e/i40e_ethdev.c:11232:
+ struct i40e_generic_seg_header *metadata_seg_hdr;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#566: FILE: drivers/net/i40e/i40e_ethdev.c:11233:
+ uint32_t track_id;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#566: FILE: drivers/net/i40e/i40e_ethdev.c:11233:
+ uint32_t track_id;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#567: FILE: drivers/net/i40e/i40e_ethdev.c:11234:
+ enum i40e_status_code status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#567: FILE: drivers/net/i40e/i40e_ethdev.c:11234:
+ enum i40e_status_code status;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#569: FILE: drivers/net/i40e/i40e_ethdev.c:11236:
+ pkg_hdr = (struct i40e_package_header *)buff;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#569: FILE: drivers/net/i40e/i40e_ethdev.c:11236:
+ pkg_hdr = (struct i40e_package_header *)buff;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#571: FILE: drivers/net/i40e/i40e_ethdev.c:11238:
+ if (!pkg_hdr) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#571: FILE: drivers/net/i40e/i40e_ethdev.c:11238:
+ if (!pkg_hdr) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#572: FILE: drivers/net/i40e/i40e_ethdev.c:11239:
+ PMD_DRV_LOG(ERR, "Failed to fill the package structure");$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#572: FILE: drivers/net/i40e/i40e_ethdev.c:11239:
+ PMD_DRV_LOG(ERR, "Failed to fill the package structure");$
ERROR:CODE_INDENT: code indent should use tabs where possible
#573: FILE: drivers/net/i40e/i40e_ethdev.c:11240:
+ return -EINVAL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#573: FILE: drivers/net/i40e/i40e_ethdev.c:11240:
+ return -EINVAL;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#574: FILE: drivers/net/i40e/i40e_ethdev.c:11241:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#574: FILE: drivers/net/i40e/i40e_ethdev.c:11241:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#576: FILE: drivers/net/i40e/i40e_ethdev.c:11243:
+ /* Find metadata segment */$
ERROR:CODE_INDENT: code indent should use tabs where possible
#577: FILE: drivers/net/i40e/i40e_ethdev.c:11244:
+ metadata_seg_hdr = i40e_find_segment_in_package(SEGMENT_TYPE_METADATA,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#577: FILE: drivers/net/i40e/i40e_ethdev.c:11244:
+ metadata_seg_hdr = i40e_find_segment_in_package(SEGMENT_TYPE_METADATA,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#578: FILE: drivers/net/i40e/i40e_ethdev.c:11245:
+ pkg_hdr);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#578: FILE: drivers/net/i40e/i40e_ethdev.c:11245:
+ pkg_hdr);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#579: FILE: drivers/net/i40e/i40e_ethdev.c:11246:
+ if (!metadata_seg_hdr) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#579: FILE: drivers/net/i40e/i40e_ethdev.c:11246:
+ if (!metadata_seg_hdr) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#580: FILE: drivers/net/i40e/i40e_ethdev.c:11247:
+ PMD_DRV_LOG(ERR, "Failed to find metadata segment header");$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#580: FILE: drivers/net/i40e/i40e_ethdev.c:11247:
+ PMD_DRV_LOG(ERR, "Failed to find metadata segment header");$
ERROR:CODE_INDENT: code indent should use tabs where possible
#581: FILE: drivers/net/i40e/i40e_ethdev.c:11248:
+ return -EINVAL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#581: FILE: drivers/net/i40e/i40e_ethdev.c:11248:
+ return -EINVAL;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#582: FILE: drivers/net/i40e/i40e_ethdev.c:11249:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#582: FILE: drivers/net/i40e/i40e_ethdev.c:11249:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#583: FILE: drivers/net/i40e/i40e_ethdev.c:11250:
+ track_id = ((struct i40e_metadata_segment *)metadata_seg_hdr)->track_id;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#583: FILE: drivers/net/i40e/i40e_ethdev.c:11250:
+ track_id = ((struct i40e_metadata_segment *)metadata_seg_hdr)->track_id;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#586: FILE: drivers/net/i40e/i40e_ethdev.c:11253:
+ profile_seg_hdr = i40e_find_segment_in_package(SEGMENT_TYPE_I40E,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#586: FILE: drivers/net/i40e/i40e_ethdev.c:11253:
+ profile_seg_hdr = i40e_find_segment_in_package(SEGMENT_TYPE_I40E,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#587: FILE: drivers/net/i40e/i40e_ethdev.c:11254:
+ pkg_hdr);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#587: FILE: drivers/net/i40e/i40e_ethdev.c:11254:
+ pkg_hdr);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#588: FILE: drivers/net/i40e/i40e_ethdev.c:11255:
+ if (!profile_seg_hdr) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#588: FILE: drivers/net/i40e/i40e_ethdev.c:11255:
+ if (!profile_seg_hdr) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#589: FILE: drivers/net/i40e/i40e_ethdev.c:11256:
+ PMD_DRV_LOG(ERR, "Failed to find profile segment header");$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#589: FILE: drivers/net/i40e/i40e_ethdev.c:11256:
+ PMD_DRV_LOG(ERR, "Failed to find profile segment header");$
ERROR:CODE_INDENT: code indent should use tabs where possible
#590: FILE: drivers/net/i40e/i40e_ethdev.c:11257:
+ return -EINVAL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#590: FILE: drivers/net/i40e/i40e_ethdev.c:11257:
+ return -EINVAL;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#591: FILE: drivers/net/i40e/i40e_ethdev.c:11258:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#591: FILE: drivers/net/i40e/i40e_ethdev.c:11258:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#593: FILE: drivers/net/i40e/i40e_ethdev.c:11260:
+ /* Write profile to HW */$
ERROR:CODE_INDENT: code indent should use tabs where possible
#594: FILE: drivers/net/i40e/i40e_ethdev.c:11261:
+ status = i40e_write_profile(hw,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#594: FILE: drivers/net/i40e/i40e_ethdev.c:11261:
+ status = i40e_write_profile(hw,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#598: FILE: drivers/net/i40e/i40e_ethdev.c:11265:
+ if (!status)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#598: FILE: drivers/net/i40e/i40e_ethdev.c:11265:
+ if (!status)$
ERROR:CODE_INDENT: code indent should use tabs where possible
#599: FILE: drivers/net/i40e/i40e_ethdev.c:11266:
+ printf("Write profile successfully.
");$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#599: FILE: drivers/net/i40e/i40e_ethdev.c:11266:
+ printf("Write profile successfully.
");$
ERROR:CODE_INDENT: code indent should use tabs where possible
#601: FILE: drivers/net/i40e/i40e_ethdev.c:11268:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#601: FILE: drivers/net/i40e/i40e_ethdev.c:11268:
+ return status;$
total: 45 errors, 43 warnings, 0 checks, 510 lines checked
parent reply other threads:[~2017-03-03 7:28 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <1488525977-15321-6-git-send-email-beilei.xing@intel.com>]
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=20170303072803.E6AC1F955@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=beilei.xing@intel.com \
--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).