From: Daniel Vetter <daniel@ffwll.ch> To: kbuild test robot <lkp@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>, kbuild-all@lists.01.org, DRI Development <dri-devel@lists.freedesktop.org>, Daniel Vetter <daniel.vetter@intel.com> Subject: Re: [PATCH 2/3] drm/hibmc: Use drm_gem_fb_create Date: Thu, 28 Nov 2019 11:14:54 +0100 [thread overview] Message-ID: <20191128101454.GR406127@phenom.ffwll.local> (raw) In-Reply-To: <201911281634.x83rjGQT%lkp@intel.com> On Thu, Nov 28, 2019 at 04:44:32PM +0800, kbuild test robot wrote: > Hi Daniel, > > I love your patch! Yet something to improve: > > [auto build test ERROR on drm-intel/for-linux-next] > [also build test ERROR on v5.4 next-20191127] > [if your patch is applied to the wrong git tree, please drop us a note to help > improve the system. BTW, we also suggest to use '--base' option to specify the > base tree in git format-patch, please see https://stackoverflow.com/a/37406982] > > url: https://github.com/0day-ci/linux/commits/Daniel-Vetter/drm-rockchip-Use-drm_gem_fb_create_with_dirty/20191128-023917 > base: git://anongit.freedesktop.org/drm-intel for-linux-next > config: arm64-randconfig-a001-20191128 (attached as .config) > compiler: aarch64-linux-gcc (GCC) 7.4.0 > reproduce: > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > GCC_VERSION=7.4.0 make.cross ARCH=arm64 > > If you fix the issue, kindly add following tag > Reported-by: kbuild test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): Oops, I meant to drop this patch from this series, but forgot. It's superseeded by the series Thomas has (which actually compiles). -Daniel > > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c: In function 'hibmc_plane_atomic_update': > >> drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c:107:28: error: 'fb' undeclared (first use in this function); did you mean 'mb'? > gbo = drm_gem_vram_of_gem(fb->obj[0]); > ^~ > mb > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c:107:28: note: each undeclared identifier is reported only once for each function it appears in > -- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c: In function 'hibmc_drm_fb_create': > >> drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c:119:37: error: 'struct drm_framebuffer' has no member named 'fb' > hi_fbdev->helper.fb = &hi_fbdev->fb->fb; > ^~ > > vim +107 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > > 93 > 94 static void hibmc_plane_atomic_update(struct drm_plane *plane, > 95 struct drm_plane_state *old_state) > 96 { > 97 struct drm_plane_state *state = plane->state; > 98 u32 reg; > 99 s64 gpu_addr = 0; > 100 unsigned int line_l; > 101 struct hibmc_drm_private *priv = plane->dev->dev_private; > 102 struct drm_gem_vram_object *gbo; > 103 > 104 if (!state->fb) > 105 return; > 106 > > 107 gbo = drm_gem_vram_of_gem(fb->obj[0]); > 108 > 109 gpu_addr = drm_gem_vram_offset(gbo); > 110 if (WARN_ON_ONCE(gpu_addr < 0)) > 111 return; /* Bug: we didn't pin the BO to VRAM in prepare_fb. */ > 112 > 113 writel(gpu_addr, priv->mmio + HIBMC_CRT_FB_ADDRESS); > 114 > 115 reg = state->fb->width * (state->fb->format->cpp[0]); > 116 /* now line_pad is 16 */ > 117 reg = PADDING(16, reg); > 118 > 119 line_l = state->fb->width * state->fb->format->cpp[0]; > 120 line_l = PADDING(16, line_l); > 121 writel(HIBMC_FIELD(HIBMC_CRT_FB_WIDTH_WIDTH, reg) | > 122 HIBMC_FIELD(HIBMC_CRT_FB_WIDTH_OFFS, line_l), > 123 priv->mmio + HIBMC_CRT_FB_WIDTH); > 124 > 125 /* SET PIXEL FORMAT */ > 126 reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); > 127 reg &= ~HIBMC_CRT_DISP_CTL_FORMAT_MASK; > 128 reg |= HIBMC_FIELD(HIBMC_CRT_DISP_CTL_FORMAT, > 129 state->fb->format->cpp[0] * 8 / 16); > 130 writel(reg, priv->mmio + HIBMC_CRT_DISP_CTL); > 131 } > 132 > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel@ffwll.ch> To: kbuild-all@lists.01.org Subject: Re: [PATCH 2/3] drm/hibmc: Use drm_gem_fb_create Date: Thu, 28 Nov 2019 11:14:54 +0100 [thread overview] Message-ID: <20191128101454.GR406127@phenom.ffwll.local> (raw) In-Reply-To: <201911281634.x83rjGQT%lkp@intel.com> [-- Attachment #1: Type: text/plain, Size: 4066 bytes --] On Thu, Nov 28, 2019 at 04:44:32PM +0800, kbuild test robot wrote: > Hi Daniel, > > I love your patch! Yet something to improve: > > [auto build test ERROR on drm-intel/for-linux-next] > [also build test ERROR on v5.4 next-20191127] > [if your patch is applied to the wrong git tree, please drop us a note to help > improve the system. BTW, we also suggest to use '--base' option to specify the > base tree in git format-patch, please see https://stackoverflow.com/a/37406982] > > url: https://github.com/0day-ci/linux/commits/Daniel-Vetter/drm-rockchip-Use-drm_gem_fb_create_with_dirty/20191128-023917 > base: git://anongit.freedesktop.org/drm-intel for-linux-next > config: arm64-randconfig-a001-20191128 (attached as .config) > compiler: aarch64-linux-gcc (GCC) 7.4.0 > reproduce: > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > GCC_VERSION=7.4.0 make.cross ARCH=arm64 > > If you fix the issue, kindly add following tag > Reported-by: kbuild test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): Oops, I meant to drop this patch from this series, but forgot. It's superseeded by the series Thomas has (which actually compiles). -Daniel > > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c: In function 'hibmc_plane_atomic_update': > >> drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c:107:28: error: 'fb' undeclared (first use in this function); did you mean 'mb'? > gbo = drm_gem_vram_of_gem(fb->obj[0]); > ^~ > mb > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c:107:28: note: each undeclared identifier is reported only once for each function it appears in > -- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c: In function 'hibmc_drm_fb_create': > >> drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c:119:37: error: 'struct drm_framebuffer' has no member named 'fb' > hi_fbdev->helper.fb = &hi_fbdev->fb->fb; > ^~ > > vim +107 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > > 93 > 94 static void hibmc_plane_atomic_update(struct drm_plane *plane, > 95 struct drm_plane_state *old_state) > 96 { > 97 struct drm_plane_state *state = plane->state; > 98 u32 reg; > 99 s64 gpu_addr = 0; > 100 unsigned int line_l; > 101 struct hibmc_drm_private *priv = plane->dev->dev_private; > 102 struct drm_gem_vram_object *gbo; > 103 > 104 if (!state->fb) > 105 return; > 106 > > 107 gbo = drm_gem_vram_of_gem(fb->obj[0]); > 108 > 109 gpu_addr = drm_gem_vram_offset(gbo); > 110 if (WARN_ON_ONCE(gpu_addr < 0)) > 111 return; /* Bug: we didn't pin the BO to VRAM in prepare_fb. */ > 112 > 113 writel(gpu_addr, priv->mmio + HIBMC_CRT_FB_ADDRESS); > 114 > 115 reg = state->fb->width * (state->fb->format->cpp[0]); > 116 /* now line_pad is 16 */ > 117 reg = PADDING(16, reg); > 118 > 119 line_l = state->fb->width * state->fb->format->cpp[0]; > 120 line_l = PADDING(16, line_l); > 121 writel(HIBMC_FIELD(HIBMC_CRT_FB_WIDTH_WIDTH, reg) | > 122 HIBMC_FIELD(HIBMC_CRT_FB_WIDTH_OFFS, line_l), > 123 priv->mmio + HIBMC_CRT_FB_WIDTH); > 124 > 125 /* SET PIXEL FORMAT */ > 126 reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); > 127 reg &= ~HIBMC_CRT_DISP_CTL_FORMAT_MASK; > 128 reg |= HIBMC_FIELD(HIBMC_CRT_DISP_CTL_FORMAT, > 129 state->fb->format->cpp[0] * 8 / 16); > 130 writel(reg, priv->mmio + HIBMC_CRT_DISP_CTL); > 131 } > 132 > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
next prev parent reply other threads:[~2019-11-28 10:14 UTC|newest] Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-11-27 18:00 [PATCH 1/3] drm/rockchip: Use drm_gem_fb_create_with_dirty Daniel Vetter 2019-11-27 18:00 ` Daniel Vetter 2019-11-27 18:00 ` [PATCH 2/3] drm/hibmc: Use drm_gem_fb_create Daniel Vetter 2019-11-28 8:44 ` kbuild test robot 2019-11-28 8:44 ` kbuild test robot 2019-11-28 8:44 ` kbuild test robot 2019-11-28 10:14 ` Daniel Vetter [this message] 2019-11-28 10:14 ` Daniel Vetter 2019-11-27 18:00 ` [PATCH 3/3] drm/todo: Add entry for fb funcs related cleanups Daniel Vetter 2019-11-29 9:34 ` Thomas Zimmermann 2019-11-29 18:57 ` Daniel Vetter 2019-11-29 19:05 ` Daniel Vetter 2019-12-02 8:42 ` Thomas Zimmermann 2019-11-28 15:58 ` [PATCH 1/3] drm/rockchip: Use drm_gem_fb_create_with_dirty Andrzej Pietrasiewicz 2019-11-28 15:58 ` Andrzej Pietrasiewicz 2019-11-29 8:59 ` Daniel Vetter 2019-11-29 8:59 ` Daniel Vetter
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=20191128101454.GR406127@phenom.ffwll.local \ --to=daniel@ffwll.ch \ --cc=daniel.vetter@ffwll.ch \ --cc=daniel.vetter@intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=kbuild-all@lists.01.org \ --cc=lkp@intel.com \ /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: linkBe 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.