Messages in this thread | | | Date | Wed, 15 Mar 2017 14:10:01 +0800 | From | Jisheng Zhang <> | Subject | Re: [PATCH] serial: 8250: 8250_core: Use dev_name() during request_irq() |
| |
On Tue, 14 Mar 2017 17:59:01 +0200 Andy Shevchenko wrote:
> On Tue, Mar 14, 2017 at 3:44 PM, Vignesh R <vigneshr@ti.com> wrote: > > On Monday 13 March 2017 01:35 AM, Andy Shevchenko wrote: > >> On Wed, Mar 8, 2017 at 2:19 PM, Vignesh R <vigneshr@ti.com> 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
| |