power: axp209: Define the chip version mask
authorOlliver Schinagl <oliver@schinagl.nl>
Wed, 21 Nov 2018 18:05:28 +0000 (20:05 +0200)
committerJagan Teki <jagan@amarulasolutions.com>
Fri, 7 Dec 2018 16:54:33 +0000 (22:24 +0530)
Use a define for the chip version mask on the axp209.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
Signed-off-by: Priit Laes <plaes@plaes.org>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
drivers/power/axp209.c
include/axp209.h

index 6de5ec6825804e309e4a70616553488bdb004166..e13c762a8109b51d320bf1302f179f1526c7ac31 100644 (file)
@@ -153,10 +153,7 @@ int axp_init(void)
        if (rc)
                return rc;
 
-       /* Low 4 bits is chip version */
-       ver &= 0x0f;
-
-       if (ver != 0x1)
+       if ((ver & AXP209_CHIP_VERSION_MASK) != 0x1)
                return -EINVAL;
 
        /* Mask all interrupts */
index 3a4fc281960133de8f36f23b006bb3aa7e90fd40..43e3873e24091cdd3c4d43f03dbcea9f88f92b4c 100644 (file)
@@ -25,6 +25,8 @@ enum axp209_reg {
 #define AXP209_POWER_STATUS_ON_BY_DC   BIT(0)
 #define AXP209_POWER_STATUS_VBUS_USABLE        BIT(4)
 
+#define AXP209_CHIP_VERSION_MASK       0x0f
+
 #define AXP209_OUTPUT_CTRL_EXTEN       BIT(0)
 #define AXP209_OUTPUT_CTRL_DCDC3       BIT(1)
 #define AXP209_OUTPUT_CTRL_LDO2                BIT(2)