From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C7F7D196C7C; Fri, 1 May 2026 04:05:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.20 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777608323; cv=none; b=hxfG5Yx15KNH3zb9w1nRXgxshSnLT5duyi3Re+mEhTIWEzxMIOCTtO2MxL+PV6nNzFL11iykr1GYHANkVxgKX1xb4MIdPYH3mQ7TMoubOcNe2/7hc5RijhEhTEyubdA1PZqaMlJqOol8s+mISgdJnsrD4jAjMrkh6QZMs8rE2Xs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777608323; c=relaxed/simple; bh=n3SVW2/BPmkjrmRDdOHgYmVGwRsqVmOibTPTW2Xi400=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ksW1mc6WW6NLBIb/gl0SaCtr0DzwfILto4ZwakDEBXXzepUyymDbIP15rD3OiyJilBKjrTRsucw2KM3jruMq5cJKELvzKa2ScWBTuhGpJYlSZt4k3sl9axhfduGxTNm9PMzFk2HOWnF0+jyNW/u9DMtWec2k2HodhrPd7OJKowI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=QRMLjEh5; arc=none smtp.client-ip=198.175.65.20 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="QRMLjEh5" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1777608320; x=1809144320; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=n3SVW2/BPmkjrmRDdOHgYmVGwRsqVmOibTPTW2Xi400=; b=QRMLjEh51PVQYxNmd046G3Ds+h6ORkgis+5tDoYGWqYos1OxHciiqEOv e6MLLybBjeqAzuFZVxG4IfIj0y2nqyZJJrhK/vPoV1/MYAS0uaJxT/UB9 CL0rvoQgphUHIeoMFCeNsxfCvBHW7pW4jShQo+ovGz4ZFDSmId4obEF5x NSOGPgRXANgLvGDnNH5lA/J9o1eppPBeXE4BfGkypTtIda96N48emHLBk ZZhO9uuEe9ruTmChyn2JXFGeIKR0NdhlFNP0EvRB++5Bb5fsyfQnh8vGF /pRc3jC7mgPMKEbnj9AqXi6xXaptgyikNSwsrRqhc87PH2AR2c02ZiZcl Q==; X-CSE-ConnectionGUID: Ph9tQobOQFKrMzA/2H0CIg== X-CSE-MsgGUID: oEaqBxrgSauftVOoP+G3VA== X-IronPort-AV: E=McAfee;i="6800,10657,11772"; a="78283599" X-IronPort-AV: E=Sophos;i="6.23,209,1770624000"; d="scan'208";a="78283599" Received: from orviesa010.jf.intel.com ([10.64.159.150]) by orvoesa112.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Apr 2026 21:05:20 -0700 X-CSE-ConnectionGUID: h85BGaGUSruEIcaAGgP9AA== X-CSE-MsgGUID: 3JL2KKsmRcmiBFyv4Jt+2A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,209,1770624000"; d="scan'208";a="233910004" Received: from lkp-server01.sh.intel.com (HELO 781826d00641) ([10.239.97.150]) by orviesa010.jf.intel.com with ESMTP; 30 Apr 2026 21:05:18 -0700 Received: from kbuild by 781826d00641 with local (Exim 4.98.2) (envelope-from ) id 1wIf7r-0000000009Z-2WKt; Fri, 01 May 2026 04:05:15 +0000 Date: Fri, 1 May 2026 12:05:13 +0800 From: kernel test robot To: wangdich9700@163.com, perex@perex.cz, tiwai@suse.com Cc: oe-kbuild-all@lists.linux.dev, kees@kernel.org, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, wangdicheng Subject: Re: [PATCH] ALSA: hda/conexant: Fix missing error check for jack detection Message-ID: <202605011107.lDwIcnm6-lkp@intel.com> References: <20260427070739.133369-1-wangdich9700@163.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260427070739.133369-1-wangdich9700@163.com> Hi, kernel test robot noticed the following build warnings: [auto build test WARNING on tiwai-sound/for-next] [also build test WARNING on v7.1-rc1] [cannot apply to tiwai-sound/for-linus linus/master next-20260430] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/wangdich9700-163-com/ALSA-hda-conexant-Fix-missing-error-check-for-jack-detection/20260427-223457 base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next patch link: https://lore.kernel.org/r/20260427070739.133369-1-wangdich9700%40163.com patch subject: [PATCH] ALSA: hda/conexant: Fix missing error check for jack detection config: i386-randconfig-062-20260430 (https://download.01.org/0day-ci/archive/20260501/202605011107.lDwIcnm6-lkp@intel.com/config) compiler: gcc-13 (Debian 13.3.0-16) 13.3.0 sparse: v0.6.5-rc1 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260501/202605011107.lDwIcnm6-lkp@intel.com/reproduce) 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 | Closes: https://lore.kernel.org/oe-kbuild-all/202605011107.lDwIcnm6-lkp@intel.com/ sparse warnings: (new ones prefixed by >>) >> sound/hda/codecs/conexant.c:1193:21: sparse: sparse: incorrect type in assignment (different base types) @@ expected int err @@ got struct hda_jack_callback * @@ sound/hda/codecs/conexant.c:1193:21: sparse: expected int err sound/hda/codecs/conexant.c:1193:21: sparse: got struct hda_jack_callback * vim +1193 sound/hda/codecs/conexant.c 1174 1175 static int cx_probe(struct hda_codec *codec, const struct hda_device_id *id) 1176 { 1177 struct conexant_spec *spec; 1178 int err; 1179 1180 codec_info(codec, "%s: BIOS auto-probing.\n", codec->core.chip_name); 1181 1182 spec = kzalloc_obj(*spec); 1183 if (!spec) 1184 return -ENOMEM; 1185 snd_hda_gen_spec_init(&spec->gen); 1186 codec->spec = spec; 1187 1188 /* init cx11880/sn6140 flag and reset headset_present_flag */ 1189 switch (codec->core.vendor_id) { 1190 case 0x14f11f86: 1191 case 0x14f11f87: 1192 spec->is_cx11880_sn6140 = true; > 1193 err = snd_hda_jack_detect_enable_callback(codec, 0x19, cx_update_headset_mic_vref); 1194 if (err < 0) 1195 goto error; 1196 break; 1197 } 1198 1199 cx_auto_parse_eapd(codec); 1200 spec->gen.own_eapd_ctl = 1; 1201 1202 switch (codec->core.vendor_id) { 1203 case 0x14f15045: 1204 codec->single_adc_amp = 1; 1205 spec->gen.mixer_nid = 0x17; 1206 spec->gen.add_stereo_mix_input = HDA_HINT_STEREO_MIX_AUTO; 1207 snd_hda_pick_fixup(codec, cxt5045_fixup_models, 1208 cxt5045_fixups, cxt_fixups); 1209 break; 1210 case 0x14f15047: 1211 codec->pin_amp_workaround = 1; 1212 spec->gen.mixer_nid = 0x19; 1213 spec->gen.add_stereo_mix_input = HDA_HINT_STEREO_MIX_AUTO; 1214 snd_hda_pick_fixup(codec, cxt5047_fixup_models, 1215 cxt5047_fixups, cxt_fixups); 1216 break; 1217 case 0x14f15051: 1218 add_cx5051_fake_mutes(codec); 1219 codec->pin_amp_workaround = 1; 1220 snd_hda_pick_fixup(codec, cxt5051_fixup_models, 1221 cxt5051_fixups, cxt_fixups); 1222 break; 1223 case 0x14f15098: 1224 codec->pin_amp_workaround = 1; 1225 spec->gen.mixer_nid = 0x22; 1226 spec->gen.add_stereo_mix_input = HDA_HINT_STEREO_MIX_AUTO; 1227 snd_hda_pick_fixup(codec, cxt5066_fixup_models, 1228 cxt5066_fixups, cxt_fixups); 1229 break; 1230 case 0x14f150f2: 1231 codec->power_save_node = 1; 1232 fallthrough; 1233 default: 1234 codec->pin_amp_workaround = 1; 1235 snd_hda_pick_fixup(codec, cxt5066_fixup_models, 1236 cxt5066_fixups, cxt_fixups); 1237 break; 1238 } 1239 1240 if (!spec->gen.vmaster_mute.hook && spec->dynamic_eapd) 1241 spec->gen.vmaster_mute.hook = cx_auto_vmaster_hook; 1242 1243 snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_PRE_PROBE); 1244 1245 err = snd_hda_parse_pin_defcfg(codec, &spec->gen.autocfg, NULL, 1246 spec->parse_flags); 1247 if (err < 0) 1248 goto error; 1249 1250 err = cx_auto_parse_beep(codec); 1251 if (err < 0) 1252 goto error; 1253 1254 err = snd_hda_gen_parse_auto_config(codec, &spec->gen.autocfg); 1255 if (err < 0) 1256 goto error; 1257 1258 /* Some laptops with Conexant chips show stalls in S3 resume, 1259 * which falls into the single-cmd mode. 1260 * Better to make reset, then. 1261 */ 1262 if (!codec->bus->core.sync_write) { 1263 codec_info(codec, 1264 "Enable sync_write for stable communication\n"); 1265 codec->bus->core.sync_write = 1; 1266 codec->bus->allow_bus_reset = 1; 1267 } 1268 1269 snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_PROBE); 1270 1271 return 0; 1272 1273 error: 1274 cx_remove(codec); 1275 return err; 1276 } 1277 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki