i40e/i40evf: Big endian fixes for handling HMC

Fix HMC handling for big endian architectures.

Change-ID: Id8c46fc341815d47bfe0af8b819f0ab9a1e9e515
Signed-off-by: Paul M Stillwell Jr <paul.m.stillwell.jr@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

1 files changed, 20 insertions, 8 deletions

diff --git a/drivers/net/ethernet/intel/i40evf/i40e_lan_hmc.h b/drivers/net/ethernet/intel/i40evf/i40e_lan_hmc.h
index d6f762241537..a5d79877354c 100644
--- a/drivers/net/ethernet/intel/i40evf/i40e_lan_hmc.h
+++ b/drivers/net/ethernet/intel/i40evf/i40e_lan_hmc.h
@@ -32,16 +32,22 @@ struct i40e_hw;
 
 /* HMC element context information */
 
-/* Rx queue context data */
+/* Rx queue context data
+ *
+ * The sizes of the variables may be larger than needed due to crossing byte
+ * boundaries. If we do not have the width of the variable set to the correct
+ * size then we could end up shifting bits off the top of the variable when the
+ * variable is at the top of a byte and crosses over into the next byte.
+ */
 struct i40e_hmc_obj_rxq {
         u16 head;
-        u8  cpuid;
+        u16 cpuid; /* bigger than needed, see above for reason */
         u64 base;
         u16 qlen;
 #define I40E_RXQ_CTX_DBUFF_SHIFT 7
-        u8  dbuff;
+        u16 dbuff; /* bigger than needed, see above for reason */
 #define I40E_RXQ_CTX_HBUFF_SHIFT 6
-        u8  hbuff;
+        u16 hbuff; /* bigger than needed, see above for reason */
         u8  dtype;
         u8  dsize;
         u8  crcstrip;
@@ -50,16 +56,22 @@ struct i40e_hmc_obj_rxq {
         u8  hsplit_0;
         u8  hsplit_1;
         u8  showiv;
-        u16 rxmax;
+        u32 rxmax; /* bigger than needed, see above for reason */
         u8  tphrdesc_ena;
         u8  tphwdesc_ena;
         u8  tphdata_ena;
         u8  tphhead_ena;
-        u8  lrxqthresh;
+        u16 lrxqthresh; /* bigger than needed, see above for reason */
         u8  prefena;    /* NOTE: normally must be set to 1 at init */
 };
 
-/* Tx queue context data */
+/* Tx queue context data
+*
+* The sizes of the variables may be larger than needed due to crossing byte
+* boundaries. If we do not have the width of the variable set to the correct
+* size then we could end up shifting bits off the top of the variable when the
+* variable is at the top of a byte and crosses over into the next byte.
+*/
 struct i40e_hmc_obj_txq {
         u16 head;
         u8  new_context;
@@ -69,7 +81,7 @@ struct i40e_hmc_obj_txq {
         u8  fd_ena;
         u8  alt_vlan_ena;
         u16 thead_wb;
-        u16 cpuid;
+        u8  cpuid;
         u8  head_wb_ena;
         u16 qlen;
         u8  tphrdesc_ena;