clk: tegra: Use match_string() helper to simplify the code
authorYueHaibing <yuehaibing@huawei.com>
Sat, 9 Nov 2019 03:42:26 +0000 (11:42 +0800)
committerStephen Boyd <sboyd@kernel.org>
Wed, 13 Nov 2019 23:03:27 +0000 (15:03 -0800)
match_string() returns the array index of a matching string.
Use it instead of the open-coded implementation.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Link: https://lkml.kernel.org/r/20191109034226.21044-1-yuehaibing@huawei.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/tegra/clk-emc.c

index ea39caf3d762255e482102d646e261a113f96003..745f9faa98d8ef00e7bcc3483bfdf4584ba525d8 100644 (file)
@@ -403,20 +403,16 @@ static int load_one_timing_from_dt(struct tegra_clk_emc *tegra,
        }
 
        timing->parent_index = 0xff;
-       for (i = 0; i < ARRAY_SIZE(emc_parent_clk_names); i++) {
-               if (!strcmp(emc_parent_clk_names[i],
-                           __clk_get_name(timing->parent))) {
-                       timing->parent_index = i;
-                       break;
-               }
-       }
-       if (timing->parent_index == 0xff) {
+       i = match_string(emc_parent_clk_names, ARRAY_SIZE(emc_parent_clk_names),
+                        __clk_get_name(timing->parent));
+       if (i < 0) {
                pr_err("timing %pOF: %s is not a valid parent\n",
                       node, __clk_get_name(timing->parent));
                clk_put(timing->parent);
                return -EINVAL;
        }
 
+       timing->parent_index = i;
        return 0;
 }