Re: [PATCH]dgrs - Fixes Warnings when CONFIG_ISA and CONFIG_PCI arenot enabled

[Richard Knutsson]

> > > This patch fixes compiler warnings when CONFIG_ISA and CONFIG_PCI are
> > > not enabled in the dgrc network driver.
> > >
> > > Signed-off-by: Ashutosh Naik <ashutosh.naik@xxxxxxxxx>
> > >
> > > --
> > > diff -Naurp linux-2.6.14/drivers/net/dgrs.c
> > > linux-2.6.14-git1/drivers/net/dgrs.c---
> > > linux-2.6.14/drivers/net/dgrs.c     2005-10-28 05:32:08.000000000
> > > +0530
> > > +++ linux-2.6.14-git1/drivers/net/dgrs.c        2005-11-01
> > > 10:30:03.000000000 +0530
> > > @@ -1549,8 +1549,12 @@ MODULE_PARM_DESC(nicmode, "Digi RightSwi
> > > static int __init dgrs_init_module (void)  {
> > >       int     i;
> > > -       int eisacount = 0, pcicount = 0;
> > > -
> > > +#ifdef CONFIG_EISA
> > > +       int eisacount = 0;
> > > +#endif
> > > +#ifdef CONFIG_PCI
> > > +       int pcicount = 0;
> > > +#endif
> > >       /*
> > >        *      Command line variable overrides
> > >        *              debug=NNN
> > > -
> > >      

> > Signed-off-by: Richard Knutsson <ricknu-0@xxxxxxxxxxxxxx>
> >
> > ---
> >
> > diff -uNr a/drivers/net/dgrs.c b/drivers/net/dgrs.c
> > --- a/drivers/net/dgrs.c        2005-08-29 01:41:01.000000000 +0200
> > +++ b/drivers/net/dgrs.c        2005-10-26 15:53:43.000000000 +0200
> > @@ -1549,7 +1549,7 @@
> > static int __init dgrs_init_module (void)
> > {
> >        int     i;
> > -       int eisacount = 0, pcicount = 0;
> > +       int     count;
> >
> >        /*
> >         *      Command line variable overrides
> > @@ -1591,14 +1591,14 @@
> >         *      Find and configure all the cards
> >         */
> > #ifdef CONFIG_EISA
> > -       eisacount = eisa_driver_register(&dgrs_eisa_driver);
> > -       if (eisacount < 0)
> > -               return eisacount;
> > +       count = eisa_driver_register(&dgrs_eisa_driver);
> > +       if (count < 0)
> > +               return count;
> > #endif
> > #ifdef CONFIG_PCI
> > -       pcicount = pci_register_driver(&dgrs_pci_driver);
> > -       if (pcicount)
> > -               return pcicount;
> > +       count = pci_register_driver(&dgrs_pci_driver);
> > +       if (count)
> > +               return count;
> > #endif
> >        return 0;
> > }
> >    
>
> Well, both of them do the same stuff, but one of these patches needs
> to be committed.
>
> Cheers
> Ashutosh
>  
Can both CONFIG_PCI and CONFIG_EISA be undefined at the same time? If 
so, I think you patch is better.
But on line 1015 in the dgrs.c-file (function dgrs_download()) there is 
an if-statement:
   if (priv0->plxreg)
   {    /* PCI bus */
       ...
   }
   else
   {    /* EISA bus */
       ...
from where I got the idea it needs either pci or eisa (or both). If this 
is true, I vote for my patch.

Live long and prosper
/Richard

PS
Rick's mail-address in the file seems invalid. Changed it to 
netdev@xxxxxxxxxxxxxxx, since that is the address in the MAINTAINERS-file.
DS

-
To unsubscribe from this list: send the line "unsubscribe linux-net" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html