On Wed, 12 Apr 2017, Sathyanarayanan Kuppuswamy Natarajan wrote:
> Hi Lee,
>
> Thanks. Will remove the code segment in next version.
Please always reply inline. Top posting is frowned upon.
> On Wed, Apr 12, 2017 at 3:45 AM, Lee Jones <lee.jones@xxxxxxxxxx> wrote:
> > On Mon, 10 Apr 2017, sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx wrote:
> >
> >> From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx>
> >>
> >> TMU interrupts are registered as a separate interrupt chip, and
> >> hence it should start its interrupt index(BXTWC_TMU_IRQ) number
> >> from 0. But currently, BXTWC_TMU_IRQ is defined as part of enum
> >> bxtwc_irqs_level2 and its index value is 11. Since this index
> >> value is used when calculating .num_irqs of regmap_irq_chip_tmu,
> >> it incorrectly reports number of irqs as 12 instead of actual
> >> value of 1.
> >>
> >> static const struct regmap_irq bxtwc_regmap_irqs_tmu[] = {
> >> REGMAP_IRQ_REG(BXTWC_TMU_IRQ, 0, 0x06),
> >> };
> >>
> >> static struct regmap_irq_chip bxtwc_regmap_irq_chip_tmu = {
> >> .name = "bxtwc_irq_chip_tmu",
> >> .status_base = BXTWC_TMUIRQ,
> >> .mask_base = BXTWC_MTMUIRQ,
> >> .irqs = bxtwc_regmap_irqs_tmu,
> >> .num_irqs = ARRAY_SIZE(bxtwc_regmap_irqs_tmu),
> >> .num_regs = 1,
> >> };
> >>
> >> This patch fixes this issue by creating new enum of tmu irqs and
> >> resetting its starting index to 0.
> >>
> >> Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx>
> >> ---
> >> drivers/mfd/intel_soc_pmic_bxtwc.c | 5 ++++-
> >> 1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > Patch looks fine, but please remove the code segment from the commit
> > log.
> >
> > For the code:
> >
> > For my own reference:
> > Acked-for-MFD-by: Lee Jones <lee.jones@xxxxxxxxxx>
> >
> >> diff --git a/drivers/mfd/intel_soc_pmic_bxtwc.c b/drivers/mfd/intel_soc_pmic_bxtwc.c
> >> index 699c8c7..bb18e20 100644
> >> --- a/drivers/mfd/intel_soc_pmic_bxtwc.c
> >> +++ b/drivers/mfd/intel_soc_pmic_bxtwc.c
> >> @@ -94,7 +94,10 @@ enum bxtwc_irqs_level2 {
> >> BXTWC_GPIO0_IRQ,
> >> BXTWC_GPIO1_IRQ,
> >> BXTWC_CRIT_IRQ,
> >> - BXTWC_TMU_IRQ,
> >> +};
> >> +
> >> +enum bxtwc_irqs_tmu {
> >> + BXTWC_TMU_IRQ = 0,
> >> };
> >>
> >> static const struct regmap_irq bxtwc_regmap_irqs[] = {
> >
>
>
>
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
