[vortex] 3c59x.c compile errors

braganca@cenpes.petrobras.com.br braganca@cenpes.petrobras.com.br
Fri, 17 Nov 2000 07:59:47 -0200


The driver compiles OK after joining continued lines
WOL finaly works and NIC goes to Full Duplex without the need of options
line
THANKS!!!

----- Repassado por Ricardo Silva Nunes Braganca/CENPES/Petrobras em
17/11/00 07:52 -----
                                                                                   
                    Donald Becker                                                  
                    <becker@scyld        Para:   braganca@cenpes.petrobras.com.br  
                    .com>                cc:     vortex@scyld.com                  
                    Enviado Por:         Assunto:     Re: [vortex] 3c59x.c compile 
                    vortex-admin@        errors                                    
                    scyld.com                                                      
                                                                                   
                                                                                   
                    14/11/00                                                       
                    17:03                                                          
                                                                                   
                                                                                   



On Tue, 14 Nov 2000 braganca@cenpes.petrobras.com.br wrote:

>   Erros found compiling 3c59x.c:
>
___________________________________________________________________________
>   [root@s2 modules]# gcc -DMODULE -Wall -Wstrict-prototypes -O6 -c
3c59x.c
>   In file included from 3c59x.c:112:
>   kern_compat.h:164: parse error before `0'
>   kern_compat.h:168: warning: type defaults to `int' in declaration of
>   `mark_bh'

You've got a buggy compiler.

>   Aditional information:
>   Linux Kernel 2.2.14
>   egcs-1.1.2
>   glibc-2.1.2

Adding a joining the continued lines, or adding a few spaces in
kern_compat.h will work around this pre-processor bug.

Add the spaces around the *_bit(0,..) functions to make them read
  *_bit( 0,..)

#define netif_pause_tx_queue(dev) \
    (test_and_set_bit( 0, (void*)&(dev)->tbusy))
#define netif_unpause_tx_queue(dev) \
    do { clear_bit( 0, (void*)&(dev)->tbusy); } while (0)
#define netif_resume_tx_queue(dev) \
    do { clear_bit( 0, (void*)&(dev)->tbusy); mark_bh(NET_BH); } while (0)
#endif


Donald Becker                                       becker@scyld.com
Scyld Computing Corporation                    http://www.scyld.com
410 Severn Ave. Suite 210                      Second Generation Beowulf
Clusters
Annapolis MD 21403                             410-990-9993