On Wed, Mar 7, 2018 at 6:36 AM, Mark Brown <broo...@kernel.org> wrote: > On Tue, Mar 06, 2018 at 06:57:49AM -0800, Tony Lindgren wrote: > >> > By using regmap_irq_update_bits to ACK the interrupts we use the masked >> > status bits so we take care not to affect any other bits then use >> > ack_invert to determine if we clear or set those bits. > >> This change to use regmap_irq_update_bits() now breaks things for >> me with cpcap interrupts. So it seems to cause a non-inverted mode >> regression. There should be no need to read the ack register, I >> gues that's the whole idea of having a separate ack register :) > > Yes, that'd be my expectation as well - the register should be just > write only. regmap_update_bits() definitely isn't the right thing here > since it will suppress the write part of the read/modify/write cycle if > it detects that it didn't actually modify anything as an optimization.
understood. I will put together a v3 soon. Thanks, Tim