Re: [PATCH 1/8] staging: lustre: Enclose complex macros in parantheses

From: Christoph Volkert
Date: Thu Dec 21 2017 - 10:50:17 EST


This is a duplicate patch. We fixed a typo and this was still left as a
temporary file. (emacsâ) Sorry. The patch with Message-Id
<1513869932-27407-3-git-send-email-fabian_huegel@xxxxxx> is the right one.

We can provide a new patch series without the fix, if needed.

Thanks
Christoph

Checkpatch wants complex macros to be enclosed in parantheses, so we
put parantheses around these four macros.

Signed-off-by: Fabian Huegel <fabian_huegel@xxxxxx>
Signed-off-by: Christoph Volkert <linux@xxxxxxxxxxxxxxxxxxxx>
---
drivers/staging/lustre/lustre/include/obd_class.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/lustre/lustre/include/obd_class.h
b/drivers/staging/lustre/lustre/include/obd_class.h
index 67c535c..5c8cf30 100644
--- a/drivers/staging/lustre/lustre/include/obd_class.h
+++ b/drivers/staging/lustre/lustre/include/obd_class.h
@@ -294,10 +294,10 @@ struct obdo;

void obdo_to_ioobj(const struct obdo *oa, struct obd_ioobj *ioobj);

-#define OBT(dev) (dev)->obd_type
-#define OBP(dev, op) (dev)->obd_type->typ_dt_ops->op
-#define MDP(dev, op) (dev)->obd_type->typ_md_ops->op
-#define CTXTP(ctxt, op) (ctxt)->loc_logops->lop_##op
+#define OBT(dev) ((dev)->obd_type)
+#define OBP(dev, op) ((dev)->obd_type->typ_dt_ops->op)
+#define MDP(dev, op) ((dev)->obd_type->typ_md_ops->op)
+#define CTXTP(ctxt, op) ((ctxt)->loc_logops->lop_##op)

/* Ensure obd_setup: used for cleanup which must be called
* while obd is stopping