mfd: rc5t583: Set regmap config reg counts properly
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Sun, 31 Jan 2016 21:58:41 +0000 (22:58 +0100)
committerLee Jones <lee.jones@linaro.org>
Wed, 16 Mar 2016 08:50:34 +0000 (08:50 +0000)
Regmap config max_register field should contain number of
device last register, however num_reg_defaults_raw field
should be set to register count instead
(usually one register more than max_register).

rc5t583 driver had both of these fields set to the same value,
fix this by introducing separate defines for max register
number and total count of registers.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/rc5t583.c
include/linux/mfd/rc5t583.h

index e10f02f5d55102f6f3a8a57f6d367394c0a690b9..fc2b2d93f354c269134191f2629cf8e646696c12 100644 (file)
@@ -241,8 +241,8 @@ static const struct regmap_config rc5t583_regmap_config = {
        .reg_bits = 8,
        .val_bits = 8,
        .volatile_reg = volatile_reg,
-       .max_register = RC5T583_MAX_REGS,
-       .num_reg_defaults_raw = RC5T583_MAX_REGS,
+       .max_register = RC5T583_MAX_REG,
+       .num_reg_defaults_raw = RC5T583_NUM_REGS,
        .cache_type = REGCACHE_RBTREE,
 };
 
index fd413ccab915c2303addb248ce0296ae877483b6..8d0a392e0a7ff6f9cf76fc73c918fc4c1f3d48f0 100644 (file)
@@ -28,8 +28,6 @@
 #include <linux/types.h>
 #include <linux/regmap.h>
 
-#define RC5T583_MAX_REGS               0xF8
-
 /* Maximum number of main interrupts */
 #define MAX_MAIN_INTERRUPT             5
 #define RC5T583_MAX_GPEDGE_REG         2
 #define RC5T583_RTC_AY_MONTH 0xF3
 #define RC5T583_RTC_AY_YEAR    0xF4
 
+#define RC5T583_MAX_REG                0xF7
+#define RC5T583_NUM_REGS       (RC5T583_MAX_REG + 1)
+
 /* RICOH_RC5T583 IRQ definitions */
 enum {
        RC5T583_IRQ_ONKEY,