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
next prev 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