sound/hda/codecs/generic.c:198 parse_user_hints() warn: assigning (-2) to unsigned variable 'spec->mixer_nid'
kernel test robot <[email protected]>
| Newsgroups | dev.linux.lists.oe-kbuild |
|---|---|
| Message-ID | <[email protected]> |
BCC: [email protected] CC: [email protected] CC: [email protected] TO: Takashi Iwai <[email protected]> CC: Richard Fitzgerald <[email protected]> Hi Takashi, FYI, the error/warning was bisected to this commit, please ignore it if it's irrelevant. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0e35b9b6ec0ffcc5e23cbdec09f5c622ad532b53 commit: 6014e9021b28e634935c776c0271b5cbcabdc5d6 ALSA: hda: Move codec drivers into sound/hda/codecs directory date: 12 months ago :::::: branch date: 3 days ago :::::: commit date: 12 months ago config: hexagon-randconfig-r072-20260710 (https://download.01.org/0day-ci/archive/20260710/[email protected]/config) compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18) smatch: v0.5.0-9185-gbcc58b9c If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Fixes: 6014e9021b28 ("ALSA: hda: Move codec drivers into sound/hda/codecs directory") | Reported-by: kernel test robot <[email protected]> | Reported-by: Dan Carpenter <[email protected]> | Closes: https://lore.kernel.org/r/[email protected]/ smatch warnings: sound/hda/codecs/generic.c:198 parse_user_hints() warn: assigning (-2) to unsigned variable 'spec->mixer_nid' vim +198 sound/hda/codecs/generic.c ^1da177e4c3f41 sound/pci/hda/hda_generic.c Linus Torvalds 2005-04-16 101 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 102 /* 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 103 * store user hints 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 104 */ 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 105 static void parse_user_hints(struct hda_codec *codec) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 106 { 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 107 struct hda_gen_spec *spec = codec->spec; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 108 int val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 109 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 110 val = snd_hda_get_bool_hint(codec, "jack_detect"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 111 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 112 codec->no_jack_detect = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 113 val = snd_hda_get_bool_hint(codec, "inv_jack_detect"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 114 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 115 codec->inv_jack_detect = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 116 val = snd_hda_get_bool_hint(codec, "trigger_sense"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 117 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 118 codec->no_trigger_sense = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 119 val = snd_hda_get_bool_hint(codec, "inv_eapd"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 120 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 121 codec->inv_eapd = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 122 val = snd_hda_get_bool_hint(codec, "pcm_format_first"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 123 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 124 codec->pcm_format_first = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 125 val = snd_hda_get_bool_hint(codec, "sticky_stream"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 126 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 127 codec->no_sticky_stream = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 128 val = snd_hda_get_bool_hint(codec, "spdif_status_reset"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 129 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 130 codec->spdif_status_reset = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 131 val = snd_hda_get_bool_hint(codec, "pin_amp_workaround"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 132 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 133 codec->pin_amp_workaround = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 134 val = snd_hda_get_bool_hint(codec, "single_adc_amp"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 135 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 136 codec->single_adc_amp = !!val; 967b1307b69b8a sound/pci/hda/hda_generic.c Takashi Iwai 2015-03-20 137 val = snd_hda_get_bool_hint(codec, "power_save_node"); e6feb5d08509be sound/pci/hda/hda_generic.c Takashi Iwai 2015-03-16 138 if (val >= 0) 967b1307b69b8a sound/pci/hda/hda_generic.c Takashi Iwai 2015-03-20 139 codec->power_save_node = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 140 f72706be354b35 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-16 141 val = snd_hda_get_bool_hint(codec, "auto_mute"); f72706be354b35 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-16 142 if (val >= 0) f72706be354b35 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-16 143 spec->suppress_auto_mute = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 144 val = snd_hda_get_bool_hint(codec, "auto_mic"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 145 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 146 spec->suppress_auto_mic = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 147 val = snd_hda_get_bool_hint(codec, "line_in_auto_switch"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 148 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 149 spec->line_in_auto_switch = !!val; 7eebffd3f4328c sound/pci/hda/hda_generic.c Takashi Iwai 2013-06-24 150 val = snd_hda_get_bool_hint(codec, "auto_mute_via_amp"); 7eebffd3f4328c sound/pci/hda/hda_generic.c Takashi Iwai 2013-06-24 151 if (val >= 0) 7eebffd3f4328c sound/pci/hda/hda_generic.c Takashi Iwai 2013-06-24 152 spec->auto_mute_via_amp = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 153 val = snd_hda_get_bool_hint(codec, "need_dac_fix"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 154 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 155 spec->need_dac_fix = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 156 val = snd_hda_get_bool_hint(codec, "primary_hp"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 157 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 158 spec->no_primary_hp = !val; da96fb5b0185d2 sound/pci/hda/hda_generic.c Takashi Iwai 2013-07-29 159 val = snd_hda_get_bool_hint(codec, "multi_io"); da96fb5b0185d2 sound/pci/hda/hda_generic.c Takashi Iwai 2013-07-29 160 if (val >= 0) da96fb5b0185d2 sound/pci/hda/hda_generic.c Takashi Iwai 2013-07-29 161 spec->no_multi_io = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 162 val = snd_hda_get_bool_hint(codec, "multi_cap_vol"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 163 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 164 spec->multi_cap_vol = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 165 val = snd_hda_get_bool_hint(codec, "inv_dmic_split"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 166 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 167 spec->inv_dmic_split = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 168 val = snd_hda_get_bool_hint(codec, "indep_hp"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 169 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 170 spec->indep_hp = !!val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 171 val = snd_hda_get_bool_hint(codec, "add_stereo_mix_input"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 172 if (val >= 0) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 173 spec->add_stereo_mix_input = !!val; f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 174 /* the following two are just for compatibility */ 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 175 val = snd_hda_get_bool_hint(codec, "add_out_jack_modes"); 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 176 if (val >= 0) f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 177 spec->add_jack_modes = !!val; 29476558deb301 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-17 178 val = snd_hda_get_bool_hint(codec, "add_in_jack_modes"); 29476558deb301 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-17 179 if (val >= 0) f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 180 spec->add_jack_modes = !!val; f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 181 val = snd_hda_get_bool_hint(codec, "add_jack_modes"); f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 182 if (val >= 0) f811c3cf8fae63 sound/pci/hda/hda_generic.c Takashi Iwai 2013-03-07 183 spec->add_jack_modes = !!val; 55196fffc95105 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-24 184 val = snd_hda_get_bool_hint(codec, "power_down_unused"); 55196fffc95105 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-24 185 if (val >= 0) 55196fffc95105 sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-24 186 spec->power_down_unused = !!val; 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 187 val = snd_hda_get_bool_hint(codec, "add_hp_mic"); 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 188 if (val >= 0) 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 189 spec->hp_mic = !!val; 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 190 val = snd_hda_get_bool_hint(codec, "hp_mic_detect"); 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 191 if (val >= 0) 967303dabc2233 sound/pci/hda/hda_generic.c Takashi Iwai 2013-02-19 192 spec->suppress_hp_mic_detect = !val; 7480316c265c9f sound/pci/hda/hda_generic.c Takashi Iwai 2017-04-10 193 val = snd_hda_get_bool_hint(codec, "vmaster"); 7480316c265c9f sound/pci/hda/hda_generic.c Takashi Iwai 2017-04-10 194 if (val >= 0) 7480316c265c9f sound/pci/hda/hda_generic.c Takashi Iwai 2017-04-10 195 spec->suppress_vmaster = !val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 196 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 197 if (!snd_hda_get_int_hint(codec, "mixer_nid", &val)) 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 @198 spec->mixer_nid = val; 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 199 } 1c70a583417e8d sound/pci/hda/hda_generic.c Takashi Iwai 2013-01-11 200 :::::: The code at line 198 was first introduced by commit :::::: 1c70a583417e8db1e1d5069d7651ba294e9499de ALSA: hda - Allow user to give hints for codec parser behavior :::::: TO: Takashi Iwai <[email protected]> :::::: CC: Takashi Iwai <[email protected]> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki