linux-kernel - [PATCH] drm/nouveau: add missing spaces after ','

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <79709251a62a1d58947d96a363e82d58@208suo.com>
Date:   Fri, 14 Jul 2023 10:08:45 +0800
From:   sunran001@...suo.com
To:     airlied@...il.com, daniel@...ll.ch
Cc:     dri-devel@...ts.freedesktop.org, nouveau@...ts.freedesktop.org,
        linux-kernel@...r.kernel.org
Subject: [PATCH] drm/nouveau: add missing spaces after ','

Add missing spaces to clear checkpatch errors.

ERROR: space required after that ',' (ctx:VxV)

Signed-off-by: Ran Sun <sunran001@...suo.com>
---
  drivers/gpu/drm/nouveau/include/nvhw/drf.h | 80 +++++++++++-----------
  1 file changed, 40 insertions(+), 40 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/include/nvhw/drf.h 
b/drivers/gpu/drm/nouveau/include/nvhw/drf.h
index d6969c0e2f29..a14f8d5663bb 100644
--- a/drivers/gpu/drm/nouveau/include/nvhw/drf.h
+++ b/drivers/gpu/drm/nouveau/include/nvhw/drf.h
@@ -33,58 +33,58 @@
  #define DRF_MX_MW(drf)      drf
  #define DRF_MX(drf)         DRF_MX_##drf
  #define DRF_MW(drf)         DRF_MX(drf)
-#define DRF_MW_SPANS(o,drf) (DRF_LW_IDX((o),drf) != 
DRF_HW_IDX((o),drf))
+#define DRF_MW_SPANS(o, drf) (DRF_LW_IDX((o), drf) != DRF_HW_IDX((o), 
drf))
  #define DRF_MW_SIZE(o)      (sizeof((o)[0]) * 8)

-#define DRF_LW_IDX(o,drf)   (DRF_LO(DRF_MW(drf)) / DRF_MW_SIZE(o))
-#define DRF_LW_LO(o,drf)    (DRF_LO(DRF_MW(drf)) % DRF_MW_SIZE(o))
-#define DRF_LW_HI(o,drf)    (DRF_MW_SPANS((o),drf) ? (DRF_MW_SIZE(o) - 
1) : DRF_HW_HI((o),drf))
-#define DRF_LW_BITS(o,drf)  (DRF_LW_HI((o),drf) - DRF_LW_LO((o),drf) + 
1)
-#define DRF_LW_MASK(o,drf)  (~0ULL >> (64 - DRF_LW_BITS((o),drf)))
-#define DRF_LW_SMASK(o,drf) (DRF_LW_MASK((o),drf) << 
DRF_LW_LO((o),drf))
-#define DRF_LW_GET(o,drf)   (((o)[DRF_LW_IDX((o),drf)] >> 
DRF_LW_LO((o),drf)) & DRF_LW_MASK((o),drf))
-#define DRF_LW_VAL(o,drf,v) (((v) & DRF_LW_MASK((o),drf)) << 
DRF_LW_LO((o),drf))
-#define DRF_LW_CLR(o,drf)   ((o)[DRF_LW_IDX((o),drf)] & 
~DRF_LW_SMASK((o),drf))
-#define DRF_LW_SET(o,drf,v) (DRF_LW_CLR((o),drf) | 
DRF_LW_VAL((o),drf,(v)))
-
-#define DRF_HW_IDX(o,drf)   (DRF_HI(DRF_MW(drf)) / DRF_MW_SIZE(o))
-#define DRF_HW_LO(o,drf)    0
-#define DRF_HW_HI(o,drf)    (DRF_HI(DRF_MW(drf)) % DRF_MW_SIZE(o))
-#define DRF_HW_BITS(o,drf)  (DRF_HW_HI((o),drf) - DRF_HW_LO((o),drf) + 
1)
-#define DRF_HW_MASK(o,drf)  (~0ULL >> (64 - DRF_HW_BITS((o),drf)))
-#define DRF_HW_SMASK(o,drf) (DRF_HW_MASK((o),drf) << 
DRF_HW_LO((o),drf))
-#define DRF_HW_GET(o,drf)   ((o)[DRF_HW_IDX(o,drf)] & 
DRF_HW_SMASK((o),drf))
-#define DRF_HW_VAL(o,drf,v) (((long long)(v) >> DRF_LW_BITS((o),drf)) & 
DRF_HW_SMASK((o),drf))
-#define DRF_HW_CLR(o,drf)   ((o)[DRF_HW_IDX((o),drf)] & 
~DRF_HW_SMASK((o),drf))
-#define DRF_HW_SET(o,drf,v) (DRF_HW_CLR((o),drf) | 
DRF_HW_VAL((o),drf,(v)))
+#define DRF_LW_IDX(o, drf)   (DRF_LO(DRF_MW(drf)) / DRF_MW_SIZE(o))
+#define DRF_LW_LO(o, drf)    (DRF_LO(DRF_MW(drf)) % DRF_MW_SIZE(o))
+#define DRF_LW_HI(o, drf)    (DRF_MW_SPANS((o), drf) ? (DRF_MW_SIZE(o) 
- 1) : DRF_HW_HI((o), drf))
+#define DRF_LW_BITS(o, drf)  (DRF_LW_HI((o), drf) - DRF_LW_LO((o), drf) 
+ 1)
+#define DRF_LW_MASK(o, drf)  (~0ULL >> (64 - DRF_LW_BITS((o), drf)))
+#define DRF_LW_SMASK(o, drf) (DRF_LW_MASK((o), drf) << DRF_LW_LO((o), 
drf))
+#define DRF_LW_GET(o, drf)   (((o)[DRF_LW_IDX((o), drf)] >> 
DRF_LW_LO((o), drf)) & DRF_LW_MASK((o), drf))
+#define DRF_LW_VAL(o, drf, v) (((v) & DRF_LW_MASK((o), drf)) << 
DRF_LW_LO((o), drf))
+#define DRF_LW_CLR(o, drf)   ((o)[DRF_LW_IDX((o), drf)] & 
~DRF_LW_SMASK((o), drf))
+#define DRF_LW_SET(o, drf, v) (DRF_LW_CLR((o), drf) | DRF_LW_VAL((o), 
drf, (v)))
+
+#define DRF_HW_IDX(o, drf)   (DRF_HI(DRF_MW(drf)) / DRF_MW_SIZE(o))
+#define DRF_HW_LO(o, drf)    0
+#define DRF_HW_HI(o, drf)    (DRF_HI(DRF_MW(drf)) % DRF_MW_SIZE(o))
+#define DRF_HW_BITS(o, drf)  (DRF_HW_HI((o), drf) - DRF_HW_LO((o), drf) 
+ 1)
+#define DRF_HW_MASK(o, drf)  (~0ULL >> (64 - DRF_HW_BITS((o), drf)))
+#define DRF_HW_SMASK(o, drf) (DRF_HW_MASK((o), drf) << DRF_HW_LO((o), 
drf))
+#define DRF_HW_GET(o, drf)   ((o)[DRF_HW_IDX(o, drf)] & 
DRF_HW_SMASK((o), drf))
+#define DRF_HW_VAL(o, drf, v) (((long long)(v) >> DRF_LW_BITS((o), 
drf)) & DRF_HW_SMASK((o), drf))
+#define DRF_HW_CLR(o, drf)   ((o)[DRF_HW_IDX((o), drf)] & 
~DRF_HW_SMASK((o), drf))
+#define DRF_HW_SET(o, drf, v) (DRF_HW_CLR((o), drf) | DRF_HW_VAL((o), 
drf, (v)))

  /* DRF accessors. */
-#define NVVAL_X(drf,v) (((v) & DRF_MASK(drf)) << DRF_LO(drf))
-#define NVVAL_N(X,d,r,f,  v) NVVAL_X(d##_##r##_##f, (v))
-#define NVVAL_I(X,d,r,f,i,v) NVVAL_X(d##_##r##_##f(i), (v))
-#define NVVAL_(X,_1,_2,_3,_4,_5,IMPL,...) IMPL
+#define NVVAL_X(drf, v) (((v) & DRF_MASK(drf)) << DRF_LO(drf))
+#define NVVAL_N(X, d, r, f,  v) NVVAL_X(d##_##r##_##f, (v))
+#define NVVAL_I(X, d, r, f, i, v) NVVAL_X(d##_##r##_##f(i), (v))
+#define NVVAL_(X, _1, _2, _3, _4, _5, IMPL, ...) IMPL
  #define NVVAL(A...) NVVAL_(X, ##A, NVVAL_I, NVVAL_N)(X, ##A)

-#define NVDEF_N(X,d,r,f,  v) NVVAL_X(d##_##r##_##f, 
d##_##r##_##f##_##v)
-#define NVDEF_I(X,d,r,f,i,v) NVVAL_X(d##_##r##_##f(i), 
d##_##r##_##f##_##v)
-#define NVDEF_(X,_1,_2,_3,_4,_5,IMPL,...) IMPL
+#define NVDEF_N(X, d, r, f, v) NVVAL_X(d##_##r##_##f, 
d##_##r##_##f##_##v)
+#define NVDEF_I(X, d, r, f, i, v) NVVAL_X(d##_##r##_##f(i), 
d##_##r##_##f##_##v)
+#define NVDEF_(X, _1, _2, _3, _4, _5, IMPL, ...) IMPL
  #define NVDEF(A...) NVDEF_(X, ##A, NVDEF_I, NVDEF_N)(X, ##A)

-#define NVVAL_GET_X(o,drf) (((o) >> DRF_LO(drf)) & DRF_MASK(drf))
-#define NVVAL_GET_N(X,o,d,r,f  ) NVVAL_GET_X(o, d##_##r##_##f)
-#define NVVAL_GET_I(X,o,d,r,f,i) NVVAL_GET_X(o, d##_##r##_##f(i))
-#define NVVAL_GET_(X,_1,_2,_3,_4,_5,IMPL,...) IMPL
+#define NVVAL_GET_X(o, drf) (((o) >> DRF_LO(drf)) & DRF_MASK(drf))
+#define NVVAL_GET_N(X, o, d, r, f) NVVAL_GET_X(o, d##_##r##_##f)
+#define NVVAL_GET_I(X, o, d, r, f, i) NVVAL_GET_X(o, d##_##r##_##f(i))
+#define NVVAL_GET_(X, _1, _2, _3, _4, _5, IMPL, ...) IMPL
  #define NVVAL_GET(A...) NVVAL_GET_(X, ##A, NVVAL_GET_I, NVVAL_GET_N)(X, 
##A)

-#define NVVAL_TEST_X(o,drf,cmp,drfv) (NVVAL_GET_X((o), drf) cmp drfv)
-#define NVVAL_TEST_N(X,o,d,r,f,  cmp,v) NVVAL_TEST_X(o, d##_##r##_##f   
, cmp, (v))
-#define NVVAL_TEST_I(X,o,d,r,f,i,cmp,v) NVVAL_TEST_X(o, 
d##_##r##_##f(i), cmp, (v))
-#define NVVAL_TEST_(X,_1,_2,_3,_4,_5,_6,_7,IMPL,...) IMPL
+#define NVVAL_TEST_X(o, drf, cmp, drfv) (NVVAL_GET_X((o), drf) cmp 
drfv)
+#define NVVAL_TEST_N(X, o, d, r, f, cmp, v) NVVAL_TEST_X(o, 
d##_##r##_##f   , cmp, (v))
+#define NVVAL_TEST_I(X, o, d, r, f, i, cmp, v) NVVAL_TEST_X(o, 
d##_##r##_##f(i), cmp, (v))
+#define NVVAL_TEST_(X, _1, _2, _3, _4, _5, _6, _7, IMPL, ...) IMPL
  #define NVVAL_TEST(A...) NVVAL_TEST_(X, ##A, NVVAL_TEST_I, 
NVVAL_TEST_N)(X, ##A)

-#define NVDEF_TEST_N(X,o,d,r,f,  cmp,v) NVVAL_TEST_X(o, d##_##r##_##f   
, cmp, d##_##r##_##f##_##v)
-#define NVDEF_TEST_I(X,o,d,r,f,i,cmp,v) NVVAL_TEST_X(o, 
d##_##r##_##f(i), cmp, d##_##r##_##f##_##v)
-#define NVDEF_TEST_(X,_1,_2,_3,_4,_5,_6,_7,IMPL,...) IMPL
+#define NVDEF_TEST_N(X, o, d, r, f, cmp, v) NVVAL_TEST_X(o, 
d##_##r##_##f   , cmp, d##_##r##_##f##_##v)
+#define NVDEF_TEST_I(X, o, d, r, f, i, cmp, v) NVVAL_TEST_X(o, 
d##_##r##_##f(i), cmp, d##_##r##_##f##_##v)
+#define NVDEF_TEST_(X, _1, _2, _3, _4, _5, _6, _7, IMPL, ...) IMPL
  #define NVDEF_TEST(A...) NVDEF_TEST_(X, ##A, NVDEF_TEST_I, 
NVDEF_TEST_N)(X, ##A)

  #define NVVAL_SET_X(o,drf,v) (((o) & ~DRF_SMASK(drf)) | NVVAL_X(drf, 
(v)))