mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Simranjit Singh <simranjit.singh@corebaby.org>
To: ibm-acpi@hmh.eng.br
Cc: dvhart@infradead.org, andy@infradead.org,
	ibm-acpi-devel@lists.sourceforge.net,
	platform-driver-x86@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 1/1] Use checkpatch.pl to make thinkpad_acpi.c error free: add parenthesis to complex macros
Date: Tue, 14 Nov 2017 15:14:41 -0800	[thread overview]
Message-ID: <1510701281.9469.20.camel@k3saltt0> (raw)

From: Simranjit Singh <simranjit.singh@corebaby.org> 

    Added parentheses around complex macros following checkpatch.pl for thinkpad_acpi.c

Signed-off-by: Simranjit Singh
---
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
index 9ad1f5b..e2616c2 100644
--- a/drivers/platform/x86/thinkpad_acpi.c
+++ b/drivers/platform/x86/thinkpad_acpi.c
@@ -1650,23 +1650,23 @@ static void tpacpi_remove_driver_attributes(struct device_driver *drv)
 	TPV_Q(PCI_VENDOR_ID_IBM, __id1, __id2, __bv1, __bv2)
 
 /* Outdated IBM BIOSes often lack the EC id string */
-#define TPV_QI1(__id1, __id2, __bv1, __bv2, __ev1, __ev2) \
+#define (TPV_QI1(__id1, __id2, __bv1, __bv2, __ev1, __ev2) \
 	TPV_Q_X(PCI_VENDOR_ID_IBM, __id1, __id2, 	\
 		__bv1, __bv2, TPID(__id1, __id2),	\
 		__ev1, __ev2),				\
 	TPV_Q_X(PCI_VENDOR_ID_IBM, __id1, __id2, 	\
 		__bv1, __bv2, TPACPI_MATCH_UNKNOWN,	\
-		__ev1, __ev2)
+		__ev1, __ev2))
 
 /* Outdated IBM BIOSes often lack the EC id string */
-#define TPV_QI2(__bid1, __bid2, __bv1, __bv2,		\
+#define (TPV_QI2(__bid1, __bid2, __bv1, __bv2,		\
 		__eid1, __eid2, __ev1, __ev2) 		\
 	TPV_Q_X(PCI_VENDOR_ID_IBM, __bid1, __bid2, 	\
 		__bv1, __bv2, TPID(__eid1, __eid2),	\
 		__ev1, __ev2),				\
 	TPV_Q_X(PCI_VENDOR_ID_IBM, __bid1, __bid2, 	\
 		__bv1, __bv2, TPACPI_MATCH_UNKNOWN,	\
-		__ev1, __ev2)
+		__ev1, __ev2))
 
 #define TPV_QL0(__id1, __id2, __bv1, __bv2) \
 	TPV_Q(PCI_VENDOR_ID_LENOVO, __id1, __id2, __bv1, __bv2)
@@ -6238,8 +6238,8 @@ static ssize_t thermal_temp_input_show(struct device *dev,
 	THERMAL_SENSOR_ATTR_TEMP(16, 15),
 };
 
-#define THERMAL_ATTRS(X) \
-	&sensor_dev_attr_thermal_temp_input[X].dev_attr.attr
+#define (THERMAL_ATTRS(X) \
+	&sensor_dev_attr_thermal_temp_input[X].dev_attr.attr)
 
 static struct attribute *thermal_temp_input_attr[] = {
 	THERMAL_ATTRS(8),

             reply	other threads:[~2017-11-14 23:14 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-14 23:14 Simranjit Singh [this message]
2017-11-15 11:04 ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1510701281.9469.20.camel@k3saltt0 \
    --to=simranjit.singh@corebaby.org \
    --cc=andy@infradead.org \
    --cc=dvhart@infradead.org \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=ibm-acpi@hmh.eng.br \
    --cc=linux-kernel@vger.kernel.org \
    --cc=platform-driver-x86@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®