On Tue, Jun 07, 2022 at 04:53:08PM +0100, Aidan MacDonald wrote: > Call sub_irq_reg() instead of calculating the offset of the register > to avoid relying on the fact that sub_irq_reg() is a linear function. Seems like unmask_reg is the only register whose address is not calculated using sub_irq_reg(). Switching to using sub_irq_reg() will bring it in line with the other calculations. Could you please incorporate this info in your commit message as well? This should be the rationale for this change; that it allows for the get_irq_reg() patch should be secondary. The change seems okay to me, but I'd ideally like someone to pick this up and test it out just to make sure it doesn't break existing behaviour for them. Thank you. Guru Das.