regmap-irq: Minor adjustments to .handle_mask_sync()
authorAidan MacDonald <aidanmacdonald.0x0@gmail.com>
Thu, 11 May 2023 09:13:41 +0000 (10:13 +0100)
committerMark Brown <broonie@kernel.org>
Fri, 12 May 2023 01:28:02 +0000 (10:28 +0900)
commita240d23ee9dce2a9fe68d614f19a463d7029fb87
tree4125126bcb0c7b91467b739f7884d71cf907d8e4
parent72cc0f523babca3886421721aa662c7d352a6d32
regmap-irq: Minor adjustments to .handle_mask_sync()

If a .handle_mask_sync() callback is provided it supersedes all
inbuilt handling of mask registers, and judging by the commit
69af4bcaa08d ("regmap-irq: Add handle_mask_sync() callback") it
is intended to completely replace all default IRQ masking logic.

The implementation has two minor inconsistencies, which can be
fixed without breaking compatibility:

(1) mask_base must be set to enable .handle_mask_sync(), even
    though mask_base is otherwise unused. This is easily fixed
    because mask_base is already optional.

(2) Unmask registers aren't accounted for -- they are part of
    the default IRQ masking logic and are just a bit-inverted
    version of mask registers. It would be a bad idea to allow
    them to be used at the same time as .handle_mask_sync(),
    as the result would be confusing and unmaintainable, so
    make sure this can't happen.

Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com
Link: https://lore.kernel.org/r/20230511091342.26604-4-aidanmacdonald.0x0@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org
drivers/base/regmap/regmap-irq.c