AMD-GFX 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,
	Alex Hung <alex.hung@amd.com>,
	Harry Wentland <harry.wentland@amd.com>
Subject: Re: [PATCH v6 29/44] drm/amd/display: Add support for sRGB EOTF in BLND block
Date: Sat, 5 Oct 2024 23:11:28 +0800	[thread overview]
Message-ID: <202410052207.G19VLhRt-lkp@intel.com> (raw)
In-Reply-To: <20241003200129.1732122-30-harry.wentland@amd.com>

Hi Harry,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm/drm-next]
[also build test ERROR 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-30-harry.wentland%40amd.com
patch subject: [PATCH v6 29/44] drm/amd/display: Add support for sRGB EOTF in BLND block
config: x86_64-randconfig-122-20241005 (https://download.01.org/0day-ci/archive/20241005/202410052207.G19VLhRt-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241005/202410052207.G19VLhRt-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/202410052207.G19VLhRt-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_color.c:1296:46: error: variable 'blend_size' is uninitialized when used here [-Werror,-Wuninitialized]
    1296 |                 return __set_input_tf(NULL, tf, blend_lut, blend_size);
         |                                                            ^~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_color.c:1278:21: note: initialize the variable 'blend_size' to silence this warning
    1278 |         uint32_t blend_size;
         |                            ^
         |                             = 0
   1 error generated.


vim +/blend_size +1296 drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_color.c

  1273	
  1274		struct dc_transfer_func *tf = &dc_plane_state->blend_tf;
  1275		struct drm_colorop *colorop = colorop_state->colorop;
  1276		struct drm_device *drm = colorop->dev;
  1277		const struct drm_color_lut *blend_lut;
  1278		uint32_t blend_size;
  1279	
  1280		if (colorop->type != DRM_COLOROP_1D_CURVE &&
  1281		    colorop_state->curve_1d_type != DRM_COLOROP_1D_CURVE_SRGB_EOTF)
  1282			return -EINVAL;
  1283	
  1284		if (colorop_state->bypass) {
  1285			tf->type = TF_TYPE_BYPASS;
  1286			tf->tf = TRANSFER_FUNCTION_LINEAR;
  1287			return 0;
  1288		}
  1289	
  1290		drm_dbg(drm, "Blend colorop with ID: %d\n", colorop->base.id);
  1291	
  1292		if (colorop->type == DRM_COLOROP_1D_CURVE) {
  1293			tf->type = TF_TYPE_DISTRIBUTED_POINTS;
  1294			tf->tf = amdgpu_colorop_tf_to_dc_tf(colorop_state->curve_1d_type);
  1295			tf->sdr_ref_white_level = SDR_WHITE_LEVEL_INIT_VALUE;
> 1296			return __set_input_tf(NULL, tf, blend_lut, blend_size);
  1297		}
  1298	
  1299		return -EINVAL;
  1300	}
  1301	

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

  parent reply	other threads:[~2024-10-05 15:12 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
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 [this message]
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=202410052207.G19VLhRt-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=alex.hung@amd.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