Re: [PATCH 2/7] fbdev: Do not include in header

2024-02-12 Thread kernel test robot
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on linus/master v6.8-rc4 next-20240212] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

[linux-next:master] BUILD REGRESSION ae00c445390b349e070a64dc62f08aa878db7248

2024-02-12 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: ae00c445390b349e070a64dc62f08aa878db7248 Add linux-next specific files for 20240212 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202402122047.ydhrzmm4-...@intel.com https

[PATCH AUTOSEL 6.7 35/58] drm/nouveau: nvkm_gsp_radix3_sg() should use nvkm_gsp_mem_ctor()

2024-02-12 Thread Sasha Levin
From: Timur Tabi [ Upstream commit 34e659f34a7559ecfd9c1f5b24d4c291f3f54711 ] Function nvkm_gsp_radix3_sg() uses nvkm_gsp_mem objects to allocate the radix3 tables, but it unnecessarily creates those objects manually instead of using the standard nvkm_gsp_mem_ctor() function like the rest of

[PATCH 1/2] [v3] nouveau: add command-line GSP-RM registry support

2024-02-12 Thread Timur Tabi
Add the NVreg_RegistryDwords command line parameter, which allows specifying additional registry keys to be sent to GSP-RM. This allows additional configuration, debugging, and experimentation with GSP-RM, which uses these keys to alter its behavior. Note that these keys are passed as-is to

[PATCH 2/2] [v3] drm/nouveau: expose GSP-RM logging buffers via debugfs

2024-02-12 Thread Timur Tabi
The LOGINIT, LOGINTR, LOGRM, and LOGPMU buffers are circular buffers that have printf-like logs from GSP-RM and PMU encoded in them. LOGINIT, LOGINTR, and LOGRM are allocated by Nouveau and their DMA addresses are passed to GSP-RM during initialization. The buffers are required for GSP-RM to

[drm-next][PATCH 0/2] command-line registry and gsp-rm logging

2024-02-12 Thread Timur Tabi
Two patches that were previosly posted, but now updated for drm-next. Timur Tabi (2): [v3] nouveau: add command-line GSP-RM registry support [v3] drm/nouveau: expose GSP-RM logging buffers via debugfs .../gpu/drm/nouveau/include/nvkm/subdev/gsp.h | 18 +

Re: [PATCH 2/7] fbdev: Do not include in header

2024-02-12 Thread kernel test robot
Hi Thomas, kernel test robot noticed the following build errors: [auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on linus/master v6.8-rc4 next-20240212] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use

Re: [PATCH 2/7] fbdev: Do not include in header

2024-02-12 Thread kernel test robot
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on linus/master v6.8-rc4 next-20240212] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

Re: [PATCH] nouveau/svm: fix kvcalloc() argument order

2024-02-12 Thread Danilo Krummrich
On 2/12/24 12:22, Arnd Bergmann wrote: From: Arnd Bergmann The conversion to kvcalloc() mixed up the object size and count arguments, causing a warning: drivers/gpu/drm/nouveau/nouveau_svm.c: In function 'nouveau_svm_fault_buffer_ctor': drivers/gpu/drm/nouveau/nouveau_svm.c:1010:40: error:

[PATCH] nouveau/svm: fix kvcalloc() argument order

2024-02-12 Thread Arnd Bergmann
From: Arnd Bergmann The conversion to kvcalloc() mixed up the object size and count arguments, causing a warning: drivers/gpu/drm/nouveau/nouveau_svm.c: In function 'nouveau_svm_fault_buffer_ctor': drivers/gpu/drm/nouveau/nouveau_svm.c:1010:40: error: 'kvcalloc' sizes specified with 'sizeof'

Re: [PATCH 0/7] fbdev: Clean up include dependencies in header

2024-02-12 Thread Jani Nikula
On Mon, 12 Feb 2024, Thomas Zimmermann wrote: > Remove unnecessary dependencies in the include statements of the > header file . Several files throughout the kernel include > the fbdev header, so reducing dependencies positively affects other > subsystems as well. Also fix up nouveau, which needs

[PATCH 3/7] fbdev: Do not include in header

2024-02-12 Thread Thomas Zimmermann
Forward declare struct inode and remove the include statement. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/fb.h b/include/linux/fb.h index 7380d959c5d53..f269ba5202809 100644 --- a/include/linux/fb.h

[PATCH 5/7] fbdev: Do not include in header

2024-02-12 Thread Thomas Zimmermann
Forward declare struct page and remove the include statement. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/fb.h b/include/linux/fb.h index 90f348f14a490..42155898374b1 100644 --- a/include/linux/fb.h

[PATCH 7/7] fbdev: Clean up include statements in header file

2024-02-12 Thread Thomas Zimmermann
Include mutex.h, printk.h and types.h, remove several unnecessary include statements, and sort the list alphabetically. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/linux/fb.h b/include/linux/fb.h

[PATCH 0/7] fbdev: Clean up include dependencies in header

2024-02-12 Thread Thomas Zimmermann
Remove unnecessary dependencies in the include statements of the header file . Several files throughout the kernel include the fbdev header, so reducing dependencies positively affects other subsystems as well. Also fix up nouveau, which needs backlight.h in one of its source files. Thomas

[PATCH 6/7] fbdev: Clean up forward declarations in header file

2024-02-12 Thread Thomas Zimmermann
Add forward declarations for struct i2c_adapter and struct module, and sort the list alphabetically. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/include/linux/fb.h b/include/linux/fb.h index

[PATCH 1/7] drm/nouveau: Include

2024-02-12 Thread Thomas Zimmermann
Resolved the proxy include via , which does not require the backlight header. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c index

[PATCH 4/7] fbdev: Do not include in header

2024-02-12 Thread Thomas Zimmermann
Forward declare struct notifier_block and remove the include statement. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/fb.h b/include/linux/fb.h index f269ba5202809..90f348f14a490 100644 ---

[PATCH 2/7] fbdev: Do not include in header

2024-02-12 Thread Thomas Zimmermann
Forward declare struct backlight_device and remove the include statement. Signed-off-by: Thomas Zimmermann --- include/linux/fb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/fb.h b/include/linux/fb.h index 2ce2f5c2fca9a..7380d959c5d53 100644 ---