All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: sound/hda/codecs/generic.c:198 parse_user_hints() warn: assigning (-2) to unsigned variable 'spec->mixer_nid'
Date: Sat, 13 Dec 2025 07:27:08 +0800	[thread overview]
Message-ID: <202512130756.vSqaKZkq-lkp@intel.com> (raw)

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Takashi Iwai <tiwai@suse.de>
CC: Richard Fitzgerald <rf@opensource.cirrus.com>

Hi Takashi,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   187d0801404f415f22c0b31531982c7ea97fa341
commit: 6014e9021b28e634935c776c0271b5cbcabdc5d6 ALSA: hda: Move codec drivers into sound/hda/codecs directory
date:   5 months ago
:::::: branch date: 13 hours ago
:::::: commit date: 5 months ago
config: mips-randconfig-r072-20251212 (https://download.01.org/0day-ci/archive/20251213/202512130756.vSqaKZkq-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 13.4.0

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
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202512130756.vSqaKZkq-lkp@intel.com/

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 <tiwai@suse.de>
:::::: CC: Takashi Iwai <tiwai@suse.de>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

             reply	other threads:[~2025-12-12 23:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-12 23:27 kernel test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-03-24 14:43 sound/hda/codecs/generic.c:198 parse_user_hints() warn: assigning (-2) to unsigned variable 'spec->mixer_nid' kernel test robot
2026-07-10  5:30 kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202512130756.vSqaKZkq-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=error27@gmail.com \
    --cc=oe-kbuild@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.