diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2012-09-11 16:46:41 -0400 |
---|---|---|
committer | Ben Hutchings <bhutchings@solarflare.com> | 2012-10-01 20:58:40 -0400 |
commit | 0a6e5008a9df678b48f8d4e57601aa4270df6c14 (patch) | |
tree | 42b1fdd0cee106cbbca7ba0fa20503255a90b800 /drivers/net | |
parent | 9724a8504c875145f5a513bb8eca50671cee23b4 (diff) |
sfc: Fix MCDI structure field lookup
The least significant bit number (LBN) of a field within an MCDI
structure is counted from the start of the structure, not the
containing dword. In MCDI_ARRAY_FIELD() we need to mask it rather
than using the usual EFX_DWORD_FIELD() macro.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/sfc/mcdi.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/ethernet/sfc/mcdi.h b/drivers/net/ethernet/sfc/mcdi.h index dc25caaa3983..3ba2e5b5a9cc 100644 --- a/drivers/net/ethernet/sfc/mcdi.h +++ b/drivers/net/ethernet/sfc/mcdi.h | |||
@@ -113,11 +113,13 @@ extern void efx_mcdi_sensor_event(struct efx_nic *efx, efx_qword_t *ev); | |||
113 | #define MCDI_EVENT_FIELD(_ev, _field) \ | 113 | #define MCDI_EVENT_FIELD(_ev, _field) \ |
114 | EFX_QWORD_FIELD(_ev, MCDI_EVENT_ ## _field) | 114 | EFX_QWORD_FIELD(_ev, MCDI_EVENT_ ## _field) |
115 | #define MCDI_ARRAY_FIELD(_buf, _field1, _type, _index, _field2) \ | 115 | #define MCDI_ARRAY_FIELD(_buf, _field1, _type, _index, _field2) \ |
116 | EFX_DWORD_FIELD( \ | 116 | EFX_EXTRACT_DWORD( \ |
117 | *((efx_dword_t *) \ | 117 | *((efx_dword_t *) \ |
118 | (MCDI_ARRAY_PTR(_buf, _field1, _type, _index) + \ | 118 | (MCDI_ARRAY_PTR(_buf, _field1, _type, _index) + \ |
119 | (MC_CMD_ ## _type ## _TYPEDEF_ ## _field2 ## _OFST & ~3))), \ | 119 | (MC_CMD_ ## _type ## _TYPEDEF_ ## _field2 ## _OFST & ~3))), \ |
120 | MC_CMD_ ## _type ## _TYPEDEF_ ## _field2) | 120 | MC_CMD_ ## _type ## _TYPEDEF_ ## _field2 ## _LBN & 0x1f, \ |
121 | (MC_CMD_ ## _type ## _TYPEDEF_ ## _field2 ## _LBN & 0x1f) + \ | ||
122 | MC_CMD_ ## _type ## _TYPEDEF_ ## _field2 ## _WIDTH - 1) | ||
121 | 123 | ||
122 | extern void efx_mcdi_print_fwver(struct efx_nic *efx, char *buf, size_t len); | 124 | extern void efx_mcdi_print_fwver(struct efx_nic *efx, char *buf, size_t len); |
123 | extern int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating, | 125 | extern int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating, |