tree 8226a4045bcc1329be55866dcd54b2b950797522
parent 3f0d23e849b33cb83cbd1f3ce9b7d23ad3863841
author Charles Keepax <ckeepax@opensource.cirrus.com> 1620652428 +0100
committer Mark Brown <broonie@kernel.org> 1620664066 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmCZXwIACgkQJNaLcl1U
 h9Bqvwf+IeGN7TFa7LVuKshqoJ+Q2a5+DVdi2LZUjaFg56/6wheZhYBsAi5tRgll
 DZyn9no/+YvTm24WLhl26U1wYvSgUDRHshX0iVIDIhBgx+5ceoDuyOgGn2yKq0fi
 c1N/WCmhbhmcY5U4F6BHucmcjuAudMkveh8JdfOZt2mJE3WGgZPCsmXxC1H9ag/1
 iDRe1M4W5g3GGe+6juS0FWzrsoWoyr01dKvcjTumlBn4kw4LznbNmhcFS4bsLSNv
 TNWen0bAnuGObbxFTS9EBrA6//OW4yx/dKcvIQp+h6XKWtEwTg1gghygOOe20EuG
 QMxG577n25h8XAPOxUwzFw8VW6lNGg==
 =Rwa8
 -----END PGP SIGNATURE-----

ASoC: cirrus: Add helper function for reading the device ID

Many of the older Cirrus devices share very similar code for reading the
device ID, and frequently this code is generating cppcheck warnings such
as:

sound/soc/codecs/cs42l42.c:1886:6: style: Variable 'ret' is reassigned
a value before the old one has been used. [redundantAssignment]
 ret = regmap_read(cs42l42->regmap, CS42L42_DEVID_CD, &reg);

Add a small helper function that older Cirrus devices can use to read
the device ID, which should help correct these issues.

Reported-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210510131357.17170-2-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>