On Tuesday 25 August 2015 01:21 PM, Lee Jones wrote:

On Tue, 25 Aug 2015, Vaibhav Hiremath wrote:

quoted

Driver had coding style issues where spaces were used instead
of tabs. This patch fixes them all.

That's not all it's doing though is it?

Yes, its just tabbing related fixes.

Please describe all of your changes.

quoted

Signed-off-by: Vaibhav Hiremath <redacted>
---
  drivers/mfd/88pm800.c | 125 ++++++++++++++++++++++++--------------------------
  1 file changed, 61 insertions(+), 64 deletions(-)

diff --git a/drivers/mfd/88pm800.c b/drivers/mfd/88pm800.c
index f104a32..c4e097d 100644
--- a/drivers/mfd/88pm800.c
+++ b/drivers/mfd/88pm800.c

@@ -30,55 +30,55 @@
  #include <linux/of_device.h>

  /* Interrupt Registers */
-#define PM800_INT_STATUS1		(0x05)
+#define PM800_INT_STATUS1		0x05
  #define PM800_ONKEY_INT_STS1		(1 << 0)
  #define PM800_EXTON_INT_STS1		(1 << 1)
-#define PM800_CHG_INT_STS1			(1 << 2)
-#define PM800_BAT_INT_STS1			(1 << 3)
-#define PM800_RTC_INT_STS1			(1 << 4)
+#define PM800_CHG_INT_STS1		(1 << 2)
+#define PM800_BAT_INT_STS1		(1 << 3)
+#define PM800_RTC_INT_STS1		(1 << 4)
  #define PM800_CLASSD_OC_INT_STS1	(1 << 5)

These should all be replaced by BIT().

Please fix the whole file.

Will do.

Thanks,
Vaibhav