diff options
author | Mitch Williams <mitch.a.williams@intel.com> | 2014-02-13 06:48:42 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-02-13 17:27:28 -0500 |
commit | a63fa1cdb667609cd9f99435ecbbcda61861b38f (patch) | |
tree | b5071a3f240253290b25fccceee590b003810564 | |
parent | af1a2a9c94d7b509ded794ef20b27f8fd36dee63 (diff) |
i40evf: clean up memsets
As politely pointed out by Dave Miller, calls to memset do not need a
void pointer cast. Additionally, it is preferred to use sizeof(*the
actual object) instead of sizeof(type).
Change-ID: Id6a02429b7040111531f3865ea03fbe619167cb3
Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
Signed-off-by: Kevin Scott <kevin.c.scott@intel.com>
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Tested-by: Kavindya Deegala <kavindya.s.deegala@intel.com>
Signed-off-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/intel/i40e/i40e_adminq.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.c b/drivers/net/ethernet/intel/i40e/i40e_adminq.c index a50e6b3479ae..ed3902bf249b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_adminq.c +++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.c | |||
@@ -647,9 +647,8 @@ static u16 i40e_clean_asq(struct i40e_hw *hw) | |||
647 | desc_cb = *desc; | 647 | desc_cb = *desc; |
648 | cb_func(hw, &desc_cb); | 648 | cb_func(hw, &desc_cb); |
649 | } | 649 | } |
650 | memset((void *)desc, 0, sizeof(struct i40e_aq_desc)); | 650 | memset(desc, 0, sizeof(*desc)); |
651 | memset((void *)details, 0, | 651 | memset(details, 0, sizeof(*details)); |
652 | sizeof(struct i40e_asq_cmd_details)); | ||
653 | ntc++; | 652 | ntc++; |
654 | if (ntc == asq->count) | 653 | if (ntc == asq->count) |
655 | ntc = 0; | 654 | ntc = 0; |