power: supply: bq27xxx: adjust whitespace and use BIT() for bitflags
authorKrzysztof Kozlowski <krzk@kernel.org>
Sat, 19 Sep 2020 14:04:15 +0000 (16:04 +0200)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Thu, 1 Oct 2020 22:01:42 +0000 (00:01 +0200)
BIT() is a preferred way to toggle bit-like flags: no problems with 32/64
bit systems, less chances for mistakes.  Remove also unneeded
whitespace.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/bq27xxx_battery.c

index 65806f668b1f1a1115a5b7e6fe819fd2123599a4..e971af43dd4546f90cee6eac56a3c8c1e937ed0f 100644 (file)
@@ -76,7 +76,7 @@
 
 /* BQ27Z561 has different layout for Flags register */
 #define BQ27Z561_FLAG_FDC      BIT(4) /* Battery fully discharged */
-#define BQ27Z561_FLAG_FC               BIT(5) /* Battery fully charged */
+#define BQ27Z561_FLAG_FC       BIT(5) /* Battery fully charged */
 #define BQ27Z561_FLAG_DIS_CH   BIT(6) /* Battery is discharging */
 
 /* control register params */
@@ -847,13 +847,14 @@ static struct bq27xxx_dm_reg bq27621_dm_regs[] = {
 
 #define bq27z561_dm_regs 0
 #define bq28z610_dm_regs 0
-
-#define BQ27XXX_O_ZERO 0x00000001
-#define BQ27XXX_O_OTDC 0x00000002 /* has OTC/OTD overtemperature flags */
-#define BQ27XXX_O_UTOT  0x00000004 /* has OT overtemperature flag */
-#define BQ27XXX_O_CFGUP        0x00000008
-#define BQ27XXX_O_RAM  0x00000010
-#define BQ27Z561_O_BITS        0x00000020
+#define bq34z100_dm_regs 0
+
+#define BQ27XXX_O_ZERO         BIT(0)
+#define BQ27XXX_O_OTDC         BIT(1) /* has OTC/OTD overtemperature flags */
+#define BQ27XXX_O_UTOT         BIT(2) /* has OT overtemperature flag */
+#define BQ27XXX_O_CFGUP                BIT(3)
+#define BQ27XXX_O_RAM          BIT(4)
+#define BQ27Z561_O_BITS                BIT(5)
 
 #define BQ27XXX_DATA(ref, key, opt) {          \
        .opts = (opt),                          \