On Wed, Aug 20, 2025 at 09:06:52AM +0530, Upadhyay, Neeraj wrote: > > > +static void apic_chipd_update_vector(struct irq_data *irqd, unsigned int newvec, > > > > What is "chipd" supposed to denote? > > chip data (struct apic_chip_data) So this function should be called chip_data_update() or so? It is static so it doesn't need the "apic_" prefix and "chipd" doesn't make a whole lotta of sense so let's call it as what it does. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette