* Re: [Intel-gfx] [PATCH v2 06/10] drm/fb-helper: Initialize fb-helper's preferred BPP in prepare function
[not found] <20230124134010.30263-7-tzimmermann@suse.de>
@ 2023-01-25 4:51 ` kernel test robot
2023-01-25 12:12 ` kernel test robot
1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-01-25 4:51 UTC (permalink / raw)
To: Thomas Zimmermann, airlied, daniel, maarten.lankhorst, mripard,
javierm
Cc: llvm, oe-kbuild-all, linux-samsung-soc, linux-arm-msm, intel-gfx,
dri-devel, amd-gfx, Thomas Zimmermann, linux-tegra, freedreno,
linux-arm-kernel
Hi Thomas,
I love your patch! Yet something to improve:
[auto build test ERROR on 7d3e7f64a42d66ba8da6e7b66a8d85457ef84570]
url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Zimmermann/drm-client-Test-for-connectors-before-sending-hotplug-event/20230124-214220
base: 7d3e7f64a42d66ba8da6e7b66a8d85457ef84570
patch link: https://lore.kernel.org/r/20230124134010.30263-7-tzimmermann%40suse.de
patch subject: [Intel-gfx] [PATCH v2 06/10] drm/fb-helper: Initialize fb-helper's preferred BPP in prepare function
config: riscv-randconfig-r042-20230123 (https://download.01.org/0day-ci/archive/20230125/202301251250.eyIn4zjn-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 4196ca3278f78c6e19246e54ab0ecb364e37d66a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/intel-lab-lkp/linux/commit/70e38534e74e4d12bb02b3b352bba2aed417f541
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Thomas-Zimmermann/drm-client-Test-for-connectors-before-sending-hotplug-event/20230124-214220
git checkout 70e38534e74e4d12bb02b3b352bba2aed417f541
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/gpu/drm/radeon/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/radeon/radeon_fb.c:352:10: error: too many arguments to function call, expected 3, have 4
&radeon_fb_helper_funcs);
^~~~~~~~~~~~~~~~~~~~~~~
include/drm/drm_fb_helper.h:295:20: note: 'drm_fb_helper_prepare' declared here
static inline void drm_fb_helper_prepare(struct drm_device *dev,
^
>> drivers/gpu/drm/radeon/radeon_fb.c:361:52: error: too few arguments to function call, expected 2, have 1
ret = drm_fb_helper_initial_config(&rfbdev->helper);
~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
include/drm/drm_fb_helper.h:459:19: note: 'drm_fb_helper_initial_config' declared here
static inline int drm_fb_helper_initial_config(struct drm_fb_helper *fb_helper,
^
2 errors generated.
vim +352 drivers/gpu/drm/radeon/radeon_fb.c
386516744ba45d Dave Airlie 2010-03-30 326
386516744ba45d Dave Airlie 2010-03-30 327 int radeon_fbdev_init(struct radeon_device *rdev)
386516744ba45d Dave Airlie 2010-03-30 328 {
8be48d924c307e Dave Airlie 2010-03-30 329 struct radeon_fbdev *rfbdev;
4abe35204af82a Dave Airlie 2010-03-30 330 int bpp_sel = 32;
5a79395b2791cc Chris Wilson 2010-06-06 331 int ret;
4abe35204af82a Dave Airlie 2010-03-30 332
e5f243bd2edd95 Alex Deucher 2016-03-10 333 /* don't enable fbdev if no connectors */
e5f243bd2edd95 Alex Deucher 2016-03-10 334 if (list_empty(&rdev->ddev->mode_config.connector_list))
e5f243bd2edd95 Alex Deucher 2016-03-10 335 return 0;
e5f243bd2edd95 Alex Deucher 2016-03-10 336
7b8bd6bb4298ac Egbert Eich 2017-07-18 337 /* select 8 bpp console on 8MB cards, or 16 bpp on RN50 or 32MB */
7b8bd6bb4298ac Egbert Eich 2017-07-18 338 if (rdev->mc.real_vram_size <= (8*1024*1024))
4abe35204af82a Dave Airlie 2010-03-30 339 bpp_sel = 8;
7b8bd6bb4298ac Egbert Eich 2017-07-18 340 else if (ASIC_IS_RN50(rdev) ||
7b8bd6bb4298ac Egbert Eich 2017-07-18 341 rdev->mc.real_vram_size <= (32*1024*1024))
7b8bd6bb4298ac Egbert Eich 2017-07-18 342 bpp_sel = 16;
8be48d924c307e Dave Airlie 2010-03-30 343
8be48d924c307e Dave Airlie 2010-03-30 344 rfbdev = kzalloc(sizeof(struct radeon_fbdev), GFP_KERNEL);
8be48d924c307e Dave Airlie 2010-03-30 345 if (!rfbdev)
8be48d924c307e Dave Airlie 2010-03-30 346 return -ENOMEM;
771fe6b912fca5 Jerome Glisse 2009-06-05 347
8be48d924c307e Dave Airlie 2010-03-30 348 rfbdev->rdev = rdev;
8be48d924c307e Dave Airlie 2010-03-30 349 rdev->mode_info.rfbdev = rfbdev;
10a231026574f9 Thierry Reding 2014-06-27 350
70e38534e74e4d Thomas Zimmermann 2023-01-24 351 drm_fb_helper_prepare(rdev->ddev, &rfbdev->helper, bpp_sel,
10a231026574f9 Thierry Reding 2014-06-27 @352 &radeon_fb_helper_funcs);
785b93ef8c3097 Dave Airlie 2009-08-28 353
2dea2d1182179e Pankaj Bharadiya 2020-03-05 354 ret = drm_fb_helper_init(rdev->ddev, &rfbdev->helper);
01934c2a691882 Thierry Reding 2014-12-19 355 if (ret)
01934c2a691882 Thierry Reding 2014-12-19 356 goto free;
5a79395b2791cc Chris Wilson 2010-06-06 357
76a39dbfb2d1bc Daniel Vetter 2013-01-20 358 /* disable all the possible outputs/crtcs before entering KMS mode */
76a39dbfb2d1bc Daniel Vetter 2013-01-20 359 drm_helper_disable_unused_functions(rdev->ddev);
76a39dbfb2d1bc Daniel Vetter 2013-01-20 360
70e38534e74e4d Thomas Zimmermann 2023-01-24 @361 ret = drm_fb_helper_initial_config(&rfbdev->helper);
01934c2a691882 Thierry Reding 2014-12-19 362 if (ret)
01934c2a691882 Thierry Reding 2014-12-19 363 goto fini;
01934c2a691882 Thierry Reding 2014-12-19 364
771fe6b912fca5 Jerome Glisse 2009-06-05 365 return 0;
01934c2a691882 Thierry Reding 2014-12-19 366
01934c2a691882 Thierry Reding 2014-12-19 367 fini:
01934c2a691882 Thierry Reding 2014-12-19 368 drm_fb_helper_fini(&rfbdev->helper);
01934c2a691882 Thierry Reding 2014-12-19 369 free:
01934c2a691882 Thierry Reding 2014-12-19 370 kfree(rfbdev);
01934c2a691882 Thierry Reding 2014-12-19 371 return ret;
386516744ba45d Dave Airlie 2010-03-30 372 }
386516744ba45d Dave Airlie 2010-03-30 373
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [Intel-gfx] [PATCH v2 06/10] drm/fb-helper: Initialize fb-helper's preferred BPP in prepare function
[not found] <20230124134010.30263-7-tzimmermann@suse.de>
2023-01-25 4:51 ` [Intel-gfx] [PATCH v2 06/10] drm/fb-helper: Initialize fb-helper's preferred BPP in prepare function kernel test robot
@ 2023-01-25 12:12 ` kernel test robot
1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-01-25 12:12 UTC (permalink / raw)
To: Thomas Zimmermann, airlied, daniel, maarten.lankhorst, mripard,
javierm
Cc: llvm, oe-kbuild-all, linux-samsung-soc, linux-arm-msm, intel-gfx,
dri-devel, amd-gfx, Thomas Zimmermann, linux-tegra, freedreno,
linux-arm-kernel
Hi Thomas,
I love your patch! Yet something to improve:
[auto build test ERROR on 7d3e7f64a42d66ba8da6e7b66a8d85457ef84570]
url: https://github.com/intel-lab-lkp/linux/commits/Thomas-Zimmermann/drm-client-Test-for-connectors-before-sending-hotplug-event/20230124-214220
base: 7d3e7f64a42d66ba8da6e7b66a8d85457ef84570
patch link: https://lore.kernel.org/r/20230124134010.30263-7-tzimmermann%40suse.de
patch subject: [Intel-gfx] [PATCH v2 06/10] drm/fb-helper: Initialize fb-helper's preferred BPP in prepare function
config: x86_64-randconfig-a014-20230123 (https://download.01.org/0day-ci/archive/20230125/202301252016.vm7ksFra-lkp@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/70e38534e74e4d12bb02b3b352bba2aed417f541
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Thomas-Zimmermann/drm-client-Test-for-connectors-before-sending-hotplug-event/20230124-214220
git checkout 70e38534e74e4d12bb02b3b352bba2aed417f541
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/drm/gma500/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/gma500/framebuffer.c:412:44: error: too many arguments to function call, expected 3, have 4
drm_fb_helper_prepare(dev, fb_helper, 32, &psb_fb_helper_funcs);
~~~~~~~~~~~~~~~~~~~~~ ^~~~~~~~~~~~~~~~~~~~
include/drm/drm_fb_helper.h:295:20: note: 'drm_fb_helper_prepare' declared here
static inline void drm_fb_helper_prepare(struct drm_device *dev,
^
>> drivers/gpu/drm/gma500/framebuffer.c:421:46: error: too few arguments to function call, expected 2, have 1
ret = drm_fb_helper_initial_config(fb_helper);
~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
include/drm/drm_fb_helper.h:459:19: note: 'drm_fb_helper_initial_config' declared here
static inline int drm_fb_helper_initial_config(struct drm_fb_helper *fb_helper,
^
2 errors generated.
vim +412 drivers/gpu/drm/gma500/framebuffer.c
397
398 int psb_fbdev_init(struct drm_device *dev)
399 {
400 struct drm_fb_helper *fb_helper;
401 struct drm_psb_private *dev_priv = to_drm_psb_private(dev);
402 int ret;
403
404 fb_helper = kzalloc(sizeof(*fb_helper), GFP_KERNEL);
405 if (!fb_helper) {
406 dev_err(dev->dev, "no memory\n");
407 return -ENOMEM;
408 }
409
410 dev_priv->fb_helper = fb_helper;
411
> 412 drm_fb_helper_prepare(dev, fb_helper, 32, &psb_fb_helper_funcs);
413
414 ret = drm_fb_helper_init(dev, fb_helper);
415 if (ret)
416 goto free;
417
418 /* disable all the possible outputs/crtcs before entering KMS mode */
419 drm_helper_disable_unused_functions(dev);
420
> 421 ret = drm_fb_helper_initial_config(fb_helper);
422 if (ret)
423 goto fini;
424
425 return 0;
426
427 fini:
428 drm_fb_helper_fini(fb_helper);
429 free:
430 kfree(fb_helper);
431 return ret;
432 }
433
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
^ permalink raw reply [flat|nested] 2+ messages in thread