[dpdk-test-report] |WARNING| pw24058 [PATCH] net/ixgbe: fix default MAC setting

checkpatch at dpdk.org checkpatch at dpdk.org
Wed May 3 03:02:24 CEST 2017


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

_coding style issues_


ERROR:BAD_SIGN_OFF: Unrecognized email address: 'Lu, Wenzhuo
'
#18: 
To: Lu, Wenzhuo

WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#25: 
[Wenzhuo] The current behavior of this set_default_mac_addr is to delete all  the existing MAC addresses and only set the assigned address as the default address. This patch is to fix a bug not change the behavior. So have to leave "ixgbe_remove_rar(dev, 0);" here.

WARNING:BAD_SIGN_OFF: email address 'stable at dpdk.org<mailto:stable at dpdk.org>
' might be better as '"stable at dpdk.org" <mailto:stable at dpdk.org>
'
#41: 
Cc: stable at dpdk.org<mailto:stable at dpdk.org>

WARNING:BAD_SIGN_OFF: email address 'Wenzhuo Lu <wenzhuo.lu at intel.com<mailto:wenzhuo.lu at intel.com>>
' might be better as '"Wenzhuo Lu <wenzhuo.lu at intel.com" <mailto:wenzhuo.lu at intel.com>
'
#43: 
Signed-off-by: Wenzhuo Lu <wenzhuo.lu at intel.com<mailto:wenzhuo.lu at intel.com>>

ERROR:DOS_LINE_ENDINGS: DOS line endings
#59: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4661:
+       struct rte_pci_device *pci_dev = IXGBE_DEV_TO_PCI(dev);^M$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#59: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4661:
+       struct rte_pci_device *pci_dev = IXGBE_DEV_TO_PCI(dev);^M$

ERROR:DOS_LINE_ENDINGS: DOS line endings
#60: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4662:
+^M$

ERROR:CORRUPTED_PATCH: patch seems to be corrupt (line wrapped?)
#62: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4663:


ERROR:DOS_LINE_ENDINGS: DOS line endings
#64: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4664:
+       ixgbe_add_rar(dev, addr, 0, pci_dev->max_vfs);^M$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#64: FILE: drivers/net/ixgbe/ixgbe_ethdev.c:4664:
+       ixgbe_add_rar(dev, addr, 0, pci_dev->max_vfs);^M$

total: 5 errors, 5 warnings, 0 checks, 12 lines checked


More information about the test-report mailing list