From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Ravi Kumar <ravi1.kumar@amd.com>
Subject: [dpdk-test-report] |WARNING| pw28722 [PATCH 2/4] net/axgbe : Add PMD support for AMD XGBE Ethernet
Date: Thu, 14 Sep 2017 10:57:33 +0200 (CEST) [thread overview]
Message-ID: <20170914085733.4EC151B19F@dpdk.org> (raw)
In-Reply-To: <1505324838-25734-2-git-send-email-ravi1.kumar@amd.com>
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/28722
_coding style issues_
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#309: FILE: drivers/net/axgbe/axgbe_common.h:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#311: FILE: drivers/net/axgbe/axgbe_common.h:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#312: FILE: drivers/net/axgbe/axgbe_common.h:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#314: FILE: drivers/net/axgbe/axgbe_common.h:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#315: FILE: drivers/net/axgbe/axgbe_common.h:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'arr' - possible side-effects?
#402: FILE: drivers/net/axgbe/axgbe_common.h:165:
+#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#1679: FILE: drivers/net/axgbe/axgbe_common.h:1442:
+#define AXGBE_REG_WRITE(val, reg) do { \
+ (*((volatile uint32_t *) (reg))) = val; \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_var' - possible side-effects?
#1746: FILE: drivers/net/axgbe/axgbe_common.h:1509:
+#define SET_BITS(_var, _index, _width, _val) \
+do { \
+ (_var) &= ~(((0x1 << (_width)) - 1) << (_index)); \
+ (_var) |= (((_val) & ((0x1 << (_width)) - 1)) << (_index)); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_index' - possible side-effects?
#1746: FILE: drivers/net/axgbe/axgbe_common.h:1509:
+#define SET_BITS(_var, _index, _width, _val) \
+do { \
+ (_var) &= ~(((0x1 << (_width)) - 1) << (_index)); \
+ (_var) |= (((_val) & ((0x1 << (_width)) - 1)) << (_index)); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_width' - possible side-effects?
#1746: FILE: drivers/net/axgbe/axgbe_common.h:1509:
+#define SET_BITS(_var, _index, _width, _val) \
+do { \
+ (_var) &= ~(((0x1 << (_width)) - 1) << (_index)); \
+ (_var) |= (((_val) & ((0x1 << (_width)) - 1)) << (_index)); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_var' - possible side-effects?
#1755: FILE: drivers/net/axgbe/axgbe_common.h:1518:
+#define SET_BITS_LE(_var, _index, _width, _val) \
+do { \
+ (_var) &= rte_cpu_to_le_32(~(((0x1 << (_width)) - 1) << (_index)));\
+ (_var) |= rte_cpu_to_le_32((((_val) & \
+ ((0x1 << (_width)) - 1)) << (_index))); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_index' - possible side-effects?
#1755: FILE: drivers/net/axgbe/axgbe_common.h:1518:
+#define SET_BITS_LE(_var, _index, _width, _val) \
+do { \
+ (_var) &= rte_cpu_to_le_32(~(((0x1 << (_width)) - 1) << (_index)));\
+ (_var) |= rte_cpu_to_le_32((((_val) & \
+ ((0x1 << (_width)) - 1)) << (_index))); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_width' - possible side-effects?
#1755: FILE: drivers/net/axgbe/axgbe_common.h:1518:
+#define SET_BITS_LE(_var, _index, _width, _val) \
+do { \
+ (_var) &= rte_cpu_to_le_32(~(((0x1 << (_width)) - 1) << (_index)));\
+ (_var) |= rte_cpu_to_le_32((((_val) & \
+ ((0x1 << (_width)) - 1)) << (_index))); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1797: FILE: drivers/net/axgbe/axgbe_common.h:1560:
+#define AXGMAC_IOREAD(_pdata, _reg) \
+ ioread32((_pdata)->xgmac_regs + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1805: FILE: drivers/net/axgbe/axgbe_common.h:1568:
+#define AXGMAC_IOWRITE(_pdata, _reg, _val) \
+ iowrite32((_val), (_pdata)->xgmac_regs + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1808: FILE: drivers/net/axgbe/axgbe_common.h:1571:
+#define AXGMAC_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1808: FILE: drivers/net/axgbe/axgbe_common.h:1571:
+#define AXGMAC_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1821: FILE: drivers/net/axgbe/axgbe_common.h:1584:
+#define AXGMAC_MTL_IOREAD(_pdata, _n, _reg) \
+ ioread32((_pdata)->xgmac_regs + \
+ MTL_Q_BASE + ((_n) * MTL_Q_INC) + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1830: FILE: drivers/net/axgbe/axgbe_common.h:1593:
+#define AXGMAC_MTL_IOWRITE(_pdata, _n, _reg, _val) \
+ iowrite32((_val), (_pdata)->xgmac_regs + \
+ MTL_Q_BASE + ((_n) * MTL_Q_INC) + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1834: FILE: drivers/net/axgbe/axgbe_common.h:1597:
+#define AXGMAC_MTL_IOWRITE_BITS(_pdata, _n, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_MTL_IOREAD((_pdata), (_n), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_MTL_IOWRITE((_pdata), (_n), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_n' - possible side-effects?
#1834: FILE: drivers/net/axgbe/axgbe_common.h:1597:
+#define AXGMAC_MTL_IOWRITE_BITS(_pdata, _n, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_MTL_IOREAD((_pdata), (_n), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_MTL_IOWRITE((_pdata), (_n), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1834: FILE: drivers/net/axgbe/axgbe_common.h:1597:
+#define AXGMAC_MTL_IOWRITE_BITS(_pdata, _n, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_MTL_IOREAD((_pdata), (_n), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_MTL_IOWRITE((_pdata), (_n), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1847: FILE: drivers/net/axgbe/axgbe_common.h:1610:
+#define AXGMAC_DMA_IOREAD(_channel, _reg) \
+ ioread32((_channel)->dma_regs + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1855: FILE: drivers/net/axgbe/axgbe_common.h:1618:
+#define AXGMAC_DMA_IOWRITE(_channel, _reg, _val) \
+ iowrite32((_val), (_channel)->dma_regs + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_channel' - possible side-effects?
#1858: FILE: drivers/net/axgbe/axgbe_common.h:1621:
+#define AXGMAC_DMA_IOWRITE_BITS(_channel, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_DMA_IOREAD((_channel), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_DMA_IOWRITE((_channel), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1858: FILE: drivers/net/axgbe/axgbe_common.h:1621:
+#define AXGMAC_DMA_IOWRITE_BITS(_channel, _reg, _field, _val) \
+do { \
+ u32 reg_val = AXGMAC_DMA_IOREAD((_channel), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ AXGMAC_DMA_IOWRITE((_channel), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1905: FILE: drivers/net/axgbe/axgbe_common.h:1668:
+#define XSIR0_IOREAD(_pdata, _reg) \
+ ioread16((_pdata)->sir0_regs + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1913: FILE: drivers/net/axgbe/axgbe_common.h:1676:
+#define XSIR0_IOWRITE(_pdata, _reg, _val) \
+ iowrite16((_val), (_pdata)->sir0_regs + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1916: FILE: drivers/net/axgbe/axgbe_common.h:1679:
+#define XSIR0_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XSIR0_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XSIR0_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1916: FILE: drivers/net/axgbe/axgbe_common.h:1679:
+#define XSIR0_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XSIR0_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XSIR0_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1925: FILE: drivers/net/axgbe/axgbe_common.h:1688:
+#define XSIR1_IOREAD(_pdata, _reg) \
+ ioread16((_pdata)->sir1_regs + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1933: FILE: drivers/net/axgbe/axgbe_common.h:1696:
+#define XSIR1_IOWRITE(_pdata, _reg, _val) \
+ iowrite16((_val), (_pdata)->sir1_regs + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1936: FILE: drivers/net/axgbe/axgbe_common.h:1699:
+#define XSIR1_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XSIR1_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XSIR1_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1936: FILE: drivers/net/axgbe/axgbe_common.h:1699:
+#define XSIR1_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XSIR1_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XSIR1_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1948: FILE: drivers/net/axgbe/axgbe_common.h:1711:
+#define XRXTX_IOREAD(_pdata, _reg) \
+ ioread16((_pdata)->rxtx_regs + _reg)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_reg' may be better as '(_reg)' to avoid precedence issues
#1956: FILE: drivers/net/axgbe/axgbe_common.h:1719:
+#define XRXTX_IOWRITE(_pdata, _reg, _val) \
+ iowrite16((_val), (_pdata)->rxtx_regs + _reg)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1959: FILE: drivers/net/axgbe/axgbe_common.h:1722:
+#define XRXTX_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XRXTX_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XRXTX_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1959: FILE: drivers/net/axgbe/axgbe_common.h:1722:
+#define XRXTX_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u16 reg_val = XRXTX_IOREAD((_pdata), _reg); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XRXTX_IOWRITE((_pdata), _reg, reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#1992: FILE: drivers/net/axgbe/axgbe_common.h:1755:
+#define XP_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = XP_IOREAD((_pdata), (_reg)); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XP_IOWRITE((_pdata), (_reg), reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#1992: FILE: drivers/net/axgbe/axgbe_common.h:1755:
+#define XP_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = XP_IOREAD((_pdata), (_reg)); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XP_IOWRITE((_pdata), (_reg), reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#2025: FILE: drivers/net/axgbe/axgbe_common.h:1788:
+#define XI2C_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = XI2C_IOREAD((_pdata), (_reg)); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XI2C_IOWRITE((_pdata), (_reg), reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#2025: FILE: drivers/net/axgbe/axgbe_common.h:1788:
+#define XI2C_IOWRITE_BITS(_pdata, _reg, _field, _val) \
+do { \
+ u32 reg_val = XI2C_IOREAD((_pdata), (_reg)); \
+ SET_BITS(reg_val, \
+ _reg##_##_field##_INDEX, \
+ _reg##_##_field##_WIDTH, (_val)); \
+ XI2C_IOWRITE((_pdata), (_reg), reg_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#2039: FILE: drivers/net/axgbe/axgbe_common.h:1802:
+#define XMDIO_READ(_pdata, _mmd, _reg) \
+ ((_pdata)->hw_if.read_mmd_regs((_pdata), 0, \
+ MII_ADDR_C45 | (_mmd << 16) | ((_reg) & 0xffff)))
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_mmd' may be better as '(_mmd)' to avoid precedence issues
#2039: FILE: drivers/net/axgbe/axgbe_common.h:1802:
+#define XMDIO_READ(_pdata, _mmd, _reg) \
+ ((_pdata)->hw_if.read_mmd_regs((_pdata), 0, \
+ MII_ADDR_C45 | (_mmd << 16) | ((_reg) & 0xffff)))
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_mask' may be better as '(_mask)' to avoid precedence issues
#2043: FILE: drivers/net/axgbe/axgbe_common.h:1806:
+#define XMDIO_READ_BITS(_pdata, _mmd, _reg, _mask) \
+ (XMDIO_READ((_pdata), _mmd, _reg) & _mask)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#2046: FILE: drivers/net/axgbe/axgbe_common.h:1809:
+#define XMDIO_WRITE(_pdata, _mmd, _reg, _val) \
+ ((_pdata)->hw_if.write_mmd_regs((_pdata), 0, \
+ MII_ADDR_C45 | (_mmd << 16) | ((_reg) & 0xffff), (_val)))
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_mmd' may be better as '(_mmd)' to avoid precedence issues
#2046: FILE: drivers/net/axgbe/axgbe_common.h:1809:
+#define XMDIO_WRITE(_pdata, _mmd, _reg, _val) \
+ ((_pdata)->hw_if.write_mmd_regs((_pdata), 0, \
+ MII_ADDR_C45 | (_mmd << 16) | ((_reg) & 0xffff), (_val)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_pdata' - possible side-effects?
#2050: FILE: drivers/net/axgbe/axgbe_common.h:1813:
+#define XMDIO_WRITE_BITS(_pdata, _mmd, _reg, _mask, _val) \
+do { \
+ u32 mmd_val = XMDIO_READ((_pdata), _mmd, _reg); \
+ mmd_val &= ~_mask; \
+ mmd_val |= (_val); \
+ XMDIO_WRITE((_pdata), _mmd, _reg, mmd_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_mmd' - possible side-effects?
#2050: FILE: drivers/net/axgbe/axgbe_common.h:1813:
+#define XMDIO_WRITE_BITS(_pdata, _mmd, _reg, _mask, _val) \
+do { \
+ u32 mmd_val = XMDIO_READ((_pdata), _mmd, _reg); \
+ mmd_val &= ~_mask; \
+ mmd_val |= (_val); \
+ XMDIO_WRITE((_pdata), _mmd, _reg, mmd_val); \
+} while (0)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_reg' - possible side-effects?
#2050: FILE: drivers/net/axgbe/axgbe_common.h:1813:
+#define XMDIO_WRITE_BITS(_pdata, _mmd, _reg, _mask, _val) \
+do { \
+ u32 mmd_val = XMDIO_READ((_pdata), _mmd, _reg); \
+ mmd_val &= ~_mask; \
+ mmd_val |= (_val); \
+ XMDIO_WRITE((_pdata), _mmd, _reg, mmd_val); \
+} while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument '_mask' may be better as '(_mask)' to avoid precedence issues
#2050: FILE: drivers/net/axgbe/axgbe_common.h:1813:
+#define XMDIO_WRITE_BITS(_pdata, _mmd, _reg, _mask, _val) \
+do { \
+ u32 mmd_val = XMDIO_READ((_pdata), _mmd, _reg); \
+ mmd_val &= ~_mask; \
+ mmd_val |= (_val); \
+ XMDIO_WRITE((_pdata), _mmd, _reg, mmd_val); \
+} while (0)
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#2182: FILE: drivers/net/axgbe/axgbe_dev.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#2184: FILE: drivers/net/axgbe/axgbe_dev.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#2185: FILE: drivers/net/axgbe/axgbe_dev.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#2187: FILE: drivers/net/axgbe/axgbe_dev.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#2188: FILE: drivers/net/axgbe/axgbe_dev.c:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#3003: FILE: drivers/net/axgbe/axgbe_dev.c:893:
+
+ }
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#3026: FILE: drivers/net/axgbe/axgbe_dev.c:916:
+ (unsigned)rxq->queue_id, j);
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#3410: FILE: drivers/net/axgbe/axgbe_ethdev.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#3412: FILE: drivers/net/axgbe/axgbe_ethdev.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#3413: FILE: drivers/net/axgbe/axgbe_ethdev.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#3415: FILE: drivers/net/axgbe/axgbe_ethdev.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#3416: FILE: drivers/net/axgbe/axgbe_ethdev.c:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:LONG_LINE: line over 90 characters
#3587: FILE: drivers/net/axgbe/axgbe_ethdev.c:249:
+ dma_ch_isr = AXGMAC_DMA_IOREAD((struct axgbe_rx_queue *)pdata->rx_queues[0],
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#4300: FILE: drivers/net/axgbe/axgbe_ethdev.h:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#4302: FILE: drivers/net/axgbe/axgbe_ethdev.h:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#4303: FILE: drivers/net/axgbe/axgbe_ethdev.h:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#4305: FILE: drivers/net/axgbe/axgbe_ethdev.h:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#4306: FILE: drivers/net/axgbe/axgbe_ethdev.h:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_x' - possible side-effects?
#4431: FILE: drivers/net/axgbe/axgbe_ethdev.h:203:
+#define AXGMAC_FLOW_CONTROL_VALUE(_x) \
+ (((_x) < 1024) ? 0 : ((_x) / AXGMAC_FLOW_CONTROL_UNIT) - 2)
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_ring' - possible side-effects?
#4474: FILE: drivers/net/axgbe/axgbe_ethdev.h:246:
+#define AXGBE_GET_DESC_DATA(_ring, _idx) \
+ ((_ring)->rdata + \
+ ((_idx) & ((_ring)->rdesc_count - 1)))
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct axgbe_port *' should also have an identifier name
#4712: FILE: drivers/net/axgbe/axgbe_ethdev.h:484:
+ unsigned int (*an_advertising)(struct axgbe_port *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct axgbe_hw_if *' should also have an identifier name
#4954: FILE: drivers/net/axgbe/axgbe_ethdev.h:726:
+void axgbe_init_function_ptrs_dev(struct axgbe_hw_if *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct axgbe_phy_if *' should also have an identifier name
#4955: FILE: drivers/net/axgbe/axgbe_ethdev.h:727:
+void axgbe_init_function_ptrs_phy(struct axgbe_phy_if *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct axgbe_phy_if *' should also have an identifier name
#4956: FILE: drivers/net/axgbe/axgbe_ethdev.h:728:
+void axgbe_init_function_ptrs_phy_v2(struct axgbe_phy_if *);
WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct axgbe_i2c_if *' should also have an identifier name
#4957: FILE: drivers/net/axgbe/axgbe_ethdev.h:729:
+void axgbe_init_function_ptrs_i2c(struct axgbe_i2c_if *);
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5037: FILE: drivers/net/axgbe/axgbe_i2c.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5039: FILE: drivers/net/axgbe/axgbe_i2c.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5040: FILE: drivers/net/axgbe/axgbe_i2c.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5042: FILE: drivers/net/axgbe/axgbe_i2c.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5043: FILE: drivers/net/axgbe/axgbe_i2c.c:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5495: FILE: drivers/net/axgbe/axgbe_logs.h:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5497: FILE: drivers/net/axgbe/axgbe_logs.h:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5498: FILE: drivers/net/axgbe/axgbe_logs.h:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5500: FILE: drivers/net/axgbe/axgbe_logs.h:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5501: FILE: drivers/net/axgbe/axgbe_logs.h:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5645: FILE: drivers/net/axgbe/axgbe_mdio.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5647: FILE: drivers/net/axgbe/axgbe_mdio.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5648: FILE: drivers/net/axgbe/axgbe_mdio.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5650: FILE: drivers/net/axgbe/axgbe_mdio.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#5651: FILE: drivers/net/axgbe/axgbe_mdio.c:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_1000baseKX_Full>
#6163: FILE: drivers/net/axgbe/axgbe_mdio.c:590:
+ if (!(pdata->phy.advertising & ADVERTISED_1000baseKX_Full) &&
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_2500baseX_Full>
#6164: FILE: drivers/net/axgbe/axgbe_mdio.c:591:
+ !(pdata->phy.advertising & ADVERTISED_2500baseX_Full))
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_10000baseKR_Full>
#6172: FILE: drivers/net/axgbe/axgbe_mdio.c:599:
+ if (!(pdata->phy.advertising & ADVERTISED_10000baseKR_Full))
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_10000baseR_FEC>
#6310: FILE: drivers/net/axgbe/axgbe_mdio.c:737:
+ if (advertising & ADVERTISED_10000baseR_FEC)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_Pause>
#6334: FILE: drivers/net/axgbe/axgbe_mdio.c:761:
+ if (advertising & ADVERTISED_Pause)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_Asym_Pause>
#6339: FILE: drivers/net/axgbe/axgbe_mdio.c:766:
+ if (advertising & ADVERTISED_Asym_Pause)
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#6374: FILE: drivers/net/axgbe/axgbe_mdio.c:801:
+ if (pdata->tx_pause != (unsigned)pdata->phy.tx_pause) {
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#6379: FILE: drivers/net/axgbe/axgbe_mdio.c:806:
+ if (pdata->rx_pause != (unsigned)pdata->phy.rx_pause) {
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_10000baseT_Full>
#6678: FILE: drivers/net/axgbe/axgbe_mdio.c:1105:
+ else if (pdata->phy.advertising & ADVERTISED_10000baseT_Full)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_1000baseT_Full>
#6684: FILE: drivers/net/axgbe/axgbe_mdio.c:1111:
+ else if (pdata->phy.advertising & ADVERTISED_1000baseT_Full)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_100baseT_Full>
#6686: FILE: drivers/net/axgbe/axgbe_mdio.c:1113:
+ else if (pdata->phy.advertising & ADVERTISED_100baseT_Full)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_Autoneg>
#6712: FILE: drivers/net/axgbe/axgbe_mdio.c:1139:
+ if (pdata->phy.advertising & ADVERTISED_Autoneg) {
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#6829: FILE: drivers/net/axgbe/axgbe_phy.h:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#6831: FILE: drivers/net/axgbe/axgbe_phy.h:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#6832: FILE: drivers/net/axgbe/axgbe_phy.h:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#6834: FILE: drivers/net/axgbe/axgbe_phy.h:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#6835: FILE: drivers/net/axgbe/axgbe_phy.h:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#6895: FILE: drivers/net/axgbe/axgbe_phy.h:138:
+/* Or MII_ADDR_C45 into regnum for read/write on mii_bus to enable the 21 bit
+ IEEE 802.3ae clause 45 addressing mode used by 10GIGE phy chips. */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#6895: FILE: drivers/net/axgbe/axgbe_phy.h:138:
+ IEEE 802.3ae clause 45 addressing mode used by 10GIGE phy chips. */
CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#6896: FILE: drivers/net/axgbe/axgbe_phy.h:139:
+#define MII_ADDR_C45 (1<<30)
^
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#6946: FILE: drivers/net/axgbe/axgbe_phy.h:189:
+ * Physical Medium Dependent */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#6984: FILE: drivers/net/axgbe/axgbe_phy.h:227:
+ * Lanes B-D are numbered 134-136. */
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_Autoneg>
#7021: FILE: drivers/net/axgbe/axgbe_phy.h:264:
+#define SUPPORTED_Autoneg (1 << 6)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_Pause>
#7028: FILE: drivers/net/axgbe/axgbe_phy.h:271:
+#define SUPPORTED_Pause (1 << 13)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_Asym_Pause>
#7029: FILE: drivers/net/axgbe/axgbe_phy.h:272:
+#define SUPPORTED_Asym_Pause (1 << 14)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_Backplane>
#7031: FILE: drivers/net/axgbe/axgbe_phy.h:274:
+#define SUPPORTED_Backplane (1 << 16)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_10000baseR_FEC>
#7036: FILE: drivers/net/axgbe/axgbe_phy.h:279:
+#define SUPPORTED_10000baseR_FEC (1 << 20)
CHECK:CAMELCASE: Avoid CamelCase: <ADVERTISED_Backplane>
#7048: FILE: drivers/net/axgbe/axgbe_phy.h:291:
+#define ADVERTISED_Backplane (1 << 16)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_1000baseKX_Full>
#7050: FILE: drivers/net/axgbe/axgbe_phy.h:293:
+#define SUPPORTED_1000baseKX_Full (1 << 17)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_10000baseKR_Full>
#7051: FILE: drivers/net/axgbe/axgbe_phy.h:294:
+#define SUPPORTED_10000baseKR_Full (1 << 19)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_2500baseX_Full>
#7052: FILE: drivers/net/axgbe/axgbe_phy.h:295:
+#define SUPPORTED_2500baseX_Full (1 << 15)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_100baseT_Full>
#7053: FILE: drivers/net/axgbe/axgbe_phy.h:296:
+#define SUPPORTED_100baseT_Full (1 << 2)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_1000baseT_Full>
#7054: FILE: drivers/net/axgbe/axgbe_phy.h:297:
+#define SUPPORTED_1000baseT_Full (1 << 5)
CHECK:CAMELCASE: Avoid CamelCase: <SUPPORTED_10000baseT_Full>
#7055: FILE: drivers/net/axgbe/axgbe_phy.h:298:
+#define SUPPORTED_10000baseT_Full (1 << 12)
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#7145: FILE: drivers/net/axgbe/axgbe_phy_impl.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#7147: FILE: drivers/net/axgbe/axgbe_phy_impl.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#7148: FILE: drivers/net/axgbe/axgbe_phy_impl.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#7150: FILE: drivers/net/axgbe/axgbe_phy_impl.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#7151: FILE: drivers/net/axgbe/axgbe_phy_impl.c:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#8371: FILE: drivers/net/axgbe/axgbe_phy_impl.c:1298:
+
+}
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#9346: FILE: drivers/net/axgbe/axgbe_rxtx.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#9348: FILE: drivers/net/axgbe/axgbe_rxtx.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#9349: FILE: drivers/net/axgbe/axgbe_rxtx.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#9351: FILE: drivers/net/axgbe/axgbe_rxtx.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#9352: FILE: drivers/net/axgbe/axgbe_rxtx.c:78:
+ * ^Iderived from this software without specific prior written permission.$
CHECK:SPACING: No space is necessary after a cast
#9495: FILE: drivers/net/axgbe/axgbe_rxtx.c:221:
+ rxq->desc = (volatile union axgbe_rx_desc *) dma->addr;
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#9623: FILE: drivers/net/axgbe/axgbe_rxtx.c:349:
+ "queue_id=%u
", (unsigned)rxq->port_id,
WARNING:MISSING_SPACE: break quoted strings at a space character
#9623: FILE: drivers/net/axgbe/axgbe_rxtx.c:349:
+ PMD_DRV_LOG(ERR, "RX mbuf alloc failed port_id=%u"
+ "queue_id=%u
", (unsigned)rxq->port_id,
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#9624: FILE: drivers/net/axgbe/axgbe_rxtx.c:350:
+ (unsigned)rxq->queue_id);
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#9645: FILE: drivers/net/axgbe/axgbe_rxtx.c:371:
+ * if partial packet error free the buffer */
CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#9670: FILE: drivers/net/axgbe/axgbe_rxtx.c:396:
+ mbuf->pkt_len = mbuf->data_len = pkt_len;
CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#9722: FILE: drivers/net/axgbe/axgbe_rxtx.c:448:
+{
+
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#9942: FILE: drivers/net/axgbe/axgbe_rxtx.c:668:
+ * mbuf is linear */
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10134: FILE: drivers/net/axgbe/axgbe_rxtx.h:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10136: FILE: drivers/net/axgbe/axgbe_rxtx.h:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10137: FILE: drivers/net/axgbe/axgbe_rxtx.h:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10139: FILE: drivers/net/axgbe/axgbe_rxtx.h:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10140: FILE: drivers/net/axgbe/axgbe_rxtx.h:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:TYPO_SPELLING: 'supress' may be misspelled - perhaps 'suppress'?
#10192: FILE: drivers/net/axgbe/axgbe_rxtx.h:130:
+/* to supress gcc warnings related to descriptor casting*/
CHECK:MACRO_ARG_REUSE: Macro argument reuse '_queue' - possible side-effects?
#10214: FILE: drivers/net/axgbe/axgbe_rxtx.h:152:
+#define AXGBE_GET_DESC_PT(_queue, _idx) \
+ (((_queue)->desc) + \
+ ((_idx) & ((_queue)->nb_desc - 1)))
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#10289: FILE: drivers/net/axgbe/axgbe_rxtx.h:227:
+ /* Start freeing TX buffers if there are less free descriptors than
+ this value */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#10289: FILE: drivers/net/axgbe/axgbe_rxtx.h:227:
+ this value */
WARNING:TYPO_SPELLING: 'trasmission' may be misspelled - perhaps 'transmission'?
#10312: FILE: drivers/net/axgbe/axgbe_rxtx.h:250:
+ after trasmission conform*/
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#10312: FILE: drivers/net/axgbe/axgbe_rxtx.h:250:
+ /* Reference to hold Tx mbufs mapped to Tx descriptors freed
+ after trasmission conform*/
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#10312: FILE: drivers/net/axgbe/axgbe_rxtx.h:250:
+ after trasmission conform*/
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10439: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:72:
+ * ^Inotice, this list of conditions and the following disclaimer.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10441: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:74:
+ * ^Inotice, this list of conditions and the following disclaimer in the$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10442: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:75:
+ * ^Idocumentation and/or other materials provided with the distribution.$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10444: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:77:
+ * ^Inames of its contributors may be used to endorse or promote products$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#10445: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:78:
+ * ^Iderived from this software without specific prior written permission.$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#10529: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:162:
+ * all 128 bits are populated */
CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#10554: FILE: drivers/net/axgbe/axgbe_rxtx_vec.c:187:
+ nb_commit = nb_pkts = RTE_MIN(txq->nb_desc_free, nb_pkts);
total: 0 errors, 86 warnings, 77 checks, 10454 lines checked
parent reply other threads:[~2017-09-14 8:57 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <1505324838-25734-2-git-send-email-ravi1.kumar@amd.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=20170914085733.4EC151B19F@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=ravi1.kumar@amd.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).