Unverified Commit 065a651e authored by Cezary Rojewski's avatar Cezary Rojewski Committed by Mark Brown
Browse files

ASoC: Intel: avs: Simplify verification of parse_int_array() result 



The function return either success or an error code, no need to involve
'<' operator.

Reviewed-by: default avatarAmadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Signed-off-by: default avatarCezary Rojewski <cezary.rojewski@intel.com>
Link: https://patch.msgid.link/20250530141025.2942936-9-cezary.rojewski@intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent b3f3ca04

sound/soc/intel/avs/board_selection.c

+1 −1
Original line number Diff line number Diff line
@@ -548,7 +548,7 @@ static int avs_register_i2s_test_boards(struct avs_dev *adev) 
	u32 *array, num_elems;



	ret = parse_int_array(i2s_test, strlen(i2s_test), (int **)&array);

	if (ret < 0) {

	if (ret) {

		dev_err(adev->dev, "failed to parse i2s_test parameter\n");

		return ret;

	}

sound/soc/intel/avs/debugfs.c

+3 −3
Original line number Diff line number Diff line
@@ -144,7 +144,7 @@ static ssize_t probe_points_write(struct file *file, const char __user *from, si 
	int ret;



	ret = parse_int_array_user(from, count, (int **)&array);

	if (ret < 0)

	if (ret)

		return ret;



	num_elems = *array;
@@ -181,7 +181,7 @@ static ssize_t probe_points_disconnect_write(struct file *file, const char __use 
	int ret;



	ret = parse_int_array_user(from, count, (int **)&array);

	if (ret < 0)

	if (ret)

		return ret;



	num_elems = *array;
@@ -369,7 +369,7 @@ static ssize_t trace_control_write(struct file *file, const char __user *from, s 
	int ret;



	ret = parse_int_array_user(from, count, (int **)&array);

	if (ret < 0)

	if (ret)

		return ret;



	num_elems = *array;