From: Thorsten Blum Date: Wed, 13 Aug 2025 10:34:16 +0000 (+0200) Subject: ALSA: hda: Improve local variable data type in snd_hda_get_num_devices() X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=113e9a284d4372322da90c05f1dc36e0a52e5fca;p=users%2Fhch%2Fmisc.git ALSA: hda: Improve local variable data type in snd_hda_get_num_devices() Use 'int' instead of 'unsigned int' because the local variable 'parm' can be negative. While an unsigned integer is harmless in practice due to the implicit type conversion, it's safer and more idiomatic to use a signed integer to properly check for -1. Signed-off-by: Thorsten Blum Link: https://patch.msgid.link/20250813103418.164110-2-thorsten.blum@linux.dev Signed-off-by: Takashi Iwai --- diff --git a/sound/hda/common/codec.c b/sound/hda/common/codec.c index eb268d442201..cab479111603 100644 --- a/sound/hda/common/codec.c +++ b/sound/hda/common/codec.c @@ -300,7 +300,7 @@ EXPORT_SYMBOL_GPL(snd_hda_get_conn_index); unsigned int snd_hda_get_num_devices(struct hda_codec *codec, hda_nid_t nid) { unsigned int wcaps = get_wcaps(codec, nid); - unsigned int parm; + int parm; if (!codec->dp_mst || !(wcaps & AC_WCAP_DIGITAL) || get_wcaps_type(wcaps) != AC_WID_PIN)