[Mjpeg-users] [PATCH 3/8] staging: media: zoran: rename debug module parameter

2021-09-03 Thread Corentin Labbe
All zoran module will be merged, so to prevent conflict, the debug module parameter need to be renamed Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/videocodec.c | 8 drivers/staging/media/zoran/zr36016.c| 12 ++-- drivers/staging/media/zoran/zr36050.c|

[Mjpeg-users] [PATCH 7/8] staging: media: zoran: remove vidmem

2021-09-03 Thread Corentin Labbe
The vidmem parameter is no longer necessary since we removed framebuffer support. Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/zoran_card.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/media/zoran/zoran_card.c b/drivers/staging/media/zoran

[Mjpeg-users] [PATCH 2/8] staging: media: zoran: use module_pci_driver

2021-09-03 Thread Corentin Labbe
Simplify code by using module_pci_driver() Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/zoran_card.c | 21 + 1 file changed, 1 insertion(+), 20 deletions(-) diff --git a/drivers/staging/media/zoran/zoran_card.c b/drivers/staging/media/zoran/zoran_card.c ind

Re: [Mjpeg-users] [PATCH 4/8] staging: media: zoran: add debugfs

2021-09-03 Thread kernel test robot
Hi Corentin, I love your patch! Perhaps something to improve: [auto build test WARNING on staging/staging-testing] url: https://github.com/0day-ci/linux/commits/Corentin-Labbe/staging-media-zoran-fusion-in-one-module/20210904-031844 base: https://git.kernel.org/pub/scm/linux/kernel/git/gre

[Mjpeg-users] [PATCH 6/8] staging: media: zoran: fusion all modules

2021-09-03 Thread Corentin Labbe
The zoran driver is split in many modules, but this lead to some problems. One of them is that load order is incorrect when everything is built-in. Having more than one module is useless, so fusion all zoran modules in one. Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/Kconfig

[Mjpeg-users] [PATCH 1/8] staging: media: zoran: move module parameter checks to zoran_probe

2021-09-03 Thread Corentin Labbe
We need to empty zoran_init() for removing it later. Furthermore, this permit to use pci_xxx instead of pr_xxx for prettier printing. Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/zoran_card.c | 64 1 file changed, 33 insertions(+), 31 deletions(-) diff

[Mjpeg-users] [PATCH 8/8] staging: media: zoran: move videodev alloc

2021-09-03 Thread Corentin Labbe
Move some code out of zr36057_init() and create new functions for handling zr->video_dev. This permit to ease code reading and fix a zr->video_dev memory leak. Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/zoran.h| 2 +- drivers/staging/media/zoran/zoran_card.c | 80 +

[Mjpeg-users] [PATCH 4/8] staging: media: zoran: add debugfs

2021-09-03 Thread Corentin Labbe
Add debugfs for displaying zoran debug and stats information. Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/Kconfig | 10 ++ drivers/staging/media/zoran/zoran.h | 5 +++ drivers/staging/media/zoran/zoran_card.c | 39 3 files changed, 54 ins

[Mjpeg-users] [PATCH 5/8] staging: media: zoran: videocode: remove procfs

2021-09-03 Thread Corentin Labbe
Now we have a debugfs, we can remove all PROCFS stuff. We keep videocodec_debugfs_show(), it will be used later Signed-off-by: Corentin Labbe --- drivers/staging/media/zoran/videocodec.c | 24 ++-- drivers/staging/media/zoran/videocodec.h | 5 + 2 files changed, 7 insert

[Mjpeg-users] [PATCH 0/8] staging: media: zoran: fusion in one module

2021-09-03 Thread Corentin Labbe
Hello The main change of this serie is to fusion all zoran related modules in one. This fixes the load order problem when everything is built-in. Regards Corentin Labbe (8): staging: media: zoran: move module parameter checks to zoran_probe staging: media: zoran: use module_pci_driver stag