Subject: Re: [PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25

From: Ben Dooks <[email-masked]>
Date: Tue, 5 Feb 2008 01:44:18 +0000

On Mon, Feb 04, 2008 at 04:19:40PM -0800, Andrew Morton wrote:
> On Tue, 05 Feb 2008 00:01:59 +0000
> Ben Dooks <[email-masked]> wrote:
> 
> > Subject: [PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25
> 
> Holy cow.
> 
> > This patch set is a series of updates for the DM9000
> > driver, to tidy-up some of the source, stop the accesses
> > to the PHY and EEPROM sitting and spinning with locks
> > held, and to add ethtool support.
> 
> Jeff, the immediate issue is that the driver doesn't compile on mips.  I
> have the below lameo fix for it, but it appears to be wrong.  Or at least
> suboptimal.
> 
> So if you're unprepared to chew on this lot (and 24 patches two weeks into the
> merge window is one hell of a chew) then we do need to get that
> regression fixed, at least.

To be fair, jeff has already acked most of these patches, this set is
a repost hopefully without the offending patches.
 
> 
> 
> From: Andrew Morton <[email-masked]>
> 
> mips:
> 
> drivers/net/dm9000.c: In function `dm9000_open':
> drivers/net/dm9000.c:627: error: `IRQT_RISING' undeclared (first use in this function)
> drivers/net/dm9000.c:627: error: (Each undeclared identifier is reported only once
> drivers/net/dm9000.c:627: error: for each function it appears in.)
> 
> Cc: Daniel Mack <[email-masked]>
> Cc: Russell King <[email-masked]>
> Signed-off-by: Andrew Morton <[email-masked]>
> ---
> 
>  drivers/net/dm9000.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff -puN drivers/net/dm9000.c~drivers-net-dm9000c-vague-probably-wrong-build-fix drivers/net/dm9000.c
> --- a/drivers/net/dm9000.c~drivers-net-dm9000c-vague-probably-wrong-build-fix
> +++ a/drivers/net/dm9000.c
> @@ -113,8 +113,10 @@
>  #define writesw	outsw
>  #define writesl	outsl
>  #define DM9000_IRQ_FLAGS	(IRQF_SHARED | IRQF_TRIGGER_HIGH)
> -#else
> +#elif defined(ARM)
>  #define DM9000_IRQ_FLAGS	(IRQF_SHARED | IRQT_RISING)
> +#else
> +#define DM9000_IRQ_FLAGS	(IRQF_SHARED)
>  #endif
>  
>  /*
> _
> 
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to [email-masked]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Ben ([email-masked], http://www.fluff.org/)

  'a smiley only costs 4 bytes'