From: Charles Keepax Date: Mon, 31 Jul 2023 21:42:47 +0000 (-0500) Subject: ASoC: intel: sof_sdw: Simplify find_codec_info_acpi X-Git-Tag: v6.6.7~1940^2^2~161^2~10 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=31a54f78ad1d294432d63e9f72db08dc14cca432;p=platform%2Fkernel%2Flinux-starfive.git ASoC: intel: sof_sdw: Simplify find_codec_info_acpi Use a return rather than a break and an additional range check. Reviewed-by: Bard Liao Reviewed-by: Ranjani Sridharan Reviewed-by: Rander Wang Signed-off-by: Charles Keepax Signed-off-by: Pierre-Louis Bossart Link: https://lore.kernel.org/r/20230731214257.444605-14-pierre-louis.bossart@linux.intel.com Signed-off-by: Mark Brown --- diff --git a/sound/soc/intel/boards/sof_sdw.c b/sound/soc/intel/boards/sof_sdw.c index 6b55bcd..d3222f4 100644 --- a/sound/soc/intel/boards/sof_sdw.c +++ b/sound/soc/intel/boards/sof_sdw.c @@ -1003,14 +1003,10 @@ static inline int find_codec_info_acpi(const u8 *acpi_id) return -EINVAL; for (i = 0; i < ARRAY_SIZE(codec_info_list); i++) - if (!memcmp(codec_info_list[i].acpi_id, acpi_id, - ACPI_ID_LEN)) - break; - - if (i == ARRAY_SIZE(codec_info_list)) - return -EINVAL; + if (!memcmp(codec_info_list[i].acpi_id, acpi_id, ACPI_ID_LEN)) + return i; - return i; + return -EINVAL; } /*