From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: Re: [PATCH 4/4] drm/mgag200: Use DMA to copy the framebuffer to the VRAM
Date: Wed, 10 May 2023 05:54:02 +0800 [thread overview]
Message-ID: <202305100554.Q6f2fDlx-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20230505124337.854845-5-jfalempe@redhat.com>
References: <20230505124337.854845-5-jfalempe@redhat.com>
TO: Jocelyn Falempe <jfalempe@redhat.com>
TO: dri-devel@lists.freedesktop.org
TO: tzimmermann@suse.de
TO: airlied@redhat.com
TO: javierm@redhat.com
TO: lyude@redhat.com
CC: Jocelyn Falempe <jfalempe@redhat.com>
Hi Jocelyn,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 457391b0380335d5e9a5babdec90ac53928b23b4]
url: https://github.com/intel-lab-lkp/linux/commits/Jocelyn-Falempe/drm-mgag200-Rename-constant-MGAREG_Status-to-MGAREG_STATUS/20230505-204705
base: 457391b0380335d5e9a5babdec90ac53928b23b4
patch link: https://lore.kernel.org/r/20230505124337.854845-5-jfalempe%40redhat.com
patch subject: [PATCH 4/4] drm/mgag200: Use DMA to copy the framebuffer to the VRAM
:::::: branch date: 4 days ago
:::::: commit date: 4 days ago
config: i386-randconfig-m021 (https://download.01.org/0day-ci/archive/20230510/202305100554.Q6f2fDlx-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Link: https://lore.kernel.org/r/202305100554.Q6f2fDlx-lkp@intel.com/
smatch warnings:
drivers/gpu/drm/mgag200/mgag200_mode.c:419 mgag200_dwg_setup() error: uninitialized symbol 'maccess'.
vim +/maccess +419 drivers/gpu/drm/mgag200/mgag200_mode.c
414c453106255b Dave Airlie 2012-04-17 400
b3cab0043427ad Jocelyn Falempe 2023-05-05 401 static void mgag200_dwg_setup(struct mga_device *mdev, struct drm_framebuffer *fb)
b3cab0043427ad Jocelyn Falempe 2023-05-05 402 {
b3cab0043427ad Jocelyn Falempe 2023-05-05 403 u32 maccess;
b3cab0043427ad Jocelyn Falempe 2023-05-05 404
b3cab0043427ad Jocelyn Falempe 2023-05-05 405 drm_dbg(&mdev->base, "Setup DWG with %dx%d %p4cc\n",
b3cab0043427ad Jocelyn Falempe 2023-05-05 406 fb->width, fb->height, &fb->format->format);
b3cab0043427ad Jocelyn Falempe 2023-05-05 407
b3cab0043427ad Jocelyn Falempe 2023-05-05 408 switch (fb->format->format) {
b3cab0043427ad Jocelyn Falempe 2023-05-05 409 case DRM_FORMAT_RGB565:
b3cab0043427ad Jocelyn Falempe 2023-05-05 410 maccess = MGAMAC_PW16;
b3cab0043427ad Jocelyn Falempe 2023-05-05 411 break;
b3cab0043427ad Jocelyn Falempe 2023-05-05 412 case DRM_FORMAT_RGB888:
b3cab0043427ad Jocelyn Falempe 2023-05-05 413 maccess = MGAMAC_PW24;
b3cab0043427ad Jocelyn Falempe 2023-05-05 414 break;
b3cab0043427ad Jocelyn Falempe 2023-05-05 415 case DRM_FORMAT_XRGB8888:
b3cab0043427ad Jocelyn Falempe 2023-05-05 416 maccess = MGAMAC_PW32;
b3cab0043427ad Jocelyn Falempe 2023-05-05 417 break;
b3cab0043427ad Jocelyn Falempe 2023-05-05 418 }
b3cab0043427ad Jocelyn Falempe 2023-05-05 @419 WREG32(MGAREG_MACCESS, maccess);
b3cab0043427ad Jocelyn Falempe 2023-05-05 420
b3cab0043427ad Jocelyn Falempe 2023-05-05 421 /* Framebuffer width in pixel */
b3cab0043427ad Jocelyn Falempe 2023-05-05 422 WREG32(MGAREG_PITCH, fb->width);
b3cab0043427ad Jocelyn Falempe 2023-05-05 423
b3cab0043427ad Jocelyn Falempe 2023-05-05 424 /* Sane default value for the drawing engine registers */
b3cab0043427ad Jocelyn Falempe 2023-05-05 425 WREG32(MGAREG_DSTORG, 0);
b3cab0043427ad Jocelyn Falempe 2023-05-05 426 WREG32(MGAREG_YDSTORG, 0);
b3cab0043427ad Jocelyn Falempe 2023-05-05 427 WREG32(MGAREG_SRCORG, 0);
b3cab0043427ad Jocelyn Falempe 2023-05-05 428 WREG32(MGAREG_CXBNDRY, 0x0FFF0000);
b3cab0043427ad Jocelyn Falempe 2023-05-05 429 WREG32(MGAREG_YTOP, 0);
b3cab0043427ad Jocelyn Falempe 2023-05-05 430 WREG32(MGAREG_YBOT, 0x00FFFFFF);
b3cab0043427ad Jocelyn Falempe 2023-05-05 431
b3cab0043427ad Jocelyn Falempe 2023-05-05 432 /* Activate blit mode DMA, only write the low part of the register */
b3cab0043427ad Jocelyn Falempe 2023-05-05 433 WREG8(MGAREG_OPMODE, MGAOPM_DMA_BLIT);
b3cab0043427ad Jocelyn Falempe 2023-05-05 434 }
b3cab0043427ad Jocelyn Falempe 2023-05-05 435
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
next reply other threads:[~2023-05-09 21:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-09 21:54 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-05-05 12:43 [RFC PATCH 0/4] drm/mgag200: Use DMA to copy the framebuffer to the VRAM Jocelyn Falempe
2023-05-05 12:43 ` [PATCH 4/4] " Jocelyn Falempe
2023-05-05 15:01 ` kernel test robot
2023-05-05 15:01 ` kernel test robot
2023-05-05 15:43 ` kernel test robot
2023-05-05 15:43 ` kernel test robot
2023-05-08 8:04 ` Thomas Zimmermann
2023-05-09 9:49 ` Jocelyn Falempe
2023-05-23 6:55 ` Jocelyn Falempe
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=202305100554.Q6f2fDlx-lkp@intel.com \
--to=lkp@intel.com \
--cc=error27@gmail.com \
--cc=oe-kbuild@lists.linux.dev \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.