Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 06380db6 authored by Mark Rustad's avatar Mark Rustad Committed by Jeff Kirsher
Browse files

ixgbevf: Use static inlines instead of macros 



Kernel coding standard prefers static inline functions instead
of macros, so use them for register accessors. This is to prepare
for adding LER, Live Error Recovery, checks to those accessors.

Signed-off-by: default avatarMark Rustad <mark.d.rustad@intel.com>
Tested-by: default avatarPhil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 19458bd4

drivers/net/ethernet/intel/ixgbevf/ixgbevf.h

+5 −0
Original line number Diff line number Diff line
@@ -315,6 +315,11 @@ static inline u16 ixgbevf_desc_unused(struct ixgbevf_ring *ring) 
	return ((ntc > ntu) ? 0 : ring->count) + ntc - ntu - 1;

}



static inline void ixgbevf_write_tail(struct ixgbevf_ring *ring, u32 value)

{

	writel(value, ring->tail);

}



#define IXGBEVF_RX_DESC(R, i)	    \

	(&(((union ixgbe_adv_rx_desc *)((R)->desc))[i]))

#define IXGBEVF_TX_DESC(R, i)	    \

drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c

+2 −2
Original line number Diff line number Diff line
@@ -111,7 +111,7 @@ static inline void ixgbevf_release_rx_desc(struct ixgbevf_ring *rx_ring, 
	 * such as IA-64).

	 */

	wmb();

	writel(val, rx_ring->tail);

	ixgbevf_write_tail(rx_ring, val);

}



/**
@@ -3060,7 +3060,7 @@ static void ixgbevf_tx_map(struct ixgbevf_ring *tx_ring, 
	tx_ring->next_to_use = i;



	/* notify HW of packet */

	writel(i, tx_ring->tail);

	ixgbevf_write_tail(tx_ring, i);



	return;

dma_error:

drivers/net/ethernet/intel/ixgbevf/regs.h

+1 −11
Original line number Diff line number Diff line 
/*******************************************************************************



  Intel 82599 Virtual Function driver

  Copyright(c) 1999 - 2012 Intel Corporation.

  Copyright(c) 1999 - 2014 Intel Corporation.



  This program is free software; you can redistribute it and/or modify it

  under the terms and conditions of the GNU General Public License,
@@ -70,16 +70,6 @@ 
#define IXGBE_VFGOTC_MSB       0x02024

#define IXGBE_VFMPRC           0x01034



#define IXGBE_WRITE_REG(a, reg, value) writel((value), ((a)->hw_addr + (reg)))



#define IXGBE_READ_REG(a, reg) readl((a)->hw_addr + (reg))



#define IXGBE_WRITE_REG_ARRAY(a, reg, offset, value) ( \

    writel((value), ((a)->hw_addr + (reg) + ((offset) << 2))))



#define IXGBE_READ_REG_ARRAY(a, reg, offset) ( \

    readl((a)->hw_addr + (reg) + ((offset) << 2)))



#define IXGBE_WRITE_FLUSH(a) (IXGBE_READ_REG(a, IXGBE_VFSTATUS))



#endif /* _IXGBEVF_REGS_H_ */

drivers/net/ethernet/intel/ixgbevf/vf.h

+27 −1
Original line number Diff line number Diff line 
/*******************************************************************************



  Intel 82599 Virtual Function driver

  Copyright(c) 1999 - 2012 Intel Corporation.

  Copyright(c) 1999 - 2014 Intel Corporation.



  This program is free software; you can redistribute it and/or modify it

  under the terms and conditions of the GNU General Public License,
@@ -172,6 +172,32 @@ struct ixgbevf_info { 
	const struct ixgbe_mac_operations *mac_ops;

};



static inline void ixgbe_write_reg(struct ixgbe_hw *hw, u32 reg, u32 value)

{

	writel(value, hw->hw_addr + reg);

}

#define IXGBE_WRITE_REG(h, r, v) ixgbe_write_reg(h, r, v)



static inline u32 ixgbe_read_reg(struct ixgbe_hw *hw, u32 reg)

{

	return readl(hw->hw_addr + reg);

}

#define IXGBE_READ_REG(h, r) ixgbe_read_reg(h, r)



static inline void ixgbe_write_reg_array(struct ixgbe_hw *hw, u32 reg,

					  u32 offset, u32 value)

{

	ixgbe_write_reg(hw, reg + (offset << 2), value);

}

#define IXGBE_WRITE_REG_ARRAY(h, r, o, v) ixgbe_write_reg_array(h, r, o, v)



static inline u32 ixgbe_read_reg_array(struct ixgbe_hw *hw, u32 reg,

					u32 offset)

{

	return ixgbe_read_reg(hw, reg + (offset << 2));

}

#define IXGBE_READ_REG_ARRAY(h, r, o) ixgbe_read_reg_array(h, r, o)



void ixgbevf_rlpml_set_vf(struct ixgbe_hw *hw, u16 max_size);

int ixgbevf_negotiate_api_version(struct ixgbe_hw *hw, int api);

int ixgbevf_get_queues(struct ixgbe_hw *hw, unsigned int *num_tcs,