Re: [PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25
On Tuesday 05 February 2008 01:01, Ben Dooks wrote: > 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. > > This set includes a pair of patches from Laurent Pinchart > for addition platform-data based configuration, please > apply these from this if you can attribute these correctly > to Laurent, otherwise ask Laurent to resubmit. I have > signed-off-by both these patches, but would be equally > happy acking them. If you can apply the patches without a resubmission that's of course less work for me :-) Otherwise I can resubmit them. > The only missing item in this series is to replace the > old timer based polling code. This series has been sitting > long enough on my own trees, and having MII link status > reports is not a driver-threatening bug. > > Hopefully this series meets up to everyone's approval and > can be applied whilst the current merge window is upon us. -- Laurent Pinchart CSE Semaphore Belgium Chaussée de Bruxelles, 732A B-1410 Waterloo Belgium T +32 (2) 387 42 59 F +32 (2) 387 42 75 pgp3voV0Kyyb4.pgp Description: PGP signature
Re: [PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25
On Mon, Feb 04, 2008 at 04:19:40PM -0800, Andrew Morton wrote: > On Tue, 05 Feb 2008 00:01:59 + > Ben Dooks <[EMAIL PROTECTED]> 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 PROTECTED]> > > 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 PROTECTED]> > Cc: Russell King <[EMAIL PROTECTED]> > Signed-off-by: Andrew Morton <[EMAIL PROTECTED]> > --- > > 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 PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Ben ([EMAIL PROTECTED], http://www.fluff.org/) 'a smiley only costs 4 bytes' -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25
On Tue, 05 Feb 2008 00:01:59 + Ben Dooks <[EMAIL PROTECTED]> 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. From: Andrew Morton <[EMAIL PROTECTED]> 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 PROTECTED]> Cc: Russell King <[EMAIL PROTECTED]> Signed-off-by: Andrew Morton <[EMAIL PROTECTED]> --- 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 writeswoutsw #define writesloutsl #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 PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 00/24 for-2.6.25] DM9000 updates for 2.6.25
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. This set includes a pair of patches from Laurent Pinchart for addition platform-data based configuration, please apply these from this if you can attribute these correctly to Laurent, otherwise ask Laurent to resubmit. I have signed-off-by both these patches, but would be equally happy acking them. The only missing item in this series is to replace the old timer based polling code. This series has been sitting long enough on my own trees, and having MII link status reports is not a driver-threatening bug. Hopefully this series meets up to everyone's approval and can be applied whilst the current merge window is upon us. -- Ben -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html