Re: [PATCH 2.6 ] MPT Fusion driver 3.01.02 update

[Jeff Garzik <jgarzik@pobox.com>]

Moore, Eric Dean wrote:
> This is an update for the MPT Fusion drivers 2.6 kernel.
> Version 3.01.02.
> 
> Changelog:
> 
> (1) Andi Kleen[ak@suse.de]
> put warning "Device (0:0:0) reported QUEUE_FULL!" into debug messages
> 
> (2) Alexander Stohr[Alexander.Stohr@gmx.de]
> fix warnings from mptscsih_setup when driver isn't compiled as module
> 
> (3) Randy.Dunlap[rddunlap@osdl.org]
> Remove unnecessary min/max macros and change calls to 
> use kernel.h macros instead.

Here are some wrappers to remove:

static inline void CHIPREG_PIO_WRITE32(volatile u32 *a, u32 v)
{
         outl(v, (unsigned long)a);
}

static inline u32 CHIPREG_PIO_READ32(volatile u32 *a)
{
         return inl((unsigned long)a);
}


Also, I'm not sure about these wrappers, but it's mainly as style issue 
due to the "if (PortIo)" test.  You should remove the 'volatile' 
markers, at least.


static inline u32 CHIPREG_READ32(volatile u32 *a)
{
         if (PortIo)
                 return inl((unsigned long)a);
         else
                 return readl(a);
}

static inline void CHIPREG_WRITE32(volatile u32 *a, u32 v)
{
         if (PortIo)
                 outl(v, (unsigned long)a);
         else
                 writel(v, a);
}