Commit 1b9e740d authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Tony Nguyen
Browse files

ice: switch: Simplify memory allocation 



'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: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: default avatarMichal Swiatkowski <michal.swiatkowski@linux.intel.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 1bd50f2d

drivers/net/ethernet/intel/ice/ice_switch.c

+2 −4
Original line number Diff line number Diff line
@@ -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;

}