Unverified Commit 31a54f78 authored by Charles Keepax's avatar Charles Keepax Committed by Mark Brown
Browse files

ASoC: intel: sof_sdw: Simplify find_codec_info_acpi 



Use a return rather than a break and an additional range check.

Reviewed-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: default avatarRanjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: default avatarRander Wang <rander.wang@intel.com>
Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20230731214257.444605-14-pierre-louis.bossart@linux.intel.com


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

sound/soc/intel/boards/sof_sdw.c

+3 −7
Original line number Diff line number Diff line
@@ -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 (!memcmp(codec_info_list[i].acpi_id, acpi_id, ACPI_ID_LEN))

			return i;



	if (i == ARRAY_SIZE(codec_info_list))

	return -EINVAL;



	return i;

}



/*