Skip to content

Commit a3fa983

Browse files
PatrickRudolphbroonie
authored andcommitted
regulator (max5970): Fix IRQ handler
The max5970 datasheet gives the impression that IRQ status bits must be cleared by writing a one to set bits, as those are marked with 'R/C', however tests showed that a zero must be written. Fixes an IRQ storm as the interrupt handler actually clears the IRQ status bits. Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> Signed-off-by: Naresh Solanki <naresh.solanki@9elements.com> Link: https://msgid.link/r/20240130150257.3643657-1-naresh.solanki@9elements.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent a67e1f0 commit a3fa983

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/regulator/max5970-regulator.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,7 @@ static int max597x_regmap_read_clear(struct regmap *map, unsigned int reg,
392392
return ret;
393393

394394
if (*val)
395-
return regmap_write(map, reg, *val);
395+
return regmap_write(map, reg, 0);
396396

397397
return 0;
398398
}

0 commit comments

Comments
 (0)