On Tue, May 17, 2022 at 01:09:53PM +0300, Mika Westerberg wrote: > On Mon, May 16, 2022 at 09:56:17PM +0300, Andy Shevchenko wrote: > > Since recently, the kernel is nagging about mutable irq_chips: > > > > "not an immutable chip, please consider fixing it!" > > > > Drop the unneeded copy, flag it as IRQCHIP_IMMUTABLE, add the new > > helper functions and call the appropriate gpiolib functions. > > > > While at it, align IRQ chip callback names with member names for > > the sake of consistency. > > Or for the sake of inconsistency? ;-) All the other structs are not > using this kind of alignment and I would like to keep this one matching > the rest too. Also I prefer the format currently used in the driver. I got your point. I will drop that part from v2. -- With Best Regards, Andy Shevchenko