diff options
Diffstat (limited to 'drivers/net/fddi')
| -rw-r--r-- | drivers/net/fddi/defxx.c | 38 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/fplustm.c | 29 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/h/smc.h | 28 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/h/supern_2.h | 96 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/h/targetos.h | 1 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/skfddi.c | 7 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/smt.c | 2 | ||||
| -rw-r--r-- | drivers/net/fddi/skfp/srf.c | 24 | 
8 files changed, 107 insertions, 118 deletions
diff --git a/drivers/net/fddi/defxx.c b/drivers/net/fddi/defxx.c index 0b40e1c46f0..2aa57270838 100644 --- a/drivers/net/fddi/defxx.c +++ b/drivers/net/fddi/defxx.c @@ -206,7 +206,6 @@  #include <linux/eisa.h>  #include <linux/errno.h>  #include <linux/fddidevice.h> -#include <linux/init.h>  #include <linux/interrupt.h>  #include <linux/ioport.h>  #include <linux/kernel.h> @@ -241,12 +240,6 @@ static char version[] =   */  #define NEW_SKB_SIZE (PI_RCV_DATA_K_SIZE_MAX+128) -#ifdef CONFIG_PCI -#define DFX_BUS_PCI(dev) (dev->bus == &pci_bus_type) -#else -#define DFX_BUS_PCI(dev) 0 -#endif -  #ifdef CONFIG_EISA  #define DFX_BUS_EISA(dev) (dev->bus == &eisa_bus_type)  #else @@ -298,7 +291,11 @@ static int		dfx_hw_dma_uninit(DFX_board_t *bp, PI_UINT32 type);  static int		dfx_rcv_init(DFX_board_t *bp, int get_buffers);  static void		dfx_rcv_queue_process(DFX_board_t *bp); +#ifdef DYNAMIC_BUFFERS  static void		dfx_rcv_flush(DFX_board_t *bp); +#else +static inline void	dfx_rcv_flush(DFX_board_t *bp) {} +#endif  static netdev_tx_t dfx_xmt_queue_pkt(struct sk_buff *skb,  				     struct net_device *dev); @@ -436,7 +433,7 @@ static void dfx_port_read_long(DFX_board_t *bp, int offset, u32 *data)  static void dfx_get_bars(struct device *bdev,  			 resource_size_t *bar_start, resource_size_t *bar_len)  { -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_eisa = DFX_BUS_EISA(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev);  	int dfx_use_mmio = DFX_MMIO || dfx_bus_tc; @@ -518,7 +515,7 @@ static const struct net_device_ops dfx_netdev_ops = {  static int dfx_register(struct device *bdev)  {  	static int version_disp; -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev);  	int dfx_use_mmio = DFX_MMIO || dfx_bus_tc;  	const char *print_name = dev_name(bdev); @@ -667,7 +664,7 @@ static void dfx_bus_init(struct net_device *dev)  {  	DFX_board_t *bp = netdev_priv(dev);  	struct device *bdev = bp->bus_dev; -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_eisa = DFX_BUS_EISA(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev);  	int dfx_use_mmio = DFX_MMIO || dfx_bus_tc; @@ -813,7 +810,7 @@ static void dfx_bus_uninit(struct net_device *dev)  {  	DFX_board_t *bp = netdev_priv(dev);  	struct device *bdev = bp->bus_dev; -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_eisa = DFX_BUS_EISA(bdev);  	u8 val; @@ -967,7 +964,7 @@ static int dfx_driver_init(struct net_device *dev, const char *print_name,  {  	DFX_board_t *bp = netdev_priv(dev);  	struct device *bdev = bp->bus_dev; -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_eisa = DFX_BUS_EISA(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev);  	int dfx_use_mmio = DFX_MMIO || dfx_bus_tc; @@ -1877,7 +1874,7 @@ static irqreturn_t dfx_interrupt(int irq, void *dev_id)  	struct net_device *dev = dev_id;  	DFX_board_t *bp = netdev_priv(dev);  	struct device *bdev = bp->bus_dev; -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_eisa = DFX_BUS_EISA(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev); @@ -2856,7 +2853,7 @@ static int dfx_hw_dma_uninit(DFX_board_t *bp, PI_UINT32 type)   *	Align an sk_buff to a boundary power of 2   *   */ - +#ifdef DYNAMIC_BUFFERS  static void my_skb_align(struct sk_buff *skb, int n)  {  	unsigned long x = (unsigned long)skb->data; @@ -2866,7 +2863,7 @@ static void my_skb_align(struct sk_buff *skb, int n)  	skb_reserve(skb, v - x);  } - +#endif  /*   * ================ @@ -3081,10 +3078,7 @@ static void dfx_rcv_queue_process(  					break;  					}  				else { -#ifndef DYNAMIC_BUFFERS -					if (! rx_in_place) -#endif -					{ +					if (!rx_in_place) {  						/* Receive buffer allocated, pass receive packet up */  						skb_copy_to_linear_data(skb, @@ -3460,10 +3454,6 @@ static void dfx_rcv_flush( DFX_board_t *bp )  		}  	} -#else -static inline void dfx_rcv_flush( DFX_board_t *bp ) -{ -}  #endif /* DYNAMIC_BUFFERS */  /* @@ -3579,7 +3569,7 @@ static void dfx_unregister(struct device *bdev)  {  	struct net_device *dev = dev_get_drvdata(bdev);  	DFX_board_t *bp = netdev_priv(dev); -	int dfx_bus_pci = DFX_BUS_PCI(bdev); +	int dfx_bus_pci = dev_is_pci(bdev);  	int dfx_bus_tc = DFX_BUS_TC(bdev);  	int dfx_use_mmio = DFX_MMIO || dfx_bus_tc;  	resource_size_t bar_start = 0;		/* pointer to port */ diff --git a/drivers/net/fddi/skfp/fplustm.c b/drivers/net/fddi/skfp/fplustm.c index a20ed1a9809..7d3779ae737 100644 --- a/drivers/net/fddi/skfp/fplustm.c +++ b/drivers/net/fddi/skfp/fplustm.c @@ -23,6 +23,7 @@  #include "h/smc.h"  #include "h/supern_2.h"  #include <linux/bitrev.h> +#include <linux/etherdevice.h>  #ifndef	lint  static const char ID_sccs[] = "@(#)fplustm.c	1.32 99/02/23 (C) SK " ; @@ -55,14 +56,14 @@ static	char cam_warning [] = "E_SMT_004: CAM still busy\n";  #define	DUMMY_READ()	smc->hw.mc_dummy = (u_short) inp(ADDR(B0_RAP)) -#define	CHECK_NPP() {	unsigned k = 10000 ;\ +#define	CHECK_NPP() {	unsigned int k = 10000 ;\  			while ((inpw(FM_A(FM_STMCHN)) & FM_SNPPND) && k) k--;\  			if (!k) { \  				SMT_PANIC(smc,SMT_E0130, SMT_E0130_MSG) ; \  			}	\  		} -#define	CHECK_CAM() {	unsigned k = 10 ;\ +#define	CHECK_CAM() {	unsigned int k = 10 ;\  			while (!(inpw(FM_A(FM_AFSTAT)) & FM_DONE) && k) k--;\  			if (!k) { \  				SMT_PANIC(smc,SMT_E0131, SMT_E0131_MSG) ; \ @@ -356,25 +357,25 @@ static	void set_formac_addr(struct s_smc *smc)  	long	t_requ = smc->mib.m[MAC0].fddiMACT_Req ;  	outpw(FM_A(FM_SAID),my_said) ;	/* set short address */ -	outpw(FM_A(FM_LAIL),(unsigned)((smc->hw.fddi_home_addr.a[4]<<8) + +	outpw(FM_A(FM_LAIL),(unsigned short)((smc->hw.fddi_home_addr.a[4]<<8) +  					smc->hw.fddi_home_addr.a[5])) ; -	outpw(FM_A(FM_LAIC),(unsigned)((smc->hw.fddi_home_addr.a[2]<<8) + +	outpw(FM_A(FM_LAIC),(unsigned short)((smc->hw.fddi_home_addr.a[2]<<8) +  					smc->hw.fddi_home_addr.a[3])) ; -	outpw(FM_A(FM_LAIM),(unsigned)((smc->hw.fddi_home_addr.a[0]<<8) + +	outpw(FM_A(FM_LAIM),(unsigned short)((smc->hw.fddi_home_addr.a[0]<<8) +  					smc->hw.fddi_home_addr.a[1])) ;  	outpw(FM_A(FM_SAGP),my_sagp) ;	/* set short group address */ -	outpw(FM_A(FM_LAGL),(unsigned)((smc->hw.fp.group_addr.a[4]<<8) + +	outpw(FM_A(FM_LAGL),(unsigned short)((smc->hw.fp.group_addr.a[4]<<8) +  					smc->hw.fp.group_addr.a[5])) ; -	outpw(FM_A(FM_LAGC),(unsigned)((smc->hw.fp.group_addr.a[2]<<8) + +	outpw(FM_A(FM_LAGC),(unsigned short)((smc->hw.fp.group_addr.a[2]<<8) +  					smc->hw.fp.group_addr.a[3])) ; -	outpw(FM_A(FM_LAGM),(unsigned)((smc->hw.fp.group_addr.a[0]<<8) + +	outpw(FM_A(FM_LAGM),(unsigned short)((smc->hw.fp.group_addr.a[0]<<8) +  					smc->hw.fp.group_addr.a[1])) ;  	/* set r_request regs. (MSW & LSW of TRT ) */ -	outpw(FM_A(FM_TREQ1),(unsigned)(t_requ>>16)) ; -	outpw(FM_A(FM_TREQ0),(unsigned)t_requ) ; +	outpw(FM_A(FM_TREQ1),(unsigned short)(t_requ>>16)) ; +	outpw(FM_A(FM_TREQ0),(unsigned short)t_requ) ;  }  static void set_int(char *p, int l) @@ -394,10 +395,10 @@ static void set_int(char *p, int l)   *	append 'end of chain' pointer   */  static void copy_tx_mac(struct s_smc *smc, u_long td, struct fddi_mac *mac, -			unsigned off, int len) +			unsigned int off, int len)  /* u_long td;		 transmit descriptor */  /* struct fddi_mac *mac; mac frame pointer */ -/* unsigned off;	 start address within buffer memory */ +/* unsigned int off;	 start address within buffer memory */  /* int len ;		 length of the frame including the FC */  {  	int	i ; @@ -453,7 +454,7 @@ static void directed_beacon(struct s_smc *smc)  	 */  	* (char *) a = (char) ((long)DBEACON_INFO<<24L) ;  	a[1] = 0 ; -	memcpy((char *)a+1,(char *) &smc->mib.m[MAC0].fddiMACUpstreamNbr,6) ; +	memcpy((char *)a+1, (char *) &smc->mib.m[MAC0].fddiMACUpstreamNbr, ETH_ALEN);  	CHECK_NPP() ;  	 /* set memory address reg for writes */ @@ -1082,7 +1083,7 @@ static struct s_fpmc* mac_get_mc_table(struct s_smc *smc,  				slot = tb ;  			continue ;  		} -		if (memcmp((char *)&tb->a,(char *)own,6)) +		if (!ether_addr_equal((char *)&tb->a, (char *)own))  			continue ;  		return tb;  	} diff --git a/drivers/net/fddi/skfp/h/smc.h b/drivers/net/fddi/skfp/h/smc.h index 3ca308b2821..bd1166bf8f6 100644 --- a/drivers/net/fddi/skfp/h/smc.h +++ b/drivers/net/fddi/skfp/h/smc.h @@ -469,20 +469,20 @@ struct s_smc {  extern const struct fddi_addr fddi_broadcast; -extern void all_selection_criteria(struct s_smc *smc); -extern void card_stop(struct s_smc *smc); -extern void init_board(struct s_smc *smc, u_char *mac_addr); -extern int init_fplus(struct s_smc *smc); -extern void init_plc(struct s_smc *smc); -extern int init_smt(struct s_smc *smc, u_char * mac_addr); -extern void mac1_irq(struct s_smc *smc, u_short stu, u_short stl); -extern void mac2_irq(struct s_smc *smc, u_short code_s2u, u_short code_s2l); -extern void mac3_irq(struct s_smc *smc, u_short code_s3u, u_short code_s3l); -extern int pcm_status_twisted(struct s_smc *smc); -extern void plc1_irq(struct s_smc *smc); -extern void plc2_irq(struct s_smc *smc); -extern void read_address(struct s_smc *smc, u_char * mac_addr); -extern void timer_irq(struct s_smc *smc); +void all_selection_criteria(struct s_smc *smc); +void card_stop(struct s_smc *smc); +void init_board(struct s_smc *smc, u_char *mac_addr); +int init_fplus(struct s_smc *smc); +void init_plc(struct s_smc *smc); +int init_smt(struct s_smc *smc, u_char *mac_addr); +void mac1_irq(struct s_smc *smc, u_short stu, u_short stl); +void mac2_irq(struct s_smc *smc, u_short code_s2u, u_short code_s2l); +void mac3_irq(struct s_smc *smc, u_short code_s3u, u_short code_s3l); +int pcm_status_twisted(struct s_smc *smc); +void plc1_irq(struct s_smc *smc); +void plc2_irq(struct s_smc *smc); +void read_address(struct s_smc *smc, u_char *mac_addr); +void timer_irq(struct s_smc *smc);  #endif	/* _SCMECM_ */ diff --git a/drivers/net/fddi/skfp/h/supern_2.h b/drivers/net/fddi/skfp/h/supern_2.h index 0b73690280f..4ee360d2dc6 100644 --- a/drivers/net/fddi/skfp/h/supern_2.h +++ b/drivers/net/fddi/skfp/h/supern_2.h @@ -92,33 +92,33 @@  union rx_descr {  	struct {  #ifdef	LITTLE_ENDIAN -	unsigned	rx_length :16 ;	/* frame length lower/upper byte */ -	unsigned	rx_erfbb  :2 ;	/* received frame byte boundary */ -	unsigned	rx_reserv2:2 ;	/* reserved */	 -	unsigned	rx_sfrmty :3 ;	/* frame type bits */ -	unsigned	rx_sadrrg :1 ;	/* DA == MA or broad-/multicast */ -	unsigned	rx_sfrmerr:1 ;	/* received frame not valid */ -	unsigned	rx_seac0  :1 ;	/* frame-copied  C-indicator */ -	unsigned	rx_seac1  :1 ;	/* address-match A-indicator */ -	unsigned	rx_seac2  :1 ;	/* frame-error   E-indicator */ -	unsigned	rx_ssrcrtg:1 ;	/* == 1 SA has MSB set */ -	unsigned	rx_reserv1:1 ;	/* reserved */	 -	unsigned	rx_msrabt :1 ;	/* memory status receive abort */ -	unsigned	rx_msvalid:1 ;	/* memory status valid */ +	unsigned int	rx_length :16 ;	/* frame length lower/upper byte */ +	unsigned int	rx_erfbb  :2 ;	/* received frame byte boundary */ +	unsigned int	rx_reserv2:2 ;	/* reserved */ +	unsigned int	rx_sfrmty :3 ;	/* frame type bits */ +	unsigned int	rx_sadrrg :1 ;	/* DA == MA or broad-/multicast */ +	unsigned int	rx_sfrmerr:1 ;	/* received frame not valid */ +	unsigned int	rx_seac0  :1 ;	/* frame-copied  C-indicator */ +	unsigned int	rx_seac1  :1 ;	/* address-match A-indicator */ +	unsigned int	rx_seac2  :1 ;	/* frame-error   E-indicator */ +	unsigned int	rx_ssrcrtg:1 ;	/* == 1 SA has MSB set */ +	unsigned int	rx_reserv1:1 ;	/* reserved */ +	unsigned int	rx_msrabt :1 ;	/* memory status receive abort */ +	unsigned int	rx_msvalid:1 ;	/* memory status valid */  #else -	unsigned	rx_msvalid:1 ;	/* memory status valid */ -	unsigned	rx_msrabt :1 ;	/* memory status receive abort */ -	unsigned	rx_reserv1:1 ;	/* reserved */	 -	unsigned	rx_ssrcrtg:1 ;	/* == 1 SA has MSB set */ -	unsigned	rx_seac2  :1 ;	/* frame-error   E-indicator */ -	unsigned	rx_seac1  :1 ;	/* address-match A-indicator */ -	unsigned	rx_seac0  :1 ;	/* frame-copied  C-indicator */ -	unsigned	rx_sfrmerr:1 ;	/* received frame not valid */ -	unsigned	rx_sadrrg :1 ;	/* DA == MA or broad-/multicast */ -	unsigned	rx_sfrmty :3 ;	/* frame type bits */ -	unsigned	rx_erfbb  :2 ;	/* received frame byte boundary */ -	unsigned	rx_reserv2:2 ;	/* reserved */	 -	unsigned	rx_length :16 ;	/* frame length lower/upper byte */ +	unsigned int	rx_msvalid:1 ;	/* memory status valid */ +	unsigned int	rx_msrabt :1 ;	/* memory status receive abort */ +	unsigned int	rx_reserv1:1 ;	/* reserved */ +	unsigned int	rx_ssrcrtg:1 ;	/* == 1 SA has MSB set */ +	unsigned int	rx_seac2  :1 ;	/* frame-error   E-indicator */ +	unsigned int	rx_seac1  :1 ;	/* address-match A-indicator */ +	unsigned int	rx_seac0  :1 ;	/* frame-copied  C-indicator */ +	unsigned int	rx_sfrmerr:1 ;	/* received frame not valid */ +	unsigned int	rx_sadrrg :1 ;	/* DA == MA or broad-/multicast */ +	unsigned int	rx_sfrmty :3 ;	/* frame type bits */ +	unsigned int	rx_erfbb  :2 ;	/* received frame byte boundary */ +	unsigned int	rx_reserv2:2 ;	/* reserved */ +	unsigned int	rx_length :16 ;	/* frame length lower/upper byte */  #endif  	} r ;  	long	i ; @@ -162,23 +162,23 @@ union rx_descr {  union tx_descr {  	struct {  #ifdef	LITTLE_ENDIAN -	unsigned	tx_length:16 ;	/* frame length lower/upper byte */ -	unsigned	tx_res	 :8 ;	/* reserved 	 (bit 16..23) */ -	unsigned	tx_xmtabt:1 ;	/* transmit abort */ -	unsigned	tx_nfcs  :1 ;	/* no frame check sequence */ -	unsigned	tx_xdone :1 ;	/* give up token */ -	unsigned	tx_rpxm  :2 ;	/* byte offset */ -	unsigned	tx_pat1  :2 ;	/* must be TXP1 */ -	unsigned	tx_more	 :1 ;	/* more frame in chain */ +	unsigned int	tx_length:16 ;	/* frame length lower/upper byte */ +	unsigned int	tx_res	 :8 ;	/* reserved 	 (bit 16..23) */ +	unsigned int	tx_xmtabt:1 ;	/* transmit abort */ +	unsigned int	tx_nfcs  :1 ;	/* no frame check sequence */ +	unsigned int	tx_xdone :1 ;	/* give up token */ +	unsigned int	tx_rpxm  :2 ;	/* byte offset */ +	unsigned int	tx_pat1  :2 ;	/* must be TXP1 */ +	unsigned int	tx_more	 :1 ;	/* more frame in chain */  #else -	unsigned	tx_more	 :1 ;	/* more frame in chain */ -	unsigned	tx_pat1  :2 ;	/* must be TXP1 */ -	unsigned	tx_rpxm  :2 ;	/* byte offset */ -	unsigned	tx_xdone :1 ;	/* give up token */ -	unsigned	tx_nfcs  :1 ;	/* no frame check sequence */ -	unsigned	tx_xmtabt:1 ;	/* transmit abort */ -	unsigned	tx_res	 :8 ;	/* reserved 	 (bit 16..23) */ -	unsigned	tx_length:16 ;	/* frame length lower/upper byte */ +	unsigned int	tx_more	 :1 ;	/* more frame in chain */ +	unsigned int	tx_pat1  :2 ;	/* must be TXP1 */ +	unsigned int	tx_rpxm  :2 ;	/* byte offset */ +	unsigned int	tx_xdone :1 ;	/* give up token */ +	unsigned int	tx_nfcs  :1 ;	/* no frame check sequence */ +	unsigned int	tx_xmtabt:1 ;	/* transmit abort */ +	unsigned int	tx_res	 :8 ;	/* reserved 	 (bit 16..23) */ +	unsigned int	tx_length:16 ;	/* frame length lower/upper byte */  #endif  	} t ;  	long	i ; @@ -202,13 +202,13 @@ union tx_descr {  union tx_pointer {  	struct t {  #ifdef	LITTLE_ENDIAN -	unsigned	tp_pointer:16 ;	/* pointer to tx_descr (low/high) */ -	unsigned	tp_res	  :8 ;	/* reserved 	 (bit 16..23) */ -	unsigned	tp_pattern:8 ;	/* fixed pattern (bit 24..31) */ +	unsigned int	tp_pointer:16 ;	/* pointer to tx_descr (low/high) */ +	unsigned int	tp_res	  :8 ;	/* reserved 	 (bit 16..23) */ +	unsigned int	tp_pattern:8 ;	/* fixed pattern (bit 24..31) */  #else -	unsigned	tp_pattern:8 ;	/* fixed pattern (bit 24..31) */ -	unsigned	tp_res	  :8 ;	/* reserved 	 (bit 16..23) */ -	unsigned	tp_pointer:16 ;	/* pointer to tx_descr (low/high) */ +	unsigned int	tp_pattern:8 ;	/* fixed pattern (bit 24..31) */ +	unsigned int	tp_res	  :8 ;	/* reserved 	 (bit 16..23) */ +	unsigned int	tp_pointer:16 ;	/* pointer to tx_descr (low/high) */  #endif  	} t ;  	long	i ; diff --git a/drivers/net/fddi/skfp/h/targetos.h b/drivers/net/fddi/skfp/h/targetos.h index 53bacc10716..355194251ff 100644 --- a/drivers/net/fddi/skfp/h/targetos.h +++ b/drivers/net/fddi/skfp/h/targetos.h @@ -48,7 +48,6 @@  #include <linux/fddidevice.h>  #include <linux/skbuff.h>  #include <linux/pci.h> -#include <linux/init.h>  // is redefined by linux, but we need our definition  #undef ADDR diff --git a/drivers/net/fddi/skfp/skfddi.c b/drivers/net/fddi/skfp/skfddi.c index f5d7305a578..d5f58121b2e 100644 --- a/drivers/net/fddi/skfp/skfddi.c +++ b/drivers/net/fddi/skfp/skfddi.c @@ -351,7 +351,6 @@ static void skfp_remove_one(struct pci_dev *pdev)  	free_netdev(p);  	pci_disable_device(pdev); -	pci_set_drvdata(pdev, NULL);  }  /* @@ -436,7 +435,7 @@ static  int skfp_driver_init(struct net_device *dev)  	}  	read_address(smc, NULL);  	pr_debug("HW-Addr: %pMF\n", smc->hw.fddi_canon_addr.a); -	memcpy(dev->dev_addr, smc->hw.fddi_canon_addr.a, 6); +	memcpy(dev->dev_addr, smc->hw.fddi_canon_addr.a, ETH_ALEN);  	smt_reset_defaults(smc, 0); @@ -503,7 +502,7 @@ static int skfp_open(struct net_device *dev)  	 *               address.  	 */  	read_address(smc, NULL); -	memcpy(dev->dev_addr, smc->hw.fddi_canon_addr.a, 6); +	memcpy(dev->dev_addr, smc->hw.fddi_canon_addr.a, ETH_ALEN);  	init_smt(smc, NULL);  	smt_online(smc, 1); @@ -1213,7 +1212,7 @@ static void CheckSourceAddress(unsigned char *frame, unsigned char *hw_addr)  	if ((unsigned short) frame[1 + 10] != 0)  		return;  	SRBit = frame[1 + 6] & 0x01; -	memcpy(&frame[1 + 6], hw_addr, 6); +	memcpy(&frame[1 + 6], hw_addr, ETH_ALEN);  	frame[8] |= SRBit;  }				// CheckSourceAddress diff --git a/drivers/net/fddi/skfp/smt.c b/drivers/net/fddi/skfp/smt.c index 08d94329c12..9edada85ed0 100644 --- a/drivers/net/fddi/skfp/smt.c +++ b/drivers/net/fddi/skfp/smt.c @@ -900,7 +900,7 @@ static void smt_send_rdf(struct s_smc *smc, SMbuf *rej, int fc, int reason,  	rdf->version.v_pad2 = 0 ;  	/* set P13 */ -	if ((unsigned) frame_len <= SMT_MAX_INFO_LEN - sizeof(*rdf) + +	if ((unsigned int) frame_len <= SMT_MAX_INFO_LEN - sizeof(*rdf) +  		2*sizeof(struct smt_header))  		len = frame_len ;  	else diff --git a/drivers/net/fddi/skfp/srf.c b/drivers/net/fddi/skfp/srf.c index f6f7baf9f27..cc27dea3414 100644 --- a/drivers/net/fddi/skfp/srf.c +++ b/drivers/net/fddi/skfp/srf.c @@ -73,7 +73,7 @@ void smt_init_evc(struct s_smc *smc)  {  	struct s_srf_evc	*evc ;  	const struct evc_init 	*init ; -	int			i ; +	unsigned int		i ;  	int			index ;  	int			offset ; @@ -84,7 +84,7 @@ void smt_init_evc(struct s_smc *smc)  	evc = smc->evcs ;  	init = evc_inits ; -	for (i = 0 ; (unsigned) i < MAX_INIT_EVC ; i++) { +	for (i = 0 ; i < MAX_INIT_EVC ; i++) {  		for (index = 0 ; index < init->n ; index++) {  			evc->evc_code = init->code ;  			evc->evc_para = init->para ; @@ -98,7 +98,7 @@ void smt_init_evc(struct s_smc *smc)  		init++ ;  	} -	if ((unsigned) (evc - smc->evcs) > MAX_EVCS) { +	if ((unsigned int) (evc - smc->evcs) > MAX_EVCS) {  		SMT_PANIC(smc,SMT_E0127, SMT_E0127_MSG) ;  	} @@ -139,7 +139,7 @@ void smt_init_evc(struct s_smc *smc)  		offset++ ;  	}  #ifdef	DEBUG -	for (i = 0, evc = smc->evcs ; (unsigned) i < MAX_EVCS ; i++, evc++) { +	for (i = 0, evc = smc->evcs ; i < MAX_EVCS ; i++, evc++) {  		if (SMT_IS_CONDITION(evc->evc_code)) {  			if (!evc->evc_cond_state) {  				SMT_PANIC(smc,SMT_E0128, SMT_E0128_MSG) ; @@ -160,10 +160,10 @@ void smt_init_evc(struct s_smc *smc)  static struct s_srf_evc *smt_get_evc(struct s_smc *smc, int code, int index)  { -	int			i ; +	unsigned int		i ;  	struct s_srf_evc	*evc ; -	for (i = 0, evc = smc->evcs ; (unsigned) i < MAX_EVCS ; i++, evc++) { +	for (i = 0, evc = smc->evcs ; i < MAX_EVCS ; i++, evc++) {  		if (evc->evc_code == code && evc->evc_index == index)  			return evc;  	} @@ -335,9 +335,9 @@ void smt_srf_event(struct s_smc *smc, int code, int index, int cond)  static void clear_all_rep(struct s_smc *smc)  {  	struct s_srf_evc	*evc ; -	int			i ; +	unsigned int		i ; -	for (i = 0, evc = smc->evcs ; (unsigned) i < MAX_EVCS ; i++, evc++) { +	for (i = 0, evc = smc->evcs ; i < MAX_EVCS ; i++, evc++) {  		evc->evc_rep_required = FALSE ;  		if (SMT_IS_CONDITION(evc->evc_code))  			*evc->evc_cond_state = FALSE ; @@ -348,10 +348,10 @@ static void clear_all_rep(struct s_smc *smc)  static void clear_reported(struct s_smc *smc)  {  	struct s_srf_evc	*evc ; -	int			i ; +	unsigned int		i ;  	smc->srf.any_report = FALSE ; -	for (i = 0, evc = smc->evcs ; (unsigned) i < MAX_EVCS ; i++, evc++) { +	for (i = 0, evc = smc->evcs ; i < MAX_EVCS ; i++, evc++) {  		if (SMT_IS_CONDITION(evc->evc_code)) {  			if (*evc->evc_cond_state == FALSE)  				evc->evc_rep_required = FALSE ; @@ -375,7 +375,7 @@ static void smt_send_srf(struct s_smc *smc)  	struct s_srf_evc	*evc ;  	SK_LOC_DECL(struct s_pcon,pcon) ;  	SMbuf			*mb ; -	int			i ; +	unsigned int		i ;  	static const struct fddi_addr SMT_SRF_DA = {  		{ 0x80, 0x01, 0x43, 0x00, 0x80, 0x08 } @@ -405,7 +405,7 @@ static void smt_send_srf(struct s_smc *smc)  	smt_add_para(smc,&pcon,(u_short) SMT_P1033,0,0) ;  	smt_add_para(smc,&pcon,(u_short) SMT_P1034,0,0) ; -	for (i = 0, evc = smc->evcs ; (unsigned) i < MAX_EVCS ; i++, evc++) { +	for (i = 0, evc = smc->evcs ; i < MAX_EVCS ; i++, evc++) {  		if (evc->evc_rep_required) {  			smt_add_para(smc,&pcon,evc->evc_para,  				(int)evc->evc_index,0) ;  | 
