lkml.org 
[lkml]   [2018]   [Feb]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net v4 09/13] net/mac8390: Convert to nubus_driver
On Mon, 12 Feb 2018, Geert Uytterhoeven wrote:

> On Mon, Feb 12, 2018 at 4:08 AM, Finn Thain <fthain@telegraphics.com.au> wrote:
> > This resolves an old bug that constrained this driver to no more than
> > one card.
> >
> > Tested-by: Stan Johnson <userm57@yahoo.com>
> > Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
>
> > --- a/drivers/net/ethernet/8390/mac8390.c
> > +++ b/drivers/net/ethernet/8390/mac8390.c
>
> > @@ -390,86 +389,86 @@ static bool __init mac8390_init(struct net_device *dev,
> > return true;
> > }
> >
> > -struct net_device * __init mac8390_probe(int unit)
> > +static int mac8390_device_probe(struct nubus_board *board)
> > {
> > struct net_device *dev;
> > - struct nubus_rsrc *ndev = NULL;
> > int err = -ENODEV;
> > - static unsigned int slots;
> > -
> > - enum mac8390_type cardtype;
> > -
> > - /* probably should check for Nubus instead */
> > + struct nubus_rsrc *fres;
> > + enum mac8390_type cardtype = MAC8390_NONE;
> >
> > if (!MACH_IS_MAC)
> > - return ERR_PTR(-ENODEV);
> > + return -ENODEV;
>
> I think this check can be removed completely,

I agree.

> as the nubus_board will exist on suitable Macs only.
>

And considering the out-of-tree Nubus PowerMac port, this check just makes
the driver less portable.

I'll resubmit these patches with the changes you have suggested (here and
elsewhere). Thanks for your review.

BTW, would you be willing to review the rest of this series sometime? I
ask because David has voiced concerns about code quality, and your
"reviewed-by" tag speaks volumes.

--

> Gr{oetje,eeting}s,
>
> Geert
>

\
 
 \ /
  Last update: 2018-02-13 06:04    [W:0.047 / U:0.572 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site