8 p.m.
Hello Lucas,
On 2/17/22 07:36, Lucas De Marchi wrote:
+Javier
On Thu, Feb 17, 2022 at 02:25:08PM +0800, kernel test robot wrote:
> tree: git://anongit.freedesktop.org/drm/drm-tip drm-tip
> head: e141e36b2871c529379f7ec7d5d6ebae3137a51b
> commit: 7ca6504c36709f35c4cc38ae6acc1c9c3d72136f [4/8] Merge remote-tracking branch
'drm-misc/drm-misc-next' into drm-tip
> config: mips-buildonly-randconfig-r002-20220217
(https://download.01.org/0day-ci/archive/20220217/202202171455.bclm1YBC-lk...)
> compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project
0bad7cb56526f2572c74449fcf97c1fcda42b41d)
> 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 mips cross compiling tool for clang build
> # apt-get install binutils-mips-linux-gnu
> git remote add drm-tip git://anongit.freedesktop.org/drm/drm-tip
> git fetch --no-tags drm-tip drm-tip
> git checkout 7ca6504c36709f35c4cc38ae6acc1c9c3d72136f
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir
ARCH=mips SHELL=/bin/bash drivers/gpu/drm/solomon/
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp(a)intel.com>
>
> All error/warnings (new ones prefixed by >>):
>
>>> drivers/gpu/drm/solomon/ssd130x.c:447:74: warning: declaration of 'struct
dma_buf_map' will not be visible outside of this function [-Wvisibility]
> static int ssd130x_fb_blit_rect(struct drm_framebuffer *fb, const struct
dma_buf_map *map,
> ^
this is now called iosys_map in drm-intel... drm-tip will need a fixup
for the merge.
I thought that the drm-intel tree was only for Intel DRM drivers changes and
subsystem wide changes should be merged through drm-mic ?
Doing refactoring in that tree will likely lead to merge conflicts like this.
Noticed your series in dri-devel but missed that already landed in drm-intel.
The resolution should just be [0] right? If you confirm that then I can post
a proper patch to dri-devel.
>>> drivers/gpu/drm/solomon/ssd130x.c:451:18: error:
incomplete definition of type 'struct dma_buf_map'
> void *vmap = map->vaddr; /* TODO: Use mapping abstraction properly */
> ~~~^
this shouldn't really be done.
Yes, I know but asked what would be the proper way and didn't get an answer.
We have many drivers doing the same and I couldn't find one that was doing
it correctly to use as a reference:
$ git grep "TODO: Use mapping abstraction properly" | wc -l
15
If you point me the proper way, I'll be happy to post a patch to change it.
Lucas De Marchi
[0]
From f8268e5b15c321b56862904665f5e312bf50d397 Mon Sep 17 00:00:00 2001
From: Javier Martinez Canillas <javierm(a)redhat.com>
Date: Thu, 17 Feb 2022 09:52:37 +0100
Subject: [PATCH] drm/solomon: Rename dma-buf-map to iosys-map
Commit 7938f4218168 ("dma-buf-map: Rename to iosys-map") renamed the struct
dma_buf_map to struct iosys_map, but this change wasn't present in drm-misc
when the ssd130x driver was merged, and it created a merge conflict.
Fix this by renaming the data structure type in the ssd130x driver.
Reported-by: kernel test robot <lkp(a)intel.com>
Suggested-by: Lucas De Marchi <lucas.demarchi(a)intel.com>
Signed-off-by: Javier Martinez Canillas <javierm(a)redhat.com>
---
drivers/gpu/drm/solomon/ssd130x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/solomon/ssd130x.c b/drivers/gpu/drm/solomon/ssd130x.c
index 19697c8c5a2c..92c1902f53e4 100644
--- a/drivers/gpu/drm/solomon/ssd130x.c
+++ b/drivers/gpu/drm/solomon/ssd130x.c
@@ -444,7 +444,7 @@ static void ssd130x_clear_screen(struct ssd130x_device *ssd130x)
kfree(buf);
}
-static int ssd130x_fb_blit_rect(struct drm_framebuffer *fb, const struct dma_buf_map
*map,
+static int ssd130x_fb_blit_rect(struct drm_framebuffer *fb, const struct iosys_map *map,
struct drm_rect *rect)
{
struct ssd130x_device *ssd130x = drm_to_ssd130x(fb->dev);
--
2.34.1
Best regards,
--
Javier Martinez Canillas
Linux Engineering
Red Hat