Re: [PATCH] typhoon: switch from 'pci_' to 'dma_' API

2020-08-23 Thread David Dillow
ile tested. Looks good, thanks! Reviewed-by: David Dillow

Re: [PATCH v3 2/2] drivers: net: ethernet: 3com: fix return value

2016-12-27 Thread David Dillow
On Sun, 2016-12-25 at 01:30 +0100, Thomas Preisner wrote: > In some cases the return value of a failing function is not being used > and the function typhoon_init_one() returns another negative error > code instead. I'm not sure these changes are especially valuable, since we'll need to look at th

Re: Re: [PATCH v2 1/2] drivers: net: ethernet: 3com: fix return value

2016-12-27 Thread David Dillow
On Sun, 2016-12-25 at 01:30 +0100, Thomas Preisner wrote: > Those spaces were actually left out purposely: The file in question > (typhoon.c) > is missing those spaces between the statements (if, for, while) and the > following opening bracket pretty much always (except 2-3 times) and we figured >

Re: [PATCH] drivers: net: ethernet: 3com: fix return value

2016-12-23 Thread David Dillow
On Sat, 2016-12-24 at 00:00 +0100, Thomas Preisner wrote: > diff --git a/drivers/net/ethernet/3com/typhoon.c > b/drivers/net/ethernet/3com/typhoon.c > index a0cacbe..9a3ab58 100644 > --- a/drivers/net/ethernet/3com/typhoon.c > +++ b/drivers/net/ethernet/3com/typhoon.c > @@ -2404,6 +2404,7 @@ typho

Re: [PATCH] net: 3com: typhoon: use new api ethtool_{get|set}_link_ksettings

2016-11-01 Thread David Dillow
On Wed, 2016-11-02 at 00:11 +0100, Philippe Reynes wrote: > The ethtool api {get|set}_settings is deprecated. > We move this driver to new api {get|set}_link_ksettings. > > Signed-off-by: Philippe Reynes LGTM. Reviewed-by: David Dillow

Re: [patch 1/7] typhoon section fix

2008-02-08 Thread David Dillow
On Fri, 2008-02-08 at 09:52 -0800, Andrew Morton wrote: > On Fri, 08 Feb 2008 08:59:10 -0500 David Dillow <[EMAIL PROTECTED]> wrote: > > On Fri, 2008-02-08 at 03:11 -0800, [EMAIL PROTECTED] wrote: > > > From: Andrew Morton <[EMAIL PROTECTED]> >

Re: [patch 1/7] typhoon section fix

2008-02-08 Thread David Dillow
On Fri, 2008-02-08 at 03:11 -0800, [EMAIL PROTECTED] wrote: > From: Andrew Morton <[EMAIL PROTECTED]> > > gcc-3.4.4 on powerpc: > > drivers/net/typhoon.c:137: error: version causes a section type conflict > > Cc: Jeff Garzik <[EMAIL PROTECTED]> > Cc: Sam Ravnborg <[EMAIL PROTECTED]> > Signed-of