Re: [PATCH v2] media:dvb-frontends:Remove dvb_pll_devcount and id module parameters.

On 01/08/19 2:31 PM, Michael Ira Krufky wrote: > Sean, > > Please pardon the late reply. See my responses inline below: > > On Wed, Jul 24, 2019 at 1:36 AM Sean Young wrote: >> On Sat, Jul 20, 2019 at 11:43:02AM +0530, Vandana BN wrote: >>> Syzbot reporte

Re: [PATCH v2] media:dvb-frontends:Remove dvb_pll_devcount and id module parameters.

On 01/08/19 10:39 AM, Sean Young wrote: > On Wed, Jul 24, 2019 at 06:36:35AM +0100, Sean Young wrote: >> On Sat, Jul 20, 2019 at 11:43:02AM +0530, Vandana BN wrote: >>> Syzbot reported global-out-of-bounds Read in dvb_pll_attach, while >>> accessing id[dvb_pll_devcount

[PATCH v2] media:dvb-frontends:Remove dvb_pll_devcount and id module parameters.

0 00 00 00 00 00 00 00 00 00 00 fa fa fa fa ^ 89c9e600: 04 fa fa fa fa fa fa fa 04 fa fa fa fa fa fa fa 89c9e680: 04 fa fa fa fa fa fa fa 04 fa fa fa fa fa fa fa ====== Signed-off-by: Vandana BN ---

[PATCH] media:dvb-frontends:Return if Max devices are added in dvb_pll_attach().

========= Signed-off-by: Vandana BN --- drivers/media/dvb-frontends/dvb-pll.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/dvb-frontends/dvb-pll.c b/drivers/media/dvb-frontends/dvb-pll.c index ba0c49107bd2..c850f1d69bce 100644 --- a/drivers/media/dvb-fronte

[PATCH] net:gue.h:Fix shifting signed 32-bit value by 31 bits problem

Fix GUE_PFLAG_REMCSUM to use "U" cast to avoid shifting signed 32-bit value by 31 bits problem. Signed-off-by: Vandana BN --- include/net/gue.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/net/gue.h b/include/net/gue.h index fdad41469b65..3a6595bfa641 10

[PATCH] net: dst.h: Fix shifting signed 32-bit value by 31 bits problem

Fix DST_FEATURE_ECN_CA to use "U" cast to avoid shifting signed 32-bit value by 31 bits problem. Signed-off-by: Vandana BN --- include/net/dst.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/net/dst.h b/include/net/dst.h index 12b31c602cb0..095c5daf9

Re: [PATCH v2] Documentation:sh:convert register-banks.txt and new-machine.txt to rst format.

On 28/06/19 7:09 PM, Mauro Carvalho Chehab wrote: > Em Fri, 28 Jun 2019 18:54:59 +0530 > Vandana BN escreveu: > >> This patch converts new-machine.txt and register-banks.txt to ReST format, >> No content >> change. >> Added new-machine.rst and register-banks

[PATCH v2] media:vivid:add sanity check to avoid divide error and set value to 1 if 0.

00() GS:8880ae80() knlGS: kobject: 'loop5' (a41f9e79): kobject_uevent_env CS: 0010 DS: ES: CR0: 80050033 kobject: 'queues' (00007ed20666): kobject_uevent_env: filter function caused the event to drop! CR2: 004

Re: [PATCH] media:vivid: add sanity check to avoid divide error and set value to 1 if 0.

On 27/06/19 12:58 PM, Hans Verkuil wrote: > On 6/26/19 8:01 AM, Vandana BN wrote: >> Syzbot reported divide error in vivid_thread_vid_cap, which has been >> seen only once and doesnot have a reproducer. >> This patch sanity checks for the denominator value with WARN_

[PATCH] media:vivid: add sanity check to avoid divide error and set value to 1 if 0.

00() GS:8880ae80() knlGS: kobject: 'loop5' (a41f9e79): kobject_uevent_env CS: 0010 DS: ES: CR0: 80050033 kobject: 'queues' (00007ed20666): kobject_uevent_env: filter function caused the event to drop! CR2: 000

[PATCH v2] media:usb:zr364xx:Fix KASAN:null-ptr-deref Read in zr364xx_vidioc_querycap

012fd4c531a1...@syzkaller.appspotmail.com Signed-off-by: Vandana BN --- v2 - removed else part as cap->card is already zeroed. drivers/media/usb/zr364xx/zr364xx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/usb/zr364xx/zr364xx.c b/drivers/media/usb/zr364xx/

[PATCH] media:usb:zr364xx:Fix KASAN:null-ptr-deref Read in zr364xx_vidioc_querycap

012fd4c531a1...@syzkaller.appspotmail.com Signed-off-by: Vandana BN --- drivers/media/usb/zr364xx/zr364xx.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/media/usb/zr364xx/zr364xx.c b/drivers/media/usb/zr364xx/zr364xx.c index 96fee8d5b865..401a1e55dbe1 100644 --- a/drive

[PATCH v4 6/8] Staging: kpc2000: kpc_dma: Resolve warning Missing blank line after declarations & labels not to be indented.

ff-by: Vandana BN --- v2 - split changes to multiple patches v3 - edit commit message, subject line v4 - edit commit message drivers/staging/kpc2000/kpc_dma/dma.c| 8 +--- drivers/staging/kpc2000/kpc_dma/fileops.c| 11 ++- drivers/staging/kpc2000/kpc_dma/kpc_dma_dri

[PATCH v4 7/8] Staging: kpc2000: kpc_dma: Resolve warning to use __func__ insted of funtion name reported by checkpatch.

This patch resolves warnings to use __func__ insted of funtion name. WARNING: Prefer using '"%s...", __func__' to using 'setup_dma_engine', this function's name, in a string Signed-off-by: Vandana BN --- v2 - split changes to multiple patches v3 - edit com

[PATCH v4 8/8] Staging: kpc2000: kpc_dma: Resolve cast warning and use const for file_operation

This Patch resolves unnecessary cast warning and const file_operations reported by checkpath.pl WARNING: unnecessary cast may hide bugs WARNING: struct file_operations should normally be const Signed-off-by: Vandana BN --- v2 - split changes to multiple patches v3 - edit commit message, subject

[PATCH v4 5/8] Staging: kpc2000: kpc_dma: Resolve checkpath errors Macros in paranthesis & trailing statements on next line.

This patch fixes below errors reported by checkpath ERROR: Macros with complex values should be enclosed in parentheses CHECK: Prefer using the BIT macro ERROR: trailing statements should be on next line ERROR: trailing statements should be on next line Signed-off-by: Vandana BN --- v2 - split

[PATCH v4 3/8] Staging: kpc2000: kpc_dma: Resolve checkpatch space errors around brace '{','!' and open paranthesis '('.

This patch resolves below errors reported by checkpath ERROR: space required before the open brace '{' ERROR: space prohibited after that '!' (ctx:BxW) ERROR: space prohibited after that open parenthesis '(' Signed-off-by: Vandana BN --- v2 - split changes to mul

[PATCH v4 4/8] Staging: kpc2000: kpc_dma: Resolve code indent and trailing statements on next line errors reported by checkpatch.

This patch fixes code indentaion error reported by checkpath ERROR: switch and case should be at the same indent ERROR: trailing statements should be on next line Signed-off-by: Vandana BN --- v2 - split changes to multiple patches v3 - edit commit message, subject line v4 - edit commit message

[PATCH v4 2/8] Staging: kpc2000: kpc_dma: Resolve space errors around pointers and function declarations reported by checkpatch.

This patch resolves below errors reported by checkpatch ERROR: "(foo*)" should be "(foo *)" ERROR: "foo * bar" should be "foo *bar" ERROR: "foo __init bar" should be "foo __init bar" ERROR: "foo __exit bar" should be "foo

[PATCH v4 1/8] Staging: kpc2000: kpc_dma: Resolve trailing whitespace error reported by checkpatch

Resolve trailing whitespace error from checkpatch.pl ERROR: trailing whitespace Signed-off-by: Vandana BN --- v2 - split changes to multiple patches v3 - edit commit message, subject line v4 - edit commit message drivers/staging/kpc2000/kpc_dma/dma.c | 86 ++--- drivers

Re: [PATCH v3 1/8] Staging: kpc2000: kpc_dma: Resolve trailing whitespace error reported by checkpatch

oh ok.. thanks i will correct it. On 13/05/19 4:19 PM, Dan Carpenter wrote: > The Signed off by has to be before the first --- cut off line. > Everything after the cut off is removed from the commit message.

Re: [PATCH v2 2/8] Staging: kpc2000: kpc_dma: Resolve coding style errors reported by checkpatch.

On 13/05/19 2:12 PM, Greg KH wrote: > On Mon, May 13, 2019 at 05:09:54AM +0530, Vandana BN wrote: >> This patch resolves below errors reported by checkpatch >> ERROR: "(foo*)" should be "(foo *)" >> ERROR: "foo * bar" should be "foo *ba

[PATCH v3 3/8] Staging: kpc2000: kpc_dma: Resolve checkpatch space errors around brace '{','!' and open paranthesis '('.

mit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c | 28 +- drivers/staging/kpc2000/kpc_dma/fileops.c | 56 +-- .../staging/kpc2000/kpc_dma/kpc_dma_driver.c | 20 +++ 3 files changed, 52 insertions(+), 52 deletions(-) diff -

[PATCH v3 4/8] Staging: kpc2000: kpc_dma: Resolve code indent and trailing statements on next line errors reported by checkpatch.

This patch fixes code indentaion error reported by checkpath ERROR: switch and case should be at the same indent ERROR: trailing statements should be on next line --- v2 - split changes to multiple patches v3 - edit commit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000

[PATCH v3 7/8] Staging: kpc2000: kpc_dma: Resolve warning to use __func__ insted of funtion name reported by checkpatch.

This patch resolves warnings to use __func__ insted of funtion name. WARNING: Prefer using '"%s...", __func__' to using 'setup_dma_engine', this function's name, in a string --- v2 - split changes to multiple patches v3 - edit commit message --- Signed-off-b

[PATCH v3 8/8] Staging: kpc2000: kpc_dma: Resolve cast warning and use const for file_operation

This Patch resolves unnecessary cast warning and const file_operations reported by checkpath.pl WARNING: unnecessary cast may hide bugs WARNING: struct file_operations should normally be const --- v2 - split changes to multiple patches v3 - edit commit message --- Signed-off-by: Vandana BN

[PATCH v3 6/8] Staging: kpc2000: kpc_dma: Resolve warning Missing blank line after declarations & labels not to be indented.

split changes to multiple patches v3 - edit commit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c| 8 +--- drivers/staging/kpc2000/kpc_dma/fileops.c| 11 ++- drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c | 15 --- dr

[PATCH v3 5/8] Staging: kpc2000: kpc_dma: Resolve checkpath errors Macros in paranthesis & trailing statements on next line.

v3 - edit commit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c b/drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c index

[PATCH v3 1/8] Staging: kpc2000: kpc_dma: Resolve trailing whitespace error reported by checkpatch

Resolve trailing whitespace error from checkpatch.pl ERROR: trailing whitespace --- v2-split changes to multiple patches v3 - edit commit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c | 86 ++--- drivers/staging/kpc2000/kpc_dma/fileops.c

[PATCH v3 2/8] Staging: kpc2000: kpc_dma: Resolve space errors around pointers and function declarations reported by checkpatch.

foo __exit bar" --- v2 - split changes to multiple patches v3 - edit commit message --- Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c| 8 drivers/staging/kpc2000/kpc_dma/fileops.c| 2 +- drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c |

[PATCH v2 1/8] Staging: kpc2000: kpc_dma: resolve trailing whitespace error reported by checkpatch

Resolve trailing whitespace error from checkpatch.pl ERROR: trailing whitespace Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c | 86 ++--- drivers/staging/kpc2000/kpc_dma/fileops.c | 114 +- .../staging/kpc2000/kpc_dma/kpc_dma_driver.c

[PATCH v2 2/8] Staging: kpc2000: kpc_dma: Resolve coding style errors reported by checkpatch.

This patch resolves below errors reported by checkpatch ERROR: "(foo*)" should be "(foo *)" ERROR: "foo * bar" should be "foo *bar" ERROR: "foo __init bar" should be "foo __init bar" ERROR: "foo __exit bar" should be &qu

[PATCH v2 8/8] Staging: kpc2000: kpc_dma: Resolve coding style warning reported by checkpatch

This Patch resolves unnecessary cast warning and const file_operations WARNING: unnecessary cast may hide bugs WARNING: struct file_operations should normally be const Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/fileops.c| 4 ++-- drivers/staging/kpc2000/kpc_dma

[PATCH v2 6/8] Staging: kpc2000: kpc_dma: Resolve coding style warning reported by checkpatch

ff-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c| 8 +--- drivers/staging/kpc2000/kpc_dma/fileops.c| 11 ++- drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.c | 15 --- drivers/staging/kpc2000/kpc_dma/kpc_dma_driver.h | 8 ++-- 4 files change

[PATCH v2 5/8] Staging: kpc2000: kpc_dma: Resolve coding style error reported by checkpatch

This patch fixes below errors reported by checkpath ERROR: Macros with complex values should be enclosed in parentheses CHECK: Prefer using the BIT macro ERROR: trailing statements should be on next line ERROR: trailing statements should be on next line Signed-off-by: Vandana BN --- drivers

[PATCH v2 3/8] Staging: kpc2000: kpc_dma: Resolve coding style errors reported by checkpatch

This patch resolves below errors reported by checkpath ERROR: space required before the open brace '{' ERROR: space prohibited after that '!' (ctx:BxW) ERROR: space prohibited after that open parenthesis '(' Signed-off-by: Vandana BN --- drivers/staging/kpc2

[PATCH v2 7/8] Staging: kpc2000: kpc_dma: Resolve coding style warning reported by checkpatch

This patch resolves warnings to use __func__ insted of funtion name. WARNING: Prefer using '"%s...", __func__' to using 'setup_dma_engine', this function's name, in a string Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/dma.c | 6 ++-

[PATCH v2 4/8] Staging: kpc2000: kpc_dma: Resolve code indent error reported by checkpatch

This patch fixes code indentaion error reported by checkpath ERROR: switch and case should be at the same indent ERROR: trailing statements should be on next line Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_dma/fileops.c | 12 1 file changed, 8 insertions(+), 4

Re: [Linux-kernel-mentees] [PATCH] Staging: kpc2000: kpc_dma: resolve checkpath errors and warnings

On 11/05/19 2:04 AM, Lukas Bulwahn wrote: > On Fri, May 10, 2019 at 9:39 PM Vandana BN wrote: >> This patch resolves coding style errors and warnings reported by chechpatch >> > I did not look at the patch in detail, but you might want to also > consider to replace the Caml

[PATCH] Staging: kpc2000: kpc_dma: resolve checkpath errors and warnings

are not necessary for any arm of this statement WARNING: unnecessary cast may hide bugs WARNING: braces {} are not necessary for single statement WARNING: struct file_operations should normally be const WARNING: labels should not be indented Signed-off-by: Vandana BN --- drivers/staging/kpc2000/kpc_d

Re: [PATCH 5.1 00/30] 5.1.1-stable review

On 10/05/19 12:12 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.1.1 release. > There are 30 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses shou

Re: [PATCH 5.0 00/95] 5.0.15-stable review

On 10/05/19 7:06 PM, Guenter Roeck wrote: > On 5/9/19 11:41 AM, Greg Kroah-Hartman wrote: >> This is the start of the stable review cycle for the 5.0.15 release. >> There are 95 patches in this series, all will be posted as a response >> to this one.  If anyone has any issues with these being app

[PATCH] fs: ubifs: Resolve sparse warning for using plain integer as NULL pointer

fs/ubifs/xattr.c:615:58: warning: Using plain integer as NULL pointer Signed-off-by: Vandana BN --- fs/ubifs/xattr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ubifs/xattr.c b/fs/ubifs/xattr.c index f5ad1ede7990..066a5666c50f 100644 --- a/fs/ubifs/xattr.c +++ b/fs

[PATCH] staging: rtl8192u: ieee80211: Resolve ERROR reported by checkpatch

7; (ctx:VxV) ERROR: spaces required around that '=' (ctx:VxW) ERROR: space required before the open parenthesis '(' ERROR: spaces required around that '?' (ctx:VxE) ERROR: spaces required around that ':' (ctx:VxE) ERROR: spaces required around that '==&#

[PATCH v4] staging: rtl8723bs: Fix checkpatch.pl warnings

This patch resolves coding style brace warning and constant on right warning. WARNING: Comparisons should place the constant on the right side of the test WARNING: braces {} are not necessary for single statement blocks CHECK: Comparison to NULL could be written "!pbuf" Signed-off-by: