Re: [PATCH 2.6.11 3/8] tg3: Add msi support for 5750 C0

[Jeff Garzik <jgarzik@xxxxxxxxx>]

Michael Chan wrote:
> Add MSI support for 5750 C0 and newer chips. MSI is handled by a new
> interrupt service routine that is very similar to the existing
> tg3_interrupt() for INTx mode. The MSI version is optimized by not checking
> for shared interrupt and not flushing the status block.
>
> The code is also changed to reference the irq from tp->pdev->irq instead of
> dev->irq.
>
> Signed-off-by: Michael Chan <mchan@xxxxxxxxxxxx

I would prefer that a new 'static inline' function was created, 
containing the core interrupt handler code.

This avoids the code duplication you have in this patch, in new function 
tg3_msi().

        Jeff