[linux-next:master 9967/10320] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c:117:11: warning: variable 'channels' set but not used

* [linux-next:master 9967/10320] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn31/dcn31_apg.c:117:11: warning: variable 'channels' set but not used
@ 2021-08-25 12:44 ` kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-08-25 12:44 UTC (permalink / raw)
  To: Fangzhi Zuo
  Cc: kbuild-all, Linux Memory Management List, Alex Deucher,
	Nicholas Kazlauskas

[-- Attachment #1: Type: text/plain, Size: 2836 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   7636510f976d75b860848884169ba985c8f844d8
commit: c45ffcf7ea9d57b79e264b6c2d8afb5e95a93097 [9967/10320] drm/amd/display: Add DP 2.0 Audio Package Generator
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=c45ffcf7ea9d57b79e264b6c2d8afb5e95a93097
        git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
        git fetch --no-tags linux-next master
        git checkout c45ffcf7ea9d57b79e264b6c2d8afb5e95a93097
        # save the attached .config to linux build tree
        make W=1 ARCH=i386 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

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:11: 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:11: 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, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65192 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread