Unverified Commit 153dbf4a authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Mark Brown
Browse files

regmap: irq: Use one way of setting all bits in the register 



Currently there are two ways of how we represent all bits set, i.e.
UINT_MAX and GENMASK(31, 0). Use the former as the single way of
doing that, which is crystal clear on how we fill the unsigned int
value.

Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20250206191644.1132869-1-andriy.shevchenko@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 995cf0e0

drivers/base/regmap/regmap-irq.c

+1 −1
Original line number Diff line number Diff line
@@ -823,7 +823,7 @@ int regmap_add_irq_chip_fwnode(struct fwnode_handle *fwnode, 
		/* Ack masked but set interrupts */

		if (d->chip->no_status) {

			/* no status register so default to all active */

			d->status_buf[i] = GENMASK(31, 0);

			d->status_buf[i] = UINT_MAX;

		} else {

			reg = d->get_irq_reg(d, d->chip->status_base, i);

			ret = regmap_read(map, reg, &d->status_buf[i]);