regmap-irq: Extend sub-irq to support non-fixed reg strides
authorGuru Das Srinagesh <gurus@codeaurora.org>
Thu, 11 Mar 2021 00:39:52 +0000 (16:39 -0800)
committerMark Brown <broonie@kernel.org>
Thu, 18 Mar 2021 13:55:33 +0000 (13:55 +0000)
commit1066cfbdfa3f5c401870fad577fe63d1171a5bcd
tree39aa854b386d850adfb9ee869931e9af4d71c0e2
parente41a962f82e7afb5b1ee644f48ad0b3aee656268
regmap-irq: Extend sub-irq to support non-fixed reg strides

Qualcomm's MFD chips have a top level interrupt status register and
sub-irqs (peripherals).  When a bit in the main status register goes
high, it means that the peripheral corresponding to that bit has an
unserviced interrupt. If the bit is not set, this means that the
corresponding peripheral does not.

Commit a2d21848d9211d ("regmap: regmap-irq: Add main status register
support") introduced the sub-irq logic that is currently applied only
when reading status registers, but not for any other functions like acking
or masking. Extend the use of sub-irq to all other functions, with two
caveats regarding the specification of offsets:

- Each member of the sub_reg_offsets array should be of length 1
- The specified offsets should be the unequal strides for each sub-irq
  device.

In QCOM's case, all the *_base registers are to be configured to the
base addresses of the first sub-irq group, with offsets of each
subsequent group calculated as a difference from these addresses.

Continuing from the example mentioned in the cover letter:

/*
 * Address of MISC_INT_MASK = 0x1011
 * Address of TEMP_ALARM_INT_MASK = 0x2011
 * Address of GPIO01_INT_MASK = 0x3011
 *
 * Calculate offsets as:
 * offset_0 = 0x1011 - 0x1011 = 0       (to access MISC's
 *   registers)
 * offset_1 = 0x2011 - 0x1011 = 0x1000
 * offset_2 = 0x3011 - 0x1011 = 0x2000
 */

static unsigned int sub_unit0_offsets[] = {0};
static unsigned int sub_unit1_offsets[] = {0x1000};
static unsigned int sub_unit2_offsets[] = {0x2000};

static struct regmap_irq_sub_irq_map chip_sub_irq_offsets[] = {
REGMAP_IRQ_MAIN_REG_OFFSET(sub_unit0_offsets),
REGMAP_IRQ_MAIN_REG_OFFSET(sub_unit0_offsets),
REGMAP_IRQ_MAIN_REG_OFFSET(sub_unit0_offsets),
};

static struct regmap_irq_chip chip_irq_chip = {
--------8<--------
.not_fixed_stride = true,
.mask_base   = MISC_INT_MASK,
.type_base   = MISC_INT_TYPE,
.ack_base   = MISC_INT_ACK,
.sub_reg_offsets  = chip_sub_irq_offsets,
--------8<--------
};

Signed-off-by: Guru Das Srinagesh <gurus@codeaurora.org>
Link: https://lore.kernel.org/r/526562423eaa58b4075362083f561841f1d6956c.1615423027.git.gurus@codeaurora.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/base/regmap/regmap-irq.c
include/linux/regmap.h