power: supply: max17042_battery: Do not use CONFIG_ prefix in regular C code
authorPrzemyslaw Chwiala <przemekchwiala@gmail.com>
Thu, 20 Jul 2023 12:31:02 +0000 (14:31 +0200)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Tue, 12 Sep 2023 18:54:46 +0000 (20:54 +0200)
Using CONFIG_ prefix for macros is not a good practice.
Use CONFIG_ prefix in Kconfig only.

Signed-off-by: Przemyslaw Chwiala <przemekchwiala@gmail.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Link: https://lore.kernel.org/r/20230720123102.154699-1-przemekchwiala@gmail.com
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/max17042_battery.c

index 17ac2ab78c4e4c3782298749840a478e6c769f13..e7d37e422c3f6ef6c89d472b3a74a8cb9ed57996 100644 (file)
@@ -36,7 +36,7 @@
 #define STATUS_BR_BIT          (1 << 15)
 
 /* Interrupt mask bits */
-#define CONFIG_ALRT_BIT_ENBL   (1 << 2)
+#define CFG_ALRT_BIT_ENBL      (1 << 2)
 
 #define VFSOC0_LOCK            0x0000
 #define VFSOC0_UNLOCK          0x0080
@@ -1116,8 +1116,8 @@ static int max17042_probe(struct i2c_client *client)
                                                chip);
                if (!ret) {
                        regmap_update_bits(chip->regmap, MAX17042_CONFIG,
-                                       CONFIG_ALRT_BIT_ENBL,
-                                       CONFIG_ALRT_BIT_ENBL);
+                                       CFG_ALRT_BIT_ENBL,
+                                       CFG_ALRT_BIT_ENBL);
                        max17042_set_soc_threshold(chip, 1);
                } else {
                        client->irq = 0;