]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ASoC: Intel: avs: Simplify verification of parse_int_array() result
authorCezary Rojewski <cezary.rojewski@intel.com>
Fri, 30 May 2025 14:10:24 +0000 (16:10 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 3 Jun 2025 11:06:04 +0000 (12:06 +0100)
The function return either success or an error code, no need to involve
'<' operator.

Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://patch.msgid.link/20250530141025.2942936-9-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/avs/board_selection.c
sound/soc/intel/avs/debugfs.c

index 636315060eb47e1d2aa37e412bd79c9261b898a2..673ccf16202302d54592a0f5dfd80e9818709090 100644 (file)
@@ -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;
        }
index 0e826ca20619cafea23d73eb3c2af1fa929e6db8..c625cf879f170276a528938c1fa3a540c1f20dc3 100644 (file)
@@ -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;