Commit 993fb90c authored by Adrian Bunk's avatar Adrian Bunk Committed by David S. Miller
Browse files

drivers/net/netxen/: cleanups 



This patch contains the following cleanups:
- static functions in .c files shouldn't be marked inline
- make needlessly global code static
- #if 0 unused code

Signed-off-by: default avatarAdrian Bunk <bunk@kernel.org>
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
parent 3957d63d

drivers/net/netxen/netxen_nic.h

+0 −14
Original line number Diff line number Diff line
@@ -1018,14 +1018,8 @@ int netxen_niu_xgbe_enable_phy_interrupts(struct netxen_adapter *adapter); 
int netxen_niu_gbe_enable_phy_interrupts(struct netxen_adapter *adapter);

int netxen_niu_xgbe_disable_phy_interrupts(struct netxen_adapter *adapter);

int netxen_niu_gbe_disable_phy_interrupts(struct netxen_adapter *adapter);

int netxen_niu_xgbe_clear_phy_interrupts(struct netxen_adapter *adapter);

int netxen_niu_gbe_clear_phy_interrupts(struct netxen_adapter *adapter);

void netxen_nic_xgbe_handle_phy_intr(struct netxen_adapter *adapter);

void netxen_nic_gbe_handle_phy_intr(struct netxen_adapter *adapter);

void netxen_niu_gbe_set_mii_mode(struct netxen_adapter *adapter, int port,

				 long enable);

void netxen_niu_gbe_set_gmii_mode(struct netxen_adapter *adapter, int port,

				  long enable);

int netxen_niu_gbe_phy_read(struct netxen_adapter *adapter, long reg,

			    __u32 * readval);

int netxen_niu_gbe_phy_write(struct netxen_adapter *adapter,
@@ -1048,7 +1042,6 @@ int netxen_nic_hw_write_wx(struct netxen_adapter *adapter, u64 off, void *data, 
			   int len);

void netxen_crb_writelit_adapter(struct netxen_adapter *adapter,

				 unsigned long off, int data);

int netxen_nic_erase_pxe(struct netxen_adapter *adapter);



/* Functions from netxen_nic_init.c */

void netxen_free_adapter_offload(struct netxen_adapter *adapter);
@@ -1067,15 +1060,10 @@ int netxen_flash_erase_secondary(struct netxen_adapter *adapter); 
int netxen_flash_erase_primary(struct netxen_adapter *adapter);

void netxen_halt_pegs(struct netxen_adapter *adapter);



int netxen_rom_fast_write(struct netxen_adapter *adapter, int addr, int data);

int netxen_rom_se(struct netxen_adapter *adapter, int addr);

int netxen_do_rom_se(struct netxen_adapter *adapter, int addr);



/* Functions from netxen_nic_isr.c */

int netxen_nic_link_ok(struct netxen_adapter *adapter);

void netxen_nic_isr_other(struct netxen_adapter *adapter);

void netxen_indicate_link_status(struct netxen_adapter *adapter, u32 link);

void netxen_handle_port_int(struct netxen_adapter *adapter, u32 enable);

void netxen_initialize_adapter_sw(struct netxen_adapter *adapter);

void netxen_initialize_adapter_hw(struct netxen_adapter *adapter);

void *netxen_alloc(struct pci_dev *pdev, size_t sz, dma_addr_t * ptr,
@@ -1092,8 +1080,6 @@ int netxen_nic_tx_has_work(struct netxen_adapter *adapter); 
void netxen_watchdog_task(struct work_struct *work);

void netxen_post_rx_buffers(struct netxen_adapter *adapter, u32 ctx,

			    u32 ringid);

void netxen_post_rx_buffers_nodb(struct netxen_adapter *adapter, u32 ctx,

				 u32 ringid);

int netxen_process_cmd_ring(unsigned long data);

u32 netxen_process_rcv_ring(struct netxen_adapter *adapter, int ctx, int max);

void netxen_nic_set_multi(struct net_device *netdev);

drivers/net/netxen/netxen_nic_hw.c

+11 −8
Original line number Diff line number Diff line
@@ -33,7 +33,6 @@ 


#include "netxen_nic.h"

#include "netxen_nic_hw.h"

#define DEFINE_GLOBAL_RECV_CRB

#include "netxen_nic_phan_reg.h"
@@ -244,12 +243,15 @@ struct netxen_recv_crb recv_crb_registers[] = { 
	},

};



u64 ctx_addr_sig_regs[][3] = {

static u64 ctx_addr_sig_regs[][3] = {

	{NETXEN_NIC_REG(0x188), NETXEN_NIC_REG(0x18c), NETXEN_NIC_REG(0x1c0)},

	{NETXEN_NIC_REG(0x190), NETXEN_NIC_REG(0x194), NETXEN_NIC_REG(0x1c4)},

	{NETXEN_NIC_REG(0x198), NETXEN_NIC_REG(0x19c), NETXEN_NIC_REG(0x1c8)},

	{NETXEN_NIC_REG(0x1a0), NETXEN_NIC_REG(0x1a4), NETXEN_NIC_REG(0x1cc)}

};

#define CRB_CTX_ADDR_REG_LO(FUNC_ID)		(ctx_addr_sig_regs[FUNC_ID][0])

#define CRB_CTX_ADDR_REG_HI(FUNC_ID)		(ctx_addr_sig_regs[FUNC_ID][2])

#define CRB_CTX_SIGNATURE_REG(FUNC_ID)		(ctx_addr_sig_regs[FUNC_ID][1])





/*  PCI Windowing for DDR regions.  */
@@ -279,7 +281,7 @@ u64 ctx_addr_sig_regs[][3] = { 


#define NETXEN_NIC_WINDOW_MARGIN 0x100000



unsigned long netxen_nic_pci_set_window(struct netxen_adapter *adapter,

static unsigned long netxen_nic_pci_set_window(struct netxen_adapter *adapter,

					       unsigned long long addr);

void netxen_free_hw_resources(struct netxen_adapter *adapter);
@@ -886,10 +888,9 @@ void netxen_nic_read_w0(struct netxen_adapter *adapter, u32 index, u32 * value) 
	netxen_nic_pci_change_crbwindow(adapter, 1);

}



int netxen_pci_set_window_warning_count = 0;

static int netxen_pci_set_window_warning_count = 0;



unsigned long

netxen_nic_pci_set_window(struct netxen_adapter *adapter,

static  unsigned long netxen_nic_pci_set_window(struct netxen_adapter *adapter,

						unsigned long long addr)

{

	static int ddr_mn_window = -1;
@@ -952,6 +953,7 @@ netxen_nic_pci_set_window(struct netxen_adapter *adapter, 
	return addr;

}



#if 0

int

netxen_nic_erase_pxe(struct netxen_adapter *adapter)

{
@@ -962,6 +964,7 @@ netxen_nic_erase_pxe(struct netxen_adapter *adapter) 
	}

	return 0;

}

#endif  /*  0  */



int netxen_nic_get_board_info(struct netxen_adapter *adapter)

{

drivers/net/netxen/netxen_nic_hw.h

+2 −6
Original line number Diff line number Diff line
@@ -520,15 +520,11 @@ int netxen_niu_set_promiscuous_mode(struct netxen_adapter *adapter, 
int netxen_niu_xg_set_promiscuous_mode(struct netxen_adapter *adapter,

				       netxen_niu_prom_mode_t mode);



/* get/set the MAC address for a given MAC */

int netxen_niu_macaddr_get(struct netxen_adapter *adapter,

			   netxen_ethernet_macaddr_t * addr);

/* set the MAC address for a given MAC */

int netxen_niu_macaddr_set(struct netxen_adapter *adapter,

			   netxen_ethernet_macaddr_t addr);



/* XG versons */

int netxen_niu_xg_macaddr_get(struct netxen_adapter *adapter,

			      netxen_ethernet_macaddr_t * addr);

/* XG version */

int netxen_niu_xg_macaddr_set(struct netxen_adapter *adapter,

			      netxen_ethernet_macaddr_t addr);

drivers/net/netxen/netxen_nic_init.c

+37 −33
Original line number Diff line number Diff line
@@ -54,13 +54,17 @@ static unsigned int crb_addr_xform[NETXEN_MAX_CRB_XFORM]; 


#define NETXEN_NIC_XDMA_RESET 0x8000ff



static inline void

netxen_nic_locked_write_reg(struct netxen_adapter *adapter,

static void netxen_post_rx_buffers_nodb(struct netxen_adapter *adapter,

					uint32_t ctx, uint32_t ringid);



#if 0

static void netxen_nic_locked_write_reg(struct netxen_adapter *adapter,

					unsigned long off, int *data)

{

	void __iomem *addr = pci_base_offset(adapter, off);

	writel(*data, addr);

}

#endif  /*  0  */



static void crb_addr_transform_setup(void)

{
@@ -255,7 +259,7 @@ void netxen_initialize_adapter_ops(struct netxen_adapter *adapter) 
 * netxen_decode_crb_addr(0 - utility to translate from internal Phantom CRB

 * address to external PCI CRB address.

 */

u32 netxen_decode_crb_addr(u32 addr)

static u32 netxen_decode_crb_addr(u32 addr)

{

	int i;

	u32 base_addr, offset, pci_base;
@@ -282,7 +286,7 @@ static long rom_max_timeout = 100; 
static long rom_lock_timeout = 10000;

static long rom_write_timeout = 700;



static inline int rom_lock(struct netxen_adapter *adapter)

static int rom_lock(struct netxen_adapter *adapter)

{

	int iter;

	u32 done = 0;
@@ -312,7 +316,7 @@ static inline int rom_lock(struct netxen_adapter *adapter) 
	return 0;

}



int netxen_wait_rom_done(struct netxen_adapter *adapter)

static int netxen_wait_rom_done(struct netxen_adapter *adapter)

{

	long timeout = 0;

	long done = 0;
@@ -329,7 +333,7 @@ int netxen_wait_rom_done(struct netxen_adapter *adapter) 
	return 0;

}



static inline int netxen_rom_wren(struct netxen_adapter *adapter)

static int netxen_rom_wren(struct netxen_adapter *adapter)

{

	/* Set write enable latch in ROM status register */

	netxen_nic_reg_write(adapter, NETXEN_ROMUSB_ROM_ABYTE_CNT, 0);
@@ -341,7 +345,7 @@ static inline int netxen_rom_wren(struct netxen_adapter *adapter) 
	return 0;

}



static inline unsigned int netxen_rdcrbreg(struct netxen_adapter *adapter,

static unsigned int netxen_rdcrbreg(struct netxen_adapter *adapter,

				    unsigned int addr)

{

	unsigned int data = 0xdeaddead;
@@ -349,7 +353,7 @@ static inline unsigned int netxen_rdcrbreg(struct netxen_adapter *adapter, 
	return data;

}



static inline int netxen_do_rom_rdsr(struct netxen_adapter *adapter)

static int netxen_do_rom_rdsr(struct netxen_adapter *adapter)

{

	netxen_nic_reg_write(adapter, NETXEN_ROMUSB_ROM_INSTR_OPCODE,

			     M25P_INSTR_RDSR);
@@ -359,7 +363,7 @@ static inline int netxen_do_rom_rdsr(struct netxen_adapter *adapter) 
	return netxen_rdcrbreg(adapter, NETXEN_ROMUSB_ROM_RDATA);

}



static inline void netxen_rom_unlock(struct netxen_adapter *adapter)

static void netxen_rom_unlock(struct netxen_adapter *adapter)

{

	u32 val;
@@ -368,7 +372,7 @@ static inline void netxen_rom_unlock(struct netxen_adapter *adapter) 


}



int netxen_rom_wip_poll(struct netxen_adapter *adapter)

static int netxen_rom_wip_poll(struct netxen_adapter *adapter)

{

	long timeout = 0;

	long wip = 1;
@@ -385,7 +389,7 @@ int netxen_rom_wip_poll(struct netxen_adapter *adapter) 
	return 0;

}



static inline int do_rom_fast_write(struct netxen_adapter *adapter, int addr,

static int do_rom_fast_write(struct netxen_adapter *adapter, int addr,

			     int data)

{

	if (netxen_rom_wren(adapter)) {
@@ -404,8 +408,8 @@ static inline int do_rom_fast_write(struct netxen_adapter *adapter, int addr, 
	return netxen_rom_wip_poll(adapter);

}



static inline int

do_rom_fast_read(struct netxen_adapter *adapter, int addr, int *valp)

static int do_rom_fast_read(struct netxen_adapter *adapter,

			    int addr, int *valp)

{

	cond_resched();
@@ -427,8 +431,7 @@ do_rom_fast_read(struct netxen_adapter *adapter, int addr, int *valp) 
	return 0;

}



static inline int 

do_rom_fast_read_words(struct netxen_adapter *adapter, int addr,

static int do_rom_fast_read_words(struct netxen_adapter *adapter, int addr,

				  u8 *bytes, size_t size)

{

	int addridx;
@@ -473,6 +476,7 @@ int netxen_rom_fast_read(struct netxen_adapter *adapter, int addr, int *valp) 
	return ret;

}



#if 0

int netxen_rom_fast_write(struct netxen_adapter *adapter, int addr, int data)

{

	int ret = 0;
@@ -484,8 +488,9 @@ int netxen_rom_fast_write(struct netxen_adapter *adapter, int addr, int data) 
	netxen_rom_unlock(adapter);

	return ret;

}

#endif  /*  0  */



static inline int do_rom_fast_write_words(struct netxen_adapter *adapter, 

static int do_rom_fast_write_words(struct netxen_adapter *adapter, 

				   int addr, u8 *bytes, size_t size)

{

	int addridx = addr;
@@ -548,7 +553,7 @@ int netxen_rom_fast_write_words(struct netxen_adapter *adapter, int addr, 
	return ret;

}



int netxen_rom_wrsr(struct netxen_adapter *adapter, int data)

static int netxen_rom_wrsr(struct netxen_adapter *adapter, int data)

{

	int ret;
@@ -567,7 +572,7 @@ int netxen_rom_wrsr(struct netxen_adapter *adapter, int data) 
	return netxen_rom_wip_poll(adapter);

}



int netxen_rom_rdsr(struct netxen_adapter *adapter)

static int netxen_rom_rdsr(struct netxen_adapter *adapter)

{

	int ret;
@@ -632,7 +637,7 @@ int netxen_backup_crbinit(struct netxen_adapter *adapter) 
	return ret;

}



int netxen_do_rom_se(struct netxen_adapter *adapter, int addr)

static int netxen_do_rom_se(struct netxen_adapter *adapter, int addr)

{

	netxen_rom_wren(adapter);

	netxen_nic_reg_write(adapter, NETXEN_ROMUSB_ROM_ADDRESS, addr);
@@ -646,7 +651,7 @@ int netxen_do_rom_se(struct netxen_adapter *adapter, int addr) 
	return netxen_rom_wip_poll(adapter);

}



void check_erased_flash(struct netxen_adapter *adapter, int addr)

static void check_erased_flash(struct netxen_adapter *adapter, int addr)

{

	int i;

	int val;
@@ -682,8 +687,8 @@ int netxen_rom_se(struct netxen_adapter *adapter, int addr) 
	return ret;

}



int

netxen_flash_erase_sections(struct netxen_adapter *adapter, int start, int end)

static int netxen_flash_erase_sections(struct netxen_adapter *adapter,

				       int start, int end)

{

	int ret = FLASH_SUCCESS;

	int i;
@@ -990,7 +995,7 @@ int netxen_nic_rx_has_work(struct netxen_adapter *adapter) 
	return 0;

}



static inline int netxen_nic_check_temp(struct netxen_adapter *adapter)

static int netxen_nic_check_temp(struct netxen_adapter *adapter)

{

	struct net_device *netdev = adapter->netdev;

	uint32_t temp, temp_state, temp_val;
@@ -1064,8 +1069,7 @@ void netxen_watchdog_task(struct work_struct *work) 
 * and if the number of receives exceeds RX_BUFFERS_REFILL, then we

 * invoke the routine to send more rx buffers to the Phantom...

 */

void

netxen_process_rcv(struct netxen_adapter *adapter, int ctxid,

static void netxen_process_rcv(struct netxen_adapter *adapter, int ctxid,

			       struct status_desc *desc)

{

	struct pci_dev *pdev = adapter->pdev;
@@ -1458,8 +1462,8 @@ void netxen_post_rx_buffers(struct netxen_adapter *adapter, u32 ctx, u32 ringid) 
	}

}



void netxen_post_rx_buffers_nodb(struct netxen_adapter *adapter, uint32_t ctx,

				 uint32_t ringid)

static void netxen_post_rx_buffers_nodb(struct netxen_adapter *adapter,

					uint32_t ctx, uint32_t ringid)

{

	struct pci_dev *pdev = adapter->ahw.pdev;

	struct sk_buff *skb;

drivers/net/netxen/netxen_nic_isr.c

+5 −2
Original line number Diff line number Diff line
@@ -66,7 +66,8 @@ struct net_device_stats *netxen_nic_get_stats(struct net_device *netdev) 
	return stats;

}



void netxen_indicate_link_status(struct netxen_adapter *adapter, u32 link)

static void netxen_indicate_link_status(struct netxen_adapter *adapter,

					u32 link)

{

	struct net_device *netdev = adapter->netdev;
@@ -76,6 +77,7 @@ void netxen_indicate_link_status(struct netxen_adapter *adapter, u32 link) 
		netif_carrier_off(netdev);

}



#if 0

void netxen_handle_port_int(struct netxen_adapter *adapter, u32 enable)

{

	__u32 int_src;
@@ -134,8 +136,9 @@ void netxen_handle_port_int(struct netxen_adapter *adapter, u32 enable) 
	if (adapter->enable_phy_interrupts)

		adapter->enable_phy_interrupts(adapter);

}

#endif  /*  0  */



void netxen_nic_isr_other(struct netxen_adapter *adapter)

static void netxen_nic_isr_other(struct netxen_adapter *adapter)

{

	int portno = adapter->portnum;

	u32 val, linkup, qg_linksup;