On Tue, 14 Mar 2017 17:59:01 +0200 Andy Shevchenko wrote: > On Tue, Mar 14, 2017 at 3:44 PM, Vignesh R <vigneshr@xxxxxx> wrote: > > On Monday 13 March 2017 01:35 AM, Andy Shevchenko wrote: > >> On Wed, Mar 8, 2017 at 2:19 PM, Vignesh R <vigneshr@xxxxxx> wrote: > >>> Passing "serial" as name during request_irq() results in all serial port > >>> irqs have same name. This does not help much to easily identify which > >>> irq belongs to which serial port instance. Therefore pass dev_name() > >>> during request_irq() so that better identifiable name is listed for > >>> serial ports in cat /proc/interrupts output. > >>> > >>> Output of cat /proc/interrupts > >>> Before this patch: > >>> 26: 689 0 GICv2 309 Edge serial > >>> After this patch: > >>> 26: 696 0 GICv2 309 Edge 2530c00.serial > > > But, this change is making interrupt name more descriptive > > It doesn't in PCI case, it makes it worse. indeed, in PCI case, I see 4: ... 00:03.3 what about "dev_driver_string(up->port.dev)[dev_name(up->port.dev)]" as we did for ahci case? Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
