lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202504270214.c0aGJBHt-lkp@intel.com>
Date: Sun, 27 Apr 2025 02:54:04 +0800
From: kernel test robot <lkp@...el.com>
To: Ovidiu Bunea <Ovidiu.Bunea@....com>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
	Alex Deucher <alexander.deucher@....com>,
	Nicholas Kazlauskas <nicholas.kazlauskas@....com>,
	Hamza Mahfooz <hamza.mahfooz@....com>, linux-doc@...r.kernel.org
Subject: drivers/gpu/drm/amd/amdgpu/../display/dc/dccg/dcn35/dcn35_dccg.c:1416:
 warning: This comment starts with '/**', but isn't a kernel-doc comment.
 Refer Documentation/doc-guide/kernel-doc.rst

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   02ddfb981de88a2c15621115dd7be2431252c568
commit: a3e6079bd93d5c66a43bf6a5f90e5b98465dc7b3 drm/amd/display: Remove PIPE_DTO_SRC_SEL programming from set_dtbclk_dto
date:   5 months ago
config: i386-buildonly-randconfig-002-20250426 (https://download.01.org/0day-ci/archive/20250427/202504270214.c0aGJBHt-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250427/202504270214.c0aGJBHt-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 <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202504270214.c0aGJBHt-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/amd/amdgpu/../display/dc/dccg/dcn35/dcn35_dccg.c:1416: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
                    * PIPEx_DTO_SRC_SEL should not be programmed during DTBCLK update since OTG may still be on, and the


vim +1416 drivers/gpu/drm/amd/amdgpu/../display/dc/dccg/dcn35/dcn35_dccg.c

  1348	
  1349	/* Controls the generation of pixel valid for OTG in (OTG -> HPO case) */
  1350	static void dccg35_set_dtbclk_dto(
  1351			struct dccg *dccg,
  1352			const struct dtbclk_dto_params *params)
  1353	{
  1354		struct dcn_dccg *dccg_dcn = TO_DCN_DCCG(dccg);
  1355		/* DTO Output Rate / Pixel Rate = 1/4 */
  1356		int req_dtbclk_khz = params->pixclk_khz / 4;
  1357	
  1358		if (params->ref_dtbclk_khz && req_dtbclk_khz) {
  1359			uint32_t modulo, phase;
  1360	
  1361			switch (params->otg_inst) {
  1362			case 0:
  1363				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P0_GATE_DISABLE, 1);
  1364				break;
  1365			case 1:
  1366				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P1_GATE_DISABLE, 1);
  1367				break;
  1368			case 2:
  1369				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P2_GATE_DISABLE, 1);
  1370				break;
  1371			case 3:
  1372				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P3_GATE_DISABLE, 1);
  1373				break;
  1374			}
  1375	
  1376			// phase / modulo = dtbclk / dtbclk ref
  1377			modulo = params->ref_dtbclk_khz * 1000;
  1378			phase = req_dtbclk_khz * 1000;
  1379	
  1380			REG_WRITE(DTBCLK_DTO_MODULO[params->otg_inst], modulo);
  1381			REG_WRITE(DTBCLK_DTO_PHASE[params->otg_inst], phase);
  1382	
  1383			REG_UPDATE(OTG_PIXEL_RATE_CNTL[params->otg_inst],
  1384					DTBCLK_DTO_ENABLE[params->otg_inst], 1);
  1385	
  1386			REG_WAIT(OTG_PIXEL_RATE_CNTL[params->otg_inst],
  1387					DTBCLKDTO_ENABLE_STATUS[params->otg_inst], 1,
  1388					1, 100);
  1389	
  1390			/* program OTG_PIXEL_RATE_DIV for DIVK1 and DIVK2 fields */
  1391			dccg35_set_pixel_rate_div(dccg, params->otg_inst, PIXEL_RATE_DIV_BY_1, PIXEL_RATE_DIV_BY_1);
  1392	
  1393			/* The recommended programming sequence to enable DTBCLK DTO to generate
  1394			 * valid pixel HPO DPSTREAM ENCODER, specifies that DTO source select should
  1395			 * be set only after DTO is enabled.
  1396			 * PIPEx_DTO_SRC_SEL should not be programmed during DTBCLK update since OTG may still be on, and the
  1397			 * programming is handled in program_pix_clk() regardless, so it can be removed from here.
  1398			 */
  1399		} else {
  1400			switch (params->otg_inst) {
  1401			case 0:
  1402				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P0_GATE_DISABLE, 0);
  1403				break;
  1404			case 1:
  1405				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P1_GATE_DISABLE, 0);
  1406				break;
  1407			case 2:
  1408				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P2_GATE_DISABLE, 0);
  1409				break;
  1410			case 3:
  1411				REG_UPDATE(DCCG_GATE_DISABLE_CNTL5, DTBCLK_P3_GATE_DISABLE, 0);
  1412				break;
  1413			}
  1414	
  1415			/**
> 1416			 * PIPEx_DTO_SRC_SEL should not be programmed during DTBCLK update since OTG may still be on, and the
  1417			 * programming is handled in program_pix_clk() regardless, so it can be removed from here.
  1418			 */
  1419			REG_UPDATE(OTG_PIXEL_RATE_CNTL[params->otg_inst],
  1420					DTBCLK_DTO_ENABLE[params->otg_inst], 0);
  1421	
  1422			REG_WRITE(DTBCLK_DTO_MODULO[params->otg_inst], 0);
  1423			REG_WRITE(DTBCLK_DTO_PHASE[params->otg_inst], 0);
  1424		}
  1425	}
  1426	

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ