Unverified Commit 21e9a1dd authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Mark Brown
Browse files

regmap-irq: Consistently use memset32() in regmap_irq_thread() 



The commit 4d60cac9 ("regmap-irq: Add no_status support") adds
an additional branch into IRQ thread handler in regmap. It wisely
chose to use memset32() as it might be optimised on some architectures
and hence give a performance benefit. At the same time the old code
continue using simple memset(). Update the old code to use memset32().

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


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

drivers/base/regmap/regmap-irq.c

+1 −4
Original line number Diff line number Diff line
@@ -364,14 +364,11 @@ static irqreturn_t regmap_irq_thread(int irq, void *d) 
		memset32(data->status_buf, GENMASK(31, 0), chip->num_regs);

	} else if (chip->num_main_regs) {

		unsigned int max_main_bits;

		unsigned long size;



		size = chip->num_regs * sizeof(unsigned int);



		max_main_bits = (chip->num_main_status_bits) ?

				 chip->num_main_status_bits : chip->num_regs;

		/* Clear the status buf as we don't read all status regs */

		memset(data->status_buf, 0, size);

		memset32(data->status_buf, 0, chip->num_regs);



		/* We could support bulk read for main status registers

		 * but I don't expect to see devices with really many main