* |WARNING| pw148402 [PATCH v8 11/17] net/r8169: implement Rx path
[not found] <20241113092854.91445-12-howard_wang@realsil.com.cn>
@ 2024-11-13 9:33 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2024-11-13 9:33 UTC (permalink / raw)
To: test-report; +Cc: Howard Wang
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/148402
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects?
#143: FILE: drivers/net/r8169/r8169_compat.h:575:
+#define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL << (n)) - 1))
CHECK:CAMELCASE: Avoid CamelCase: <_DescStatusBit>
#389: FILE: drivers/net/r8169/r8169_rxtx.c:58:
+enum _DescStatusBit {
CHECK:CAMELCASE: Avoid CamelCase: <DescOwn>
#390: FILE: drivers/net/r8169/r8169_rxtx.c:59:
+ DescOwn = (1 << 31), /* Descriptor is owned by NIC. */
CHECK:CAMELCASE: Avoid CamelCase: <RingEnd>
#391: FILE: drivers/net/r8169/r8169_rxtx.c:60:
+ RingEnd = (1 << 30), /* End of descriptor ring */
CHECK:CAMELCASE: Avoid CamelCase: <FirstFrag>
#392: FILE: drivers/net/r8169/r8169_rxtx.c:61:
+ FirstFrag = (1 << 29), /* First segment of a packet */
CHECK:CAMELCASE: Avoid CamelCase: <LastFrag>
#393: FILE: drivers/net/r8169/r8169_rxtx.c:62:
+ LastFrag = (1 << 28), /* Final segment of a packet */
CHECK:CAMELCASE: Avoid CamelCase: <DescOwn_V3>
#395: FILE: drivers/net/r8169/r8169_rxtx.c:64:
+ DescOwn_V3 = DescOwn, /* Descriptor is owned by NIC. */
CHECK:CAMELCASE: Avoid CamelCase: <RingEnd_V3>
#396: FILE: drivers/net/r8169/r8169_rxtx.c:65:
+ RingEnd_V3 = RingEnd, /* End of descriptor ring */
CHECK:CAMELCASE: Avoid CamelCase: <FirstFrag_V3>
#397: FILE: drivers/net/r8169/r8169_rxtx.c:66:
+ FirstFrag_V3 = (1 << 25), /* First segment of a packet */
CHECK:CAMELCASE: Avoid CamelCase: <LastFrag_V3>
#398: FILE: drivers/net/r8169/r8169_rxtx.c:67:
+ LastFrag_V3 = (1 << 24), /* Final segment of a packet */
CHECK:CAMELCASE: Avoid CamelCase: <LargeSend>
#402: FILE: drivers/net/r8169/r8169_rxtx.c:71:
+ LargeSend = (1 << 27), /* TCP Large Send Offload (TSO) */
CHECK:CAMELCASE: Avoid CamelCase: <GiantSendv4>
#403: FILE: drivers/net/r8169/r8169_rxtx.c:72:
+ GiantSendv4 = (1 << 26), /* TCP Giant Send Offload V4 (GSOv4) */
CHECK:CAMELCASE: Avoid CamelCase: <GiantSendv6>
#404: FILE: drivers/net/r8169/r8169_rxtx.c:73:
+ GiantSendv6 = (1 << 25), /* TCP Giant Send Offload V6 (GSOv6) */
CHECK:CAMELCASE: Avoid CamelCase: <LargeSend_DP>
#405: FILE: drivers/net/r8169/r8169_rxtx.c:74:
+ LargeSend_DP = (1 << 16), /* TCP Large Send Offload (TSO) */
CHECK:CAMELCASE: Avoid CamelCase: <MSSShift>
#406: FILE: drivers/net/r8169/r8169_rxtx.c:75:
+ MSSShift = 16, /* MSS value position */
CHECK:CAMELCASE: Avoid CamelCase: <MSSMask>
#407: FILE: drivers/net/r8169/r8169_rxtx.c:76:
+ MSSMask = 0x7FFU, /* MSS value 11 bits */
CHECK:CAMELCASE: Avoid CamelCase: <TxIPCS>
#408: FILE: drivers/net/r8169/r8169_rxtx.c:77:
+ TxIPCS = (1 << 18), /* Calculate IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxUDPCS>
#409: FILE: drivers/net/r8169/r8169_rxtx.c:78:
+ TxUDPCS = (1 << 17), /* Calculate UDP/IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxTCPCS>
#410: FILE: drivers/net/r8169/r8169_rxtx.c:79:
+ TxTCPCS = (1 << 16), /* Calculate TCP/IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxVlanTag>
#411: FILE: drivers/net/r8169/r8169_rxtx.c:80:
+ TxVlanTag = (1 << 17), /* Add VLAN tag */
CHECK:CAMELCASE: Avoid CamelCase: <TxUDPCS_C>
#414: FILE: drivers/net/r8169/r8169_rxtx.c:83:
+ TxUDPCS_C = (1 << 31), /* Calculate UDP/IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxTCPCS_C>
#415: FILE: drivers/net/r8169/r8169_rxtx.c:84:
+ TxTCPCS_C = (1 << 30), /* Calculate TCP/IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxIPCS_C>
#416: FILE: drivers/net/r8169/r8169_rxtx.c:85:
+ TxIPCS_C = (1 << 29), /* Calculate IP checksum */
CHECK:CAMELCASE: Avoid CamelCase: <TxIPV6F_C>
#417: FILE: drivers/net/r8169/r8169_rxtx.c:86:
+ TxIPV6F_C = (1 << 28), /* Indicate it is an IPv6 packet */
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoUDP>
#425: FILE: drivers/net/r8169/r8169_rxtx.c:94:
+#define RxProtoUDP PID1
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoTCP>
#426: FILE: drivers/net/r8169/r8169_rxtx.c:95:
+#define RxProtoTCP PID0
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoIP>
#427: FILE: drivers/net/r8169/r8169_rxtx.c:96:
+#define RxProtoIP (PID1 | PID0)
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoMask>
#428: FILE: drivers/net/r8169/r8169_rxtx.c:97:
+#define RxProtoMask RxProtoIP
CHECK:CAMELCASE: Avoid CamelCase: <RxIPF>
#430: FILE: drivers/net/r8169/r8169_rxtx.c:99:
+ RxIPF = (1 << 16), /* IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxUDPF>
#431: FILE: drivers/net/r8169/r8169_rxtx.c:100:
+ RxUDPF = (1 << 15), /* UDP/IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxTCPF>
#432: FILE: drivers/net/r8169/r8169_rxtx.c:101:
+ RxTCPF = (1 << 14), /* TCP/IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxVlanTag>
#433: FILE: drivers/net/r8169/r8169_rxtx.c:102:
+ RxVlanTag = (1 << 16), /* VLAN tag available */
CHECK:CAMELCASE: Avoid CamelCase: <RxUDPT>
#436: FILE: drivers/net/r8169/r8169_rxtx.c:105:
+ RxUDPT = (1 << 18),
CHECK:CAMELCASE: Avoid CamelCase: <RxTCPT>
#437: FILE: drivers/net/r8169/r8169_rxtx.c:106:
+ RxTCPT = (1 << 17),
CHECK:CAMELCASE: Avoid CamelCase: <RxV6F>
#441: FILE: drivers/net/r8169/r8169_rxtx.c:110:
+ RxV6F = (1 << 31),
CHECK:CAMELCASE: Avoid CamelCase: <RxV4F>
#442: FILE: drivers/net/r8169/r8169_rxtx.c:111:
+ RxV4F = (1 << 30),
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoUDP_v3>
#448: FILE: drivers/net/r8169/r8169_rxtx.c:117:
+#define RxProtoUDP_v3 PID1_v3
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoTCP_v3>
#449: FILE: drivers/net/r8169/r8169_rxtx.c:118:
+#define RxProtoTCP_v3 PID0_v3
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoIP_v3>
#450: FILE: drivers/net/r8169/r8169_rxtx.c:119:
+#define RxProtoIP_v3 (PID1_v3 | PID0_v3)
CHECK:CAMELCASE: Avoid CamelCase: <RxProtoMask_v3>
#451: FILE: drivers/net/r8169/r8169_rxtx.c:120:
+#define RxProtoMask_v3 RxProtoIP_v3
CHECK:CAMELCASE: Avoid CamelCase: <RxIPF_v3>
#453: FILE: drivers/net/r8169/r8169_rxtx.c:122:
+ RxIPF_v3 = (1 << 26), /* IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxUDPF_v3>
#454: FILE: drivers/net/r8169/r8169_rxtx.c:123:
+ RxUDPF_v3 = (1 << 25), /* UDP/IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxTCPF_v3>
#455: FILE: drivers/net/r8169/r8169_rxtx.c:124:
+ RxTCPF_v3 = (1 << 24), /* TCP/IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxSCTPF_v3>
#456: FILE: drivers/net/r8169/r8169_rxtx.c:125:
+ RxSCTPF_v3 = (1 << 23), /* TCP/IP checksum failed */
CHECK:CAMELCASE: Avoid CamelCase: <RxVlanTag_v3>
#457: FILE: drivers/net/r8169/r8169_rxtx.c:126:
+ RxVlanTag_v3 = (RxVlanTag), /* VLAN tag available */
CHECK:CAMELCASE: Avoid CamelCase: <RxUDPT_v3>
#460: FILE: drivers/net/r8169/r8169_rxtx.c:129:
+ RxUDPT_v3 = (1 << 29),
CHECK:CAMELCASE: Avoid CamelCase: <RxTCPT_v3>
#461: FILE: drivers/net/r8169/r8169_rxtx.c:130:
+ RxTCPT_v3 = (1 << 28),
CHECK:CAMELCASE: Avoid CamelCase: <RxSCTP_v3>
#462: FILE: drivers/net/r8169/r8169_rxtx.c:131:
+ RxSCTP_v3 = (1 << 27),
CHECK:CAMELCASE: Avoid CamelCase: <RxV6F_v3>
#466: FILE: drivers/net/r8169/r8169_rxtx.c:135:
+ RxV6F_v3 = RxV6F,
CHECK:CAMELCASE: Avoid CamelCase: <RxV4F_v3>
#467: FILE: drivers/net/r8169/r8169_rxtx.c:136:
+ RxV4F_v3 = RxV4F,
CHECK:CAMELCASE: Avoid CamelCase: <RxConfig>
#677: FILE: drivers/net/r8169/r8169_rxtx.c:342:
+ rx_config = RTL_R32(hw, RxConfig);
CHECK:CAMELCASE: Avoid CamelCase: <EnableInnerVlan>
#679: FILE: drivers/net/r8169/r8169_rxtx.c:344:
+ rx_config |= (EnableInnerVlan | EnableOuterVlan);
CHECK:CAMELCASE: Avoid CamelCase: <EnableOuterVlan>
#679: FILE: drivers/net/r8169/r8169_rxtx.c:344:
+ rx_config |= (EnableInnerVlan | EnableOuterVlan);
CHECK:CAMELCASE: Avoid CamelCase: <CPlusCmd>
#685: FILE: drivers/net/r8169/r8169_rxtx.c:350:
+ cp_cmd = RTL_R16(hw, CPlusCmd);
CHECK:CAMELCASE: Avoid CamelCase: <RxChkSum>
#688: FILE: drivers/net/r8169/r8169_rxtx.c:353:
+ cp_cmd |= RxChkSum;
CHECK:CAMELCASE: Avoid CamelCase: <AcceptBroadcast>
#704: FILE: drivers/net/r8169/r8169_rxtx.c:369:
+ rx_mode = AcceptBroadcast | AcceptMyPhys;
CHECK:CAMELCASE: Avoid CamelCase: <AcceptMyPhys>
#704: FILE: drivers/net/r8169/r8169_rxtx.c:369:
+ rx_mode = AcceptBroadcast | AcceptMyPhys;
CHECK:CAMELCASE: Avoid CamelCase: <RxDescAddrLow>
#724: FILE: drivers/net/r8169/r8169_rxtx.c:389:
+ RTL_W32(hw, RxDescAddrLow, ((u64)rxq->hw_ring_phys_addr & DMA_BIT_MASK(32)));
CHECK:CAMELCASE: Avoid CamelCase: <RxDescAddrHigh>
#725: FILE: drivers/net/r8169/r8169_rxtx.c:390:
+ RTL_W32(hw, RxDescAddrHigh, ((u64)rxq->hw_ring_phys_addr >> 32));
CHECK:CAMELCASE: Avoid CamelCase: <RxMaxSize>
#740: FILE: drivers/net/r8169/r8169_rxtx.c:405:
+ RTL_W16(hw, RxMaxSize, max_rx_pkt_size);
CHECK:CAMELCASE: Avoid CamelCase: <ChipCmd>
#760: FILE: drivers/net/r8169/r8169_rxtx.c:425:
+ RTL_W8(hw, ChipCmd, RTL_R8(hw, ChipCmd) | CmdRxEnb);
CHECK:CAMELCASE: Avoid CamelCase: <CmdRxEnb>
#760: FILE: drivers/net/r8169/r8169_rxtx.c:425:
+ RTL_W8(hw, ChipCmd, RTL_R8(hw, ChipCmd) | CmdRxEnb);
CHECK:CAMELCASE: Avoid CamelCase: <RxRES>
#838: FILE: drivers/net/r8169/r8169_rxtx.c:503:
+ if (unlikely(opts1 & RxRES)) {
CHECK:CAMELCASE: Avoid CamelCase: <RxOK>
#919: FILE: drivers/net/r8169/r8169_rxtx.c:584:
+ RTL_W32(hw, ISR0_8125, (RxOK | RxErr | RxDescUnavail));
CHECK:CAMELCASE: Avoid CamelCase: <RxErr>
#919: FILE: drivers/net/r8169/r8169_rxtx.c:584:
+ RTL_W32(hw, ISR0_8125, (RxOK | RxErr | RxDescUnavail));
CHECK:CAMELCASE: Avoid CamelCase: <RxDescUnavail>
#919: FILE: drivers/net/r8169/r8169_rxtx.c:584:
+ RTL_W32(hw, ISR0_8125, (RxOK | RxErr | RxDescUnavail));
total: 0 errors, 0 warnings, 66 checks, 1038 lines checked
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2024-11-13 9:33 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20241113092854.91445-12-howard_wang@realsil.com.cn>
2024-11-13 9:33 ` |WARNING| pw148402 [PATCH v8 11/17] net/r8169: implement Rx path checkpatch
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).