diff mbox series

3c59x: Add space around '='

Message ID 304ca645a55aa0affe830bd36edaf24d@208suo.com (mailing list archive)
State Rejected
Delegated to: Netdev Maintainers
Headers show
Series 3c59x: Add space around '=' | expand

Checks

Context Check Description
netdev/tree_selection success Guessing tree name failed - patch did not apply

Commit Message

hanyu001@208suo.com July 17, 2023, 3:27 a.m. UTC
Fix checkpatch warnings:

./drivers/net/ethernet/3com/3c59x.c:716: ERROR: spaces required around 
that '=' (ctx:VxV)
./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
that '=' (ctx:VxV)
./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
that '=' (ctx:VxV)
./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
that '=' (ctx:VxV)
./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
that '=' (ctx:VxV)

Signed-off-by: maqimei <2433033762@qq.com>
---
  drivers/net/ethernet/3com/3c59x.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

XCVR_100baseTx,
+    XCVR_100baseFx, XCVR_MII = 6, XCVR_NWAY = 8, XCVR_ExtMII = 9, 
XCVR_Default = 10,
  };

  static const struct media_table {

Comments

Jakub Kicinski July 18, 2023, 5:42 p.m. UTC | #1
On Mon, 17 Jul 2023 11:27:10 +0800 hanyu001@208suo.com wrote:
> Fix checkpatch warnings:
> 
> ./drivers/net/ethernet/3com/3c59x.c:716: ERROR: spaces required around 
> that '=' (ctx:VxV)
> ./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
> that '=' (ctx:VxV)
> ./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
> that '=' (ctx:VxV)
> ./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
> that '=' (ctx:VxV)
> ./drivers/net/ethernet/3com/3c59x.c:717: ERROR: spaces required around 
> that '=' (ctx:VxV)

Networking does not accept checkpatch-based coding style fixes as others
already pointed out. Plus your patches are broken (white space damaged),
and you sent 10 of them before waiting for any feedback, please stop.
diff mbox series

Patch

diff --git a/drivers/net/ethernet/3com/3c59x.c 
b/drivers/net/ethernet/3com/3c59x.c
index 082388b..9aa3244 100644
--- a/drivers/net/ethernet/3com/3c59x.c
+++ b/drivers/net/ethernet/3com/3c59x.c
@@ -713,8 +713,8 @@  static void window_set(struct vortex_private *vp, 
int window)
     Note that we deviate from the 3Com order by checking 10base2 before 
AUI.
   */
  enum xcvr_types {
-    XCVR_10baseT=0, XCVR_AUI, XCVR_10baseTOnly, XCVR_10base2, 
XCVR_100baseTx,
-    XCVR_100baseFx, XCVR_MII=6, XCVR_NWAY=8, XCVR_ExtMII=9, 
XCVR_Default=10,
+    XCVR_10baseT = 0, XCVR_AUI, XCVR_10baseTOnly, XCVR_10base2,