Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) produced these warnings:

drivers/gpu/drm/bochs/bochs_fbdev.c: In function 'bochsfb_create': drivers/gpu/drm/bochs/bochs_fbdev.c:63:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &dev->pdev->dev; ^

Caused by commit

394111a2b303 ("drm/boschs: Use new drm_fb_helper functions")

drivers/gpu/drm/cirrus/cirrus_fbdev.c: In function 'cirrusfb_create': drivers/gpu/drm/cirrus/cirrus_fbdev.c:247:1: warning: label 'out_iounmap' defined but not used [-Wunused-label] out_iounmap: ^

Caused by commit fd6ab591ce15 ("drm/cirrus: Use new drm_fb_helper functions")

drivers/gpu/drm/ast/ast_fb.c: In function 'astfb_create': drivers/gpu/drm/ast/ast_fb.c:196:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &dev->pdev->dev; ^

Caused by commit

6f4900cd28c9 ("drm/ast: Use new drm_fb_helper functions")

drivers/gpu/drm/gma500/framebuffer.c: In function 'psbfb_create': drivers/gpu/drm/gma500/framebuffer.c:346:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &dev->pdev->dev; ^

Caused by commit

3b9a13e85365 ("drm/gma500: Use new drm_fb_helper functions")

drivers/gpu/drm/mgag200/mgag200_fb.c: In function 'mgag200fb_create': drivers/gpu/drm/mgag200/mgag200_fb.c:232:1: warning: label 'out' defined but not used [-Wunused-label] out: ^ drivers/gpu/drm/mgag200/mgag200_fb.c:169:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &dev->pdev->dev; ^

Caused by commit

53ebb642ccba ("drm/mgag200: Use new drm_fb_helper functions")

drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c: In function 'amdgpufb_create': drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c:206:9: warning: passing argument 1 of 'PTR_ERR' makes pointer from integer without a cast ret = PTR_ERR(ret); ^ In file included from arch/x86/include/asm/processor.h:31:0, from arch/x86/include/asm/thread_info.h:49, from include/linux/thread_info.h:54, from arch/x86/include/asm/preempt.h:6, from include/linux/preempt.h:64, from include/linux/spinlock.h:50, from include/linux/seqlock.h:35, from include/linux/time.h:5, from include/linux/stat.h:18, from include/linux/module.h:10, from drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c:26: include/linux/err.h:28:33: note: expected 'const void *' but argument is of type 'int' static inline long __must_check PTR_ERR(__force const void *ptr) ^ drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c:182:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &adev->pdev->dev; ^ drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c: In function 'amdgpu_fbdev_destroy': drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c:283:18: warning: unused variable 'info' [-Wunused-variable] struct fb_info *info; ^

Caused by commit

9d70561ba3b1 ("drm/amdgpu: Use new drm_fb_helper functions")

drivers/gpu/drm/udl/udl_fb.c: In function 'udlfb_create': drivers/gpu/drm/udl/udl_fb.c:479:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = dev->dev; ^

Caused by commit

457959f2ed5f ("drm/udl: Use new drm_fb_helper functions")

drivers/gpu/drm/radeon/radeon_fb.c: In function 'radeonfb_create': drivers/gpu/drm/radeon/radeon_fb.c:230:17: warning: unused variable 'device' [-Wunused-variable] struct device *device = &rdev->pdev->dev; ^

Caused by commit

4bc29bb98bae ("drm/radeon: Use new drm_fb_helper functions")

drivers/gpu/drm/nouveau/nouveau_fbcon.c: In function 'nouveau_fbcon_create': drivers/gpu/drm/nouveau/nouveau_fbcon.c:322:18: warning: unused variable 'pdev' [-Wunused-variable] struct pci_dev *pdev = dev->pdev; ^

Caused by commit

237fc6452ca5 ("drm/nouveau: Use new drm_fb_helper functions")

I think "COMPILE TESTED ONLY" should at least include eliminating such warnings ...