dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Harry Wentland <harry.wentland@amd.com>,
	dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org
Cc: oe-kbuild-all@lists.linux.dev,
	wayland-devel@lists.freedesktop.org,
	Harry Wentland <harry.wentland@amd.com>
Subject: Re: [PATCH v6 21/44] drm/vkms: Add tests for CTM handling
Date: Sat, 5 Oct 2024 19:03:03 +0800	[thread overview]
Message-ID: <202410051850.cGJoBKkg-lkp@intel.com> (raw)
In-Reply-To: <20241003200129.1732122-22-harry.wentland@amd.com>

Hi Harry,

kernel test robot noticed the following build warnings:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on drm-exynos/exynos-drm-next drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-misc/drm-misc-next drm-tip/drm-tip linus/master v6.12-rc1 next-20241004]
[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/Harry-Wentland/drm-Add-helper-for-conversion-from-signed-magnitude/20241004-040629
base:   git://anongit.freedesktop.org/drm/drm drm-next
patch link:    https://lore.kernel.org/r/20241003200129.1732122-22-harry.wentland%40amd.com
patch subject: [PATCH v6 21/44] drm/vkms: Add tests for CTM handling
config: csky-randconfig-r112-20241005 (https://download.01.org/0day-ci/archive/20241005/202410051850.cGJoBKkg-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 14.1.0
reproduce: (https://download.01.org/0day-ci/archive/20241005/202410051850.cGJoBKkg-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410051850.cGJoBKkg-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
   drivers/gpu/drm/vkms/tests/vkms_color_test.c:34:29: sparse: sparse: symbol 'test_linear_lut' was not declared. Should it be static?
>> drivers/gpu/drm/vkms/tests/vkms_color_test.c:193:32: sparse: sparse: symbol 'test_matrix_3x4_50_desat' was not declared. Should it be static?
>> drivers/gpu/drm/vkms/tests/vkms_color_test.c:259:32: sparse: sparse: symbol 'test_matrix_3x4_bt709_enc' was not declared. Should it be static?

vim +/test_matrix_3x4_50_desat +193 drivers/gpu/drm/vkms/tests/vkms_color_test.c

   192	
 > 193	const struct drm_color_ctm_3x4 test_matrix_3x4_50_desat = { {
   194		FIXPT_HALF, FIXPT_QUARTER, FIXPT_QUARTER, 0,
   195		FIXPT_QUARTER, FIXPT_HALF, FIXPT_QUARTER, 0,
   196		FIXPT_QUARTER, FIXPT_QUARTER, FIXPT_HALF, 0
   197	} };
   198	
   199	static void vkms_color_ctm_3x4_50_desat(struct kunit *test)
   200	{
   201		struct pixel_argb_s32 ref, out;
   202	
   203		/* full white */
   204		ref.a = 0xffff;
   205		ref.r = 0xffff;
   206		ref.g = 0xffff;
   207		ref.b = 0xffff;
   208	
   209		memcpy(&out, &ref, sizeof(out));
   210		apply_3x4_matrix(&out, &test_matrix_3x4_50_desat);
   211	
   212		KUNIT_EXPECT_MEMEQ(test, &ref, &out, sizeof(out));
   213	
   214		/* full black */
   215		ref.a = 0xffff;
   216		ref.r = 0x0;
   217		ref.g = 0x0;
   218		ref.b = 0x0;
   219	
   220		memcpy(&out, &ref, sizeof(out));
   221		apply_3x4_matrix(&out, &test_matrix_3x4_50_desat);
   222	
   223		KUNIT_EXPECT_MEMEQ(test, &ref, &out, sizeof(out));
   224	
   225		/* 50% grey */
   226		ref.a = 0xffff;
   227		ref.r = 0x8000;
   228		ref.g = 0x8000;
   229		ref.b = 0x8000;
   230	
   231		memcpy(&out, &ref, sizeof(out));
   232		apply_3x4_matrix(&out, &test_matrix_3x4_50_desat);
   233	
   234		KUNIT_EXPECT_MEMEQ(test, &ref, &out, sizeof(out));
   235	
   236		/* full red to 50% desat */
   237		ref.a = 0xffff;
   238		ref.r = 0x8000;
   239		ref.g = 0x4000;
   240		ref.b = 0x4000;
   241	
   242		out.a = 0xffff;
   243		out.r = 0xffff;
   244		out.g = 0x0;
   245		out.b = 0x0;
   246	
   247		apply_3x4_matrix(&out, &test_matrix_3x4_50_desat);
   248	
   249		KUNIT_EXPECT_MEMEQ(test, &ref, &out, sizeof(out));
   250	}
   251	
   252	/*
   253	 * BT.709 encoding matrix
   254	 *
   255	 * Values printed from within IGT when converting
   256	 * igt_matrix_3x4_bt709_enc to the fixed-point format expected
   257	 * by DRM/KMS.
   258	 */
 > 259	const struct drm_color_ctm_3x4 test_matrix_3x4_bt709_enc = { {
   260		0x00000000366cf400ull, 0x00000000b7175900ull, 0x0000000127bb300ull, 0,
   261		0x800000001993b3a0ull, 0x800000005609fe80ull, 0x000000006f9db200ull, 0,
   262		0x000000009d70a400ull, 0x800000008f011100ull, 0x800000000e6f9330ull, 0
   263	} };
   264	

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

  parent reply	other threads:[~2024-10-05 11:03 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-03 20:00 [PATCH v6 00/44] Color Pipeline API w/ VKMS Harry Wentland
2024-10-03 20:00 ` [PATCH v6 01/44] drm: Add helper for conversion from signed-magnitude Harry Wentland
2024-10-03 20:00 ` [PATCH v6 02/44] drm/vkms: Round fixp2int conversion in lerp_u16 Harry Wentland
2024-10-03 20:00 ` [PATCH v6 03/44] drm/vkms: Add kunit tests for VKMS LUT handling Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-05  8:49   ` kernel test robot
2024-10-08 15:16   ` Jeff Johnson
2024-10-03 20:00 ` [PATCH v6 04/44] drm/doc/rfc: Describe why prescriptive color pipeline is needed Harry Wentland
2024-10-03 20:00 ` [PATCH v6 05/44] drm/colorop: Introduce new drm_colorop mode object Harry Wentland
2024-10-13 15:19   ` Simon Ser
2024-10-13 15:30     ` Simon Ser
2024-10-03 20:00 ` [PATCH v6 06/44] drm/colorop: Add TYPE property Harry Wentland
2024-10-03 20:00 ` [PATCH v6 07/44] drm/colorop: Add 1D Curve subtype Harry Wentland
2024-10-03 20:00 ` [PATCH v6 08/44] Documentation/gpu: document drm_colorop Harry Wentland
2024-10-03 20:00 ` [PATCH v6 09/44] drm/colorop: Add BYPASS property Harry Wentland
2024-10-03 20:00 ` [PATCH v6 10/44] drm/colorop: Add NEXT property Harry Wentland
2024-10-03 20:00 ` [PATCH v6 11/44] drm/colorop: Add atomic state print for drm_colorop Harry Wentland
2024-10-03 20:00 ` [PATCH v6 12/44] drm/plane: Add COLOR PIPELINE property Harry Wentland
2024-10-03 20:00 ` [PATCH v6 13/44] drm/colorop: Add NEXT to colorop state print Harry Wentland
2024-10-13 15:16   ` Simon Ser
2024-10-03 20:00 ` [PATCH v6 14/44] drm/vkms: Add enumerated 1D curve colorop Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-03 20:00 ` [PATCH v6 15/44] drm/vkms: Add kunit tests for linear and sRGB LUTs Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-05  5:31   ` kernel test robot
2024-10-03 20:00 ` [PATCH v6 16/44] drm/colorop: Introduce DRM_CLIENT_CAP_PLANE_COLOR_PIPELINE Harry Wentland
2024-10-13 15:15   ` Simon Ser
2024-10-03 20:00 ` [PATCH v6 17/44] drm/colorop: Add 3x4 CTM type Harry Wentland
2024-10-03 20:01 ` [PATCH v6 18/44] drm/vkms: Use s32 for internal color pipeline precision Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-12-18 21:12     ` Harry Wentland
2024-10-03 20:01 ` [PATCH v6 19/44] drm/vkms: add 3x4 matrix in color pipeline Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-03 20:01 ` [PATCH v6 20/44] drm/tests: Add a few tests around drm_fixed.h Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-03 20:01 ` [PATCH v6 21/44] drm/vkms: Add tests for CTM handling Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-10-05 11:03   ` kernel test robot [this message]
2024-10-03 20:01 ` [PATCH v6 22/44] drm/colorop: pass plane_color_pipeline client cap to atomic check Harry Wentland
2024-10-03 20:01 ` [PATCH v6 23/44] drm/colorop: define a new macro for_each_new_colorop_in_state Harry Wentland
2024-10-03 20:01 ` [PATCH v6 24/44] drm/amd/display: Ignore deprecated props when plane_color_pipeline set Harry Wentland
2024-10-03 20:01 ` [PATCH v6 25/44] drm/amd/display: Add bypass COLOR PIPELINE Harry Wentland
2024-10-03 20:01 ` [PATCH v6 26/44] drm/amd/display: Skip color pipeline initialization for cursor plane Harry Wentland
2024-10-03 20:01 ` [PATCH v6 27/44] drm/amd/display: Add support for sRGB EOTF in DEGAM block Harry Wentland
2024-10-03 20:01 ` [PATCH v6 28/44] drm/amd/display: Add support for sRGB Inverse EOTF in SHAPER block Harry Wentland
2024-10-03 20:01 ` [PATCH v6 29/44] drm/amd/display: Add support for sRGB EOTF in BLND block Harry Wentland
2024-10-05 12:25   ` kernel test robot
2024-10-05 15:11   ` kernel test robot
2024-10-03 20:01 ` [PATCH v6 30/44] drm/colorop: Add PQ 125 EOTF and its inverse Harry Wentland
2024-10-03 20:01 ` [PATCH v6 31/44] drm/amd/display: Enable support for PQ 125 EOTF and Inverse Harry Wentland
2024-10-03 20:01 ` [PATCH v6 32/44] drm/colorop: add BT2020/BT709 OETF and Inverse OETF Harry Wentland
2024-10-03 20:01 ` [PATCH v6 33/44] drm/amd/display: Add support for BT.709 and BT.2020 TFs Harry Wentland
2024-10-03 20:01 ` [PATCH v6 34/44] drm/colorop: Add 1D Curve Custom LUT type Harry Wentland
2024-10-03 20:01 ` [PATCH v6 35/44] drm/amd/display: add shaper and blend colorops for 1D Curve Custom LUT Harry Wentland
2024-10-03 20:01 ` [PATCH v6 36/44] drm/amd/display: add 3x4 matrix colorop Harry Wentland
2024-10-05 11:03   ` kernel test robot
2024-10-03 20:01 ` [PATCH v6 37/44] drm/colorop: Add mutliplier type Harry Wentland
2024-10-03 20:01 ` [PATCH v6 38/44] drm/amd/display: add multiplier colorop Harry Wentland
2024-10-03 20:01 ` [PATCH v6 39/44] drm/amd/display: Swap matrix and multiplier Harry Wentland
2024-10-03 20:01 ` [PATCH v6 40/44] drm/colorop: Define LUT_1D interpolation Harry Wentland
2024-10-03 20:01 ` [PATCH v6 41/44] drm/colorop: allow non-bypass colorops Harry Wentland
2024-10-04 11:43   ` Louis Chauvet
2024-12-20  5:02     ` Alex Hung
2024-10-03 20:01 ` [PATCH v6 42/44] drm/colorop: Add 3D LUT supports to color pipeline Harry Wentland
2024-10-13 15:58   ` Simon Ser
2024-10-18 20:23     ` Alex Hung
2024-10-03 20:01 ` [PATCH v6 43/44] drm/amd/display: add 3D LUT colorop Harry Wentland
2024-10-03 20:01 ` [PATCH v6 44/44] drm/amd/display: Add AMD color pipeline doc Harry Wentland

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=202410051850.cGJoBKkg-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=harry.wentland@amd.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=wayland-devel@lists.freedesktop.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox