ice: switch: Simplify memory allocation
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 4 Sep 2022 14:18:02 +0000 (16:18 +0200)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Tue, 6 Sep 2022 20:55:47 +0000 (13:55 -0700)
'rbuf' is locale to the ice_get_initial_sw_cfg() function.
There is no point in using devm_kzalloc()/devm_kfree().

use kzalloc()/kfree() instead.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/ice/ice_switch.c

index 697feb8..eb6e19d 100644 (file)
@@ -2274,9 +2274,7 @@ int ice_get_initial_sw_cfg(struct ice_hw *hw)
        int status;
        u16 i;
 
-       rbuf = devm_kzalloc(ice_hw_to_dev(hw), ICE_SW_CFG_MAX_BUF_LEN,
-                           GFP_KERNEL);
-
+       rbuf = kzalloc(ICE_SW_CFG_MAX_BUF_LEN, GFP_KERNEL);
        if (!rbuf)
                return -ENOMEM;
 
@@ -2324,7 +2322,7 @@ int ice_get_initial_sw_cfg(struct ice_hw *hw)
                }
        } while (req_desc && !status);
 
-       devm_kfree(ice_hw_to_dev(hw), rbuf);
+       kfree(rbuf);
        return status;
 }