Re: [PATCH 2/5] mfd: tps65218: make INT1 our status_base register
On Fri, 26 Dec 2014, Felipe Balbi wrote: > If we don't tell regmap-irq that our first status > register is at offset 1, it will try to read offset > zero, which is the chipid register. > > Fixes: 44b4dc6 mfd: tps65218: Add driver for the TPS65218 PMIC > Cc: # v3.15+ > Cc: Keerthy > Cc: Lee Jones > Signed-off-by: Felipe Balbi > --- > drivers/mfd/tps65218.c | 1 + > 1 file changed, 1 insertion(+) Sorry for the delay. It's difficult to get a WiFi signal 2000m up in an Austrian mountain. :) Applied now, thanks. > diff --git a/drivers/mfd/tps65218.c b/drivers/mfd/tps65218.c > index 2243f75..d6b7643 100644 > --- a/drivers/mfd/tps65218.c > +++ b/drivers/mfd/tps65218.c > @@ -204,6 +204,7 @@ static struct regmap_irq_chip tps65218_irq_chip = { > > .num_regs = 2, > .mask_base = TPS65218_REG_INT_MASK1, > + .status_base = TPS65218_REG_INT1, > }; > > static const struct of_device_id of_tps65218_match_table[] = { -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 2/5] mfd: tps65218: make INT1 our status_base register
On Fri, Dec 26, 2014 at 01:28:21PM -0600, Felipe Balbi wrote: > If we don't tell regmap-irq that our first status > register is at offset 1, it will try to read offset > zero, which is the chipid register. > > Fixes: 44b4dc6 mfd: tps65218: Add driver for the TPS65218 PMIC > Cc: # v3.15+ > Cc: Keerthy > Cc: Lee Jones > Signed-off-by: Felipe Balbi ping > --- > drivers/mfd/tps65218.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mfd/tps65218.c b/drivers/mfd/tps65218.c > index 2243f75..d6b7643 100644 > --- a/drivers/mfd/tps65218.c > +++ b/drivers/mfd/tps65218.c > @@ -204,6 +204,7 @@ static struct regmap_irq_chip tps65218_irq_chip = { > > .num_regs = 2, > .mask_base = TPS65218_REG_INT_MASK1, > + .status_base = TPS65218_REG_INT1, > }; > > static const struct of_device_id of_tps65218_match_table[] = { > -- > 2.2.0 > -- balbi signature.asc Description: Digital signature
[PATCH 2/5] mfd: tps65218: make INT1 our status_base register
If we don't tell regmap-irq that our first status register is at offset 1, it will try to read offset zero, which is the chipid register. Fixes: 44b4dc6 mfd: tps65218: Add driver for the TPS65218 PMIC Cc: # v3.15+ Cc: Keerthy Cc: Lee Jones Signed-off-by: Felipe Balbi --- drivers/mfd/tps65218.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mfd/tps65218.c b/drivers/mfd/tps65218.c index 2243f75..d6b7643 100644 --- a/drivers/mfd/tps65218.c +++ b/drivers/mfd/tps65218.c @@ -204,6 +204,7 @@ static struct regmap_irq_chip tps65218_irq_chip = { .num_regs = 2, .mask_base = TPS65218_REG_INT_MASK1, + .status_base = TPS65218_REG_INT1, }; static const struct of_device_id of_tps65218_match_table[] = { -- 2.2.0 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html