Re: [PATCH v2 -next] media: zoran: convert comma to semicolon

Le Fri, Jan 08, 2021 at 05:21:19PM +0800, Zheng Yongjun a écrit : > Replace a comma between expression statements by a semicolon. > > Signed-off-by: Zheng Yongjun > --- > drivers/staging/media/zoran/zoran_driver.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH -next] media: zoran: use resource_size

Le Wed, Jan 06, 2021 at 05:51:00PM +0300, Dan Carpenter a écrit : > On Wed, Jan 06, 2021 at 09:17:02PM +0800, Zheng Yongjun wrote: > > Use resource_size rather than a verbose computation on > > the end and start fields. > > > > Signed-off-by: Zheng Yongjun > > --- > >

Re: [PATCH] staging:media:zoran: Fixed grammar issue

On Mon, Nov 30, 2020 at 05:06:59PM -0600, Travis Carter wrote: > Removed repeated word 'in' > > Signed-off-by: Travis Carter > --- > drivers/staging/media/zoran/zoran_card.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/media/zoran/zoran_card.c >

Re: [PATCH][next] media: zoran: fix spelling mistake and make error message more meaningful

On Fri, Oct 02, 2020 at 11:16:20AM +0100, Colin King wrote: > From: Colin Ian King > > There is a spelling mistake in a pci_err error message. Fix this and > make the error message a little more meaningful. > > Signed-off-by: Colin Ian King > --- > drivers/staging/media/zoran/zoran_driver.c |

Re: [PATCH] media: zoran.rst: place it at the right place this time

On Sun, Oct 04, 2020 at 06:00:30PM +0200, Mauro Carvalho Chehab wrote: > I was too quick moving zoran.rst... it ends that the original > patch didn't do the right thing and forgot to update the files > that references it. > > Fix it. > > Fixes: 6b90346919d4 ("media: zoran: move documentation

Re: [PATCH 3/3] media: zoran: fix mixed case on vars

On Thu, Oct 01, 2020 at 11:28:15AM +0200, Mauro Carvalho Chehab wrote: > Use this small script to replace CamelCase and wrong case > on vars: > > > FILES=$(find "$1" -type f|grep -e '.c$' -e '.h$') > CAMEL_VARS=$(cat tags|perl -ne 'print "$1\n" if > (m/^(\w*[A-Z]\w*[a-z]\w*)\s/)') > for i in

Re: [PATCH 2/3] media: zoran: get rid of an unused var

On Thu, Oct 01, 2020 at 11:28:14AM +0200, Mauro Carvalho Chehab wrote: > The jpeg_error in lowercase is not used anywhere. Drop it. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/staging/media/zoran/zoran.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git

Re: [PATCH RFT/RFC 37/49] staging: media: zoran: add fallthrough keyword

On Mon, Sep 21, 2020 at 01:59:45PM +0300, Dan Carpenter wrote: > On Mon, Sep 21, 2020 at 10:20:12AM +, Corentin Labbe wrote: > > This patch adds fallthrough keyword where appropriate. > > > > Signed-off-by: Corentin Labbe > > --- > > drivers/staging/media/zoran/zoran_device.c | 6 ++ > >

Re: [PATCH RFT/RFC 01/49] staging: media: Revert "media: zoran: remove deprecated driver"

On Tue, Sep 22, 2020 at 06:16:01AM +0100, Christoph Hellwig wrote: > > + fh->buffers.buffer[i].v4l.fbuffer = mem; > > + fh->buffers.buffer[i].v4l.fbuffer_phys = virt_to_phys(mem); > > + fh->buffers.buffer[i].v4l.fbuffer_bus = virt_to_bus(mem); > > + for (off

Re: [PATCH] staging: media: atomisp2: remove unused headers

On Mon, Jan 29, 2018 at 03:22:33PM +0200, Andy Shevchenko wrote: > On Mon, Jan 29, 2018 at 3:11 PM, Corentin Labbe wrote: > > All thoses headers are not used by any source files. > > Lets just remove them. > > How did you test this? > > P.S. I like the patch, but since

Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig

On Tue, Jan 23, 2018 at 07:20:12PM +0100, Greg Kroah-Hartman wrote: > On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe wrote: > > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 > > >

[PATCH] staging: lustre: Annotate user buffer with __user

Fix the warning reported by sparse on drivers/staging/lustre/lnet/lnet/router_proc.c warning: incorrect type in argument 4 (different address spaces) The correction is to annotate all user buffer variable by __user Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- .../staging/lustre

[PATCH 1/4] Solves some problems reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 75ca15e..177e4b9 100644 --- a/drivers/staging/sep

[PATCH 4/4] Fix blank lines issue reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 47 -- 1 file changed, 4 insertions(+), 43 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 6f98881..89a1d53 100644

[PATCH 3/4] Fix misceanellous problems reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 1580d95f..6f98881 100644 --- a/drivers/staging/sep/sep_main.c

[PATCH 2/4] No else is necessary after a break (reported by checkpatch)

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 177e4b9..1580d95f 100644 --- a/drivers/staging/sep/sep_main.c

[PATCH] staging: sep: Fix issues reported by checkpatch

Hello The drivers/staging/sep/sep_main.c have lots of checkpatch issue. This patch series solves them and since there are many, I have splitted corrections in 4 patch. Note that this work is done for the Eudyptulla Challenge. Best regards ___ devel

[PATCH 2/4] staging: sep: No else is necessary after a break (reported by checkpatch)

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 177e4b9..1580d95f 100644 --- a/drivers/staging/sep/sep_main.c

[PATCH 3/4] staging: sep: Fix misceanellous problems reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 1580d95f..6f98881 100644 --- a/drivers/staging/sep/sep_main.c

[PATCH 1/4] staging: sep: Solves some problems reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 75ca15e..177e4b9 100644 --- a/drivers/staging/sep

[PATCH 4/4] staging: sep: Fix blank lines issue reported by checkpatch

Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com --- drivers/staging/sep/sep_main.c | 47 -- 1 file changed, 4 insertions(+), 43 deletions(-) diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep/sep_main.c index 6f98881..89a1d53 100644