nouveau.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [Nouveau] [PATCH] drm/nouveau: add missing spaces after ','
       [not found] <20230711064830.80096-1-xujianghui@cdjrlc.com>
@ 2023-07-11  6:50 ` sunran001
  0 siblings, 0 replies; 2+ messages in thread
From: sunran001 @ 2023-07-11  6:50 UTC (permalink / raw)
  To: airlied, daniel; +Cc: nouveau, linux-kernel, dri-devel

Add missing spaces to clear checkpatch errors.

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

Signed-off-by: Ran Sun <sunran001@208suo.com>
---
  drivers/gpu/drm/nouveau/nouveau_svm.c | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nouveau_svm.c 
b/drivers/gpu/drm/nouveau/nouveau_svm.c
index a74ba8d84ba7..78ea3b541a5b 100644
--- a/drivers/gpu/drm/nouveau/nouveau_svm.c
+++ b/drivers/gpu/drm/nouveau/nouveau_svm.c
@@ -75,8 +75,8 @@ struct nouveau_svm {
  #define FAULT_ACCESS_ATOMIC 2
  #define FAULT_ACCESS_PREFETCH 3

-#define SVM_DBG(s,f,a...) NV_DEBUG((s)->drm, "svm: "f"\n", ##a)
-#define SVM_ERR(s,f,a...) NV_WARN((s)->drm, "svm: "f"\n", ##a)
+#define SVM_DBG(s, f, a...) NV_DEBUG((s)->drm, "svm: "f"\n", ##a)
+#define SVM_ERR(s, f, a...) NV_WARN((s)->drm, "svm: "f"\n", ##a)

  struct nouveau_pfnmap_args {
      struct nvif_ioctl_v0 i;
@@ -101,9 +101,9 @@ nouveau_ivmm_find(struct nouveau_svm *svm, u64 inst)
      return NULL;
  }

-#define SVMM_DBG(s,f,a...)                                              
        \
+#define SVMM_DBG(s, f, a...)                                            
          \
      NV_DEBUG((s)->vmm->cli->drm, "svm-%p: "f"\n", (s), ##a)
-#define SVMM_ERR(s,f,a...)                                              
        \
+#define SVMM_ERR(s, f, a...)                                            
          \
      NV_WARN((s)->vmm->cli->drm, "svm-%p: "f"\n", (s), ##a)

  int

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Nouveau] [PATCH] drm/nouveau: add missing spaces after ','
       [not found] <20230714012736.13031-1-xujianghui@cdjrlc.com>
@ 2023-07-14  2:08 ` sunran001
  0 siblings, 0 replies; 2+ messages in thread
From: sunran001 @ 2023-07-14  2:08 UTC (permalink / raw)
  To: airlied, daniel; +Cc: nouveau, linux-kernel, dri-devel

Add missing spaces to clear checkpatch errors.

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

Signed-off-by: Ran Sun <sunran001@208suo.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)))

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-07-14  2:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20230711064830.80096-1-xujianghui@cdjrlc.com>
2023-07-11  6:50 ` [Nouveau] [PATCH] drm/nouveau: add missing spaces after ',' sunran001
     [not found] <20230714012736.13031-1-xujianghui@cdjrlc.com>
2023-07-14  2:08 ` sunran001

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).