drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c:117:18: warning: variable 'channels' set but not used

From: kernel test robot
Date: Mon Sep 05 2022 - 14:04:56 EST


Hi Fangzhi,

FYI, the error/warning still remains.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 7e18e42e4b280c85b76967a9106a13ca61c16179
commit: 61452908a79ec936660494fb4b9f2a35ee42e6e0 drm/amd/display: Add DP 2.0 Audio Package Generator
date: 1 year ago
config: i386-buildonly-randconfig-r004-20220905 (https://download.01.org/0day-ci/archive/20220906/202209060159.tIsVfTJM-lkp@xxxxxxxxx/config)
compiler: gcc-11 (Debian 11.3.0-5) 11.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=61452908a79ec936660494fb4b9f2a35ee42e6e0
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 61452908a79ec936660494fb4b9f2a35ee42e6e0
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/drm/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c: In function 'apg31_se_audio_setup':
>> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c:117:18: warning: variable 'channels' set but not used [-Wunused-but-set-variable]
117 | uint32_t channels = 0;
| ^~~~~~~~
>> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c:116:18: warning: variable 'speakers' set but not used [-Wunused-but-set-variable]
116 | uint32_t speakers = 0;
| ^~~~~~~~


vim +/channels +117 drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c

108
109 static void apg31_se_audio_setup(
110 struct apg *apg,
111 unsigned int az_inst,
112 struct audio_info *audio_info)
113 {
114 struct dcn31_apg *apg31 = DCN31_APG_FROM_APG(apg);
115
> 116 uint32_t speakers = 0;
> 117 uint32_t channels = 0;
118
119 ASSERT(audio_info);
120 /* This should not happen.it does so we don't get BSOD*/
121 if (audio_info == NULL)
122 return;
123
124 speakers = audio_info->flags.info.ALLSPEAKERS;
125 channels = speakers_to_channels(audio_info->flags.speaker_flags).all;
126
127 /* DisplayPort only allows for one audio stream with stream ID 0 */
128 REG_UPDATE(APG_CONTROL2, APG_DP_AUDIO_STREAM_ID, 0);
129
130 /* When running in "pair mode", pairs of audio channels have their own enable
131 * this is for really old audio drivers */
132 REG_UPDATE(APG_DBG_GEN_CONTROL, APG_DBG_AUDIO_CHANNEL_ENABLE, 0xF);
133 // REG_UPDATE(APG_DBG_GEN_CONTROL, APG_DBG_AUDIO_CHANNEL_ENABLE, channels);
134
135 /* Disable forced mem power off */
136 REG_UPDATE(APG_MEM_PWR, APG_MEM_PWR_FORCE, 0);
137
138 apg31_enable(apg);
139 }
140

--
0-DAY CI Kernel Test Service
https://01.org/lkp