Info Regarding how the Redeon driver access EDID info

2005-04-06 Thread karthik
operty->value field is Initialized. Is this get_EDID is called only at boot time. how the value of the device _node structure is initilized. If any have any ideas regarding this please mail me. Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the b

Info Regarding MCR tool

2005-04-06 Thread karthik
Hi, If anybody is having any idea of what is MCR and what is its use, just tell me. i think its some Monitor related software. But i want in more detail of what is it and how is it working. Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" i

accessing i2c-driver

2005-04-07 Thread karthik
write one application to check what is happening in i2c-matroxfb module. if anybody got anyidea regarding this please mail me. Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo inf

what is lmsensor package

2005-04-08 Thread karthik
Hi, Can anybody tell me what is lmsensor package used for and from where can we obtain its programs (that access i2c-interface or general source). Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTE

where the agpcard driver creats its device

2005-04-08 Thread karthik
write. Karthik. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

How to compile a single module in linux kernel tree (2.6 kernel)

2005-04-08 Thread karthik
to__this_module undefined rreference to printk etc so please tell me how to comiple a single module or driver in 2.6 kernel. Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo inf

How to access a kernel module function

2005-04-10 Thread karthik
function we need to create a node or device. if its not necessary then how can i access my module's function from user program. Karthik - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordom

Location of character device of Matrox agpcard driver

2005-04-10 Thread karthik
Hi,           Can anybody tell me where the device corresponding to the graphics driver is created ie whether its in /dev or /sys/class etc in 2.6 kernel.         Also can i uses ioctls after opening the device to see its working. Thanks & Regards Karthik R - To unsubscribe from this

[PATCH] arm: dts: omap3: Remove interrupt-parent property

2017-06-26 Thread karthik
From: Karthik Tummala All nodes inhert "interrupt-parent" property from root node. So removed that property from usbhsohci, usbhsehci, ssi_port1, ssi_port2 nodes. Signed-off-by: Karthik Tummala --- arch/arm/boot/dts/omap3.dtsi | 4 1 file changed, 4 deletions(-) diff --git

[PATCH v2] arm: dts: omap3: Remove needless interrupt-parent property

2017-06-26 Thread karthik
From: Karthik Tummala All nodes inhert "interrupt-parent" property from root node. Removed the aforementioned property from usbhsohci, usbhsehci, ssi_port1, ssi_port2 nodes to avoid duplication. Signed-off-by: Karthik Tummala --- Changes for v2: - Made changes to subject &

[PATCH 1/2] staging: ccree: Use __func__ instead of function name

2017-06-28 Thread karthik
From: Karthik Tummala Fixed following checkpatch.pl warning: WARNING: Prefer using '"%s...", __func__' to using the function's name, in a string It is prefered to use '%s & __func__' instead of function name for logging. Signed-off-by: Kart

[PATCH 0/2] staging: ccree: Fix coding style and remove warnings

2017-06-28 Thread karthik
From: Karthik Tummala This patch series fixes coding style and removes the following checkpatch.pl warnings: Prefer using '"%s...", __func__' to using the function's name, in a string. Braces {} are not necessary for single statement blocks. Rebased on top of ne

[PATCH 2/2] staging: ccree: Remove braces {} for single statement blocks

2017-06-28 Thread karthik
From: Karthik Tummala Fixed following checkpatch.pl warning: WARNING: braces {} are not necessary for single statement blocks. Signed-off-by: Karthik Tummala --- drivers/staging/ccree/ssi_aead.c | 38 ++ 1 file changed, 14 insertions(+), 24

Re: [PATCH 0/2] staging: ccree: Fix coding style and remove warnings

2017-06-29 Thread karthik
On Wednesday 28 June 2017 03:15 PM, Gilad Ben-Yossef wrote: Thank you Karthik, On Wed, Jun 28, 2017 at 12:37 PM, wrote: From: Karthik Tummala This patch series fixes coding style and removes the following checkpatch.pl warnings: Prefer using '"%s...", __func__' to

[PATCH v2] staging: ccree: Use __func__ instead of function name

2017-06-29 Thread karthik
From: Karthik Tummala Fixed following checkpatch.pl warning: WARNING: Prefer using '"%s...", __func__' to using the function's name, in a string It is prefered to use '%s & __func__' instead of function name for logging. Signed-off-by: Karthi

[PATCH] staging: greybus: loopback: Enclose multiple macro statements in do-while loop

2017-07-18 Thread karthik
From: Karthik Tummala Enclosed multiple macro statements in a do - while loop as per kernel coding standard, pointed by checkpatch. Signed-off-by: Karthik Tummala --- drivers/staging/greybus/loopback.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers

Re: [PATCH] staging: greybus: loopback: Enclose multiple macro statements in do-while loop

2017-07-18 Thread karthik
On Tuesday 18 July 2017 04:41 PM, Bryan O'Donoghue wrote: On 18/07/17 11:56, kart...@techveda.org wrote: From: Karthik Tummala Enclosed multiple macro statements in a do - while loop as per kernel coding standard, pointed by checkpatch. Signed-off-by: Karthik Tummala --- drivers/st

[PATCH v2] Staging: greybus: light: Prefer kcalloc over kzalloc

2017-05-08 Thread karthik
From: Karthik Tummala Fixed following checkpatch.pl warning: * WARNING: Prefer kcalloc over kzalloc with multiply Instead of specifying no.of bytes * size as argument in kzalloc, prefer kcalloc. Signed-off-by: Karthik Tummala Reviewed-by: Rui Miguel Silva --- Changes for v2: - Changed

[PATCH] Staging: greybus: Prefer kcalloc over kzalloc

2017-05-06 Thread karthik
From: Karthik Tummala Fixed following checkpatch.pl warning: * WARNING: Prefer kcalloc over kzalloc with multiply Instead of specifying no.of bytes * size as arugment in kzalloc, prefer kcalloc. Signed-off-by: Karthik Tummala --- drivers/staging/greybus/light.c | 4 ++-- 1 file changed, 2

[PATCH] staging: ccree: Fix Kernel coding style issues

2017-09-22 Thread karthik
From: Karthik Tummala Fixed following checkpatch warnings & checks: CHECK: Unnecessary parentheses WARNING: suspect code indent for conditional statements WARNING: Missing a blank line after declarations Signed-off-by: Karthik Tummala --- Note: - Patch was tested & built (ARCH=arm) on

[RESEND PATCH] arm: dts: omap3: Remove needless interrupt-parent property

2017-07-25 Thread karthik
From: Karthik Tummala All nodes inhert "interrupt-parent" property from root node. Removed the aforementioned property from usbhsohci, usbhsehci, ssi_port1, ssi_port2 nodes to avoid duplication. Signed-off-by: Karthik Tummala --- arch/arm/boot/dts/omap3.dtsi | 4 1 file

Re: Revert "serial: 8250: Make SERIAL_8250_RUNTIME_UARTS work correctly"

2013-06-03 Thread Karthik Manamcheri
> No worries, I undertand what Karthik is talking about, so I'll try to > figure this out. > > CONFIG_SERIAL_8250_NR_UARTS=32 > CONFIG_SERIAL_8250_RUNTIME_UARTS=4 The description for "CONFIG_SERIAL_8250_RUNTIME_UARTS" is "Set this to the maximum number of

[PATCH] n_tty: Unthrottle tty when flushing read buffer

2013-01-23 Thread Karthik Manamcheri
When the tty input buffer is full and thereby throttled, flushing/resetting the read buffer should unthrottle to allow more data to be received. Signed-off-by: Karthik Manamcheri --- drivers/tty/n_tty.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/tty/n_tty.c b/drivers/tty

Re: [PATCH v7 1/4] i2c: i2c-qcom-geni: Add bus driver for the Qualcomm GENI I2C controller

2018-07-30 Thread Karthik Ramasubramanian
; >> +pm_runtime_use_autosuspend(gi2c->se.dev); >> +pm_runtime_enable(gi2c->se.dev); >> +i2c_add_adapter(&gi2c->adap); >> + >> +return 0; > > i2c_add_adapter can fail. So, I'd guess you want to check the return > value and move it above

Re: [PATCH] i2c: i2c-qcom-geni: Fix suspend clock handling

2018-07-13 Thread Karthik Ramasubramanian
On 7/9/2018 5:49 PM, Evan Green wrote: > pm_runtime_suspended can return 0 even if the last runtime power > management function called in the device was a suspend call. This > trips up the i2c-qcom-geni's suspend_noirq accounting, which was > trying to use the function to determine if it should

Re: [PATCH v5 4/4] arm64: dts: sdm845: Add serial console support

2018-03-05 Thread Karthik Ramasubramanian
quot;se" in v3 of the GENI patchset, with that this boots nicely again. Thanks Bjorn for testing. Karthik, it would be good if you could include just this patch from this series as part of your GENI series for the next (and subsequent) repost, with updates as needed based on binding c

[PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-10 Thread karthik alapati
fix some checkpatch "Block comments use * on subsequent lines" warnings and remove obsolete code Suggested-by: Dan Carpenter Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 44 --- 1 file changed, 9 insertions(+), 35 deletions(-) di

[PATCH] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-10 Thread karthik alapati
fix checkpatch.pl warning for "block comments should align the * on each line" and make function comments follow kernel-doc Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 185 +++--- 1 file changed, 73 insertions(+), 112 deletions(-)

[PATCH v4 1/2] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-11 Thread karthik alapati
there are some good function comments not following kernel-doc. Make them follow kernel-doc style Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 185 +++--- 1 file changed, 73 insertions(+), 112 deletions(-) diff --git a/drivers/staging/rtl8723bs

[PATCH v4 0/2] staging: rtl8723bs: driver cleanup

2021-02-11 Thread karthik alapati
karthik alapati (2): staging: rtl8723bs: fix function comments to follow kernel-doc staging: rtl8723bs: remove obsolete commented out code .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 225 ++ 1 file changed, 75 insertions(+), 150 deletions(-) -- 2.30.0

[PATCH v4 2/2] staging: rtl8723bs: remove obsolete commented out code

2021-02-11 Thread karthik alapati
There is a bunch of messy, commented out code. Just delete it. Suggested-by: Dan Carpenter Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 40 +-- 1 file changed, 2 insertions(+), 38 deletions(-) diff --git a/drivers/staging/rtl8723bs/hal

[PATCH] use explicit host byte-order types in comparison

2021-02-18 Thread karthik alapati
convert le32 types to host byte-order types before comparison Signed-off-by: karthik alapati --- i wonder how these could be false-positives drivers/staging/wimax/i2400m/fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wimax/i2400m/fw.c b/drivers

[PATCH] staging: i2400m: use explicit host byte-order types in comparison

2021-02-18 Thread karthik alapati
convert le32 types to host byte-order types before comparison Signed-off-by: karthik alapati --- i wonder how these could be false-positives drivers/staging/wimax/i2400m/fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wimax/i2400m/fw.c b/drivers

[PATCH] staging: rtl8723bs: fix block comments alignment

2021-02-09 Thread karthik alapati
fix checkpatch.pl warning for "block comments should align the * on each line" Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 204 +- 1 file changed, 102 insertions(+), 102 deletions(-) diff --git a/drivers/staging/rtl

[PATCH] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-09 Thread karthik alapati
fix checkpatch.pl warning for "block comments should align the * on each line" and make function comments follow kernel-doc Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 209 -- 1 file changed, 91 insertions(+), 118 deletions(-)

[PATCH] staging: wimax: fix sparse incorrect type issue

2021-02-20 Thread karthik alapati
fix sparse warning by casting to explicit user address-space pointer type Signed-off-by: karthik alapati --- drivers/staging/wlan-ng/p80211netdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/p80211netdev.c b/drivers/staging/wlan-ng

[PATCH] staging: wimax/i2400m: fix byte-order issue

2021-02-20 Thread karthik alapati
fix sparse byte-order warnings by converting host byte-order types to le32 types Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/op-rfkill.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/op-rfkill.c b/drivers/staging/wimax

[PATCH] staging: wimax/i2400m: fix byte-order type issue

2021-02-20 Thread karthik alapati
fix sparse type warning by converting le32 types to host byte-order types before comparison Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/fw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/fw.c b/drivers/staging/wimax/i2400m

[PATCH] staging: media/atomisp: don't compile unused code

2021-02-20 Thread karthik alapati
currently the functions defined in ibuf_ctrl_rmgr.c file are only used by isys_init.c when CONFIG_VIDEO_ATOMISP_ISP2401 is selected so dont't compile it when not needed, also fixes some sparse warnings Signed-off-by: karthik alapati --- is this the right patch or just compiling out

[PATCH 1/3] staging: wimax/i2400m: fix byte-order issue

2021-02-21 Thread karthik alapati
fix sparse byte-order warnings by converting host byte-order type to __le16 byte-order types before assigning to hdr.length Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/op-rfkill.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax

[PATCH 0/3] fix sparse warnings

2021-02-21 Thread karthik alapati
:6: warning: symbol 'ia_css_isys_ibuf_rmgr_acquire' was not declared. Should it be static? drivers/staging//media/atomisp/pci/runtime/isys/src/ibuf_ctrl_rmgr.c:106:6: warning: symbol 'ia_css_isys_ibuf_rmgr_release' was not declared. Should it be static? karthik alapati (3):

[PATCH 2/3] staging: wimax/i2400m: convert __le32 type to host byte-order

2021-02-21 Thread karthik alapati
fix sparse type warning by converting __le32 types to host byte-order types before comparison Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/fw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/fw.c b/drivers/staging/wimax

[PATCH 3/3] staging: media/atomisp: don't compile unused code

2021-02-21 Thread karthik alapati
currently the functions defined in ibuf_ctrl_rmgr.c file are only used by isys_init.c when CONFIG_VIDEO_ATOMISP_ISP2401 is selected so dont't compile it when not needed, also fixes some sparse warnings Signed-off-by: karthik alapati --- drivers/staging/media/atomisp/Makefile | 5 +++-- 1

[PATCH 2/2] staging: wimax/i2400m: convert __le32 type to host byte-order

2021-02-21 Thread karthik alapati
fix sparse type warning by converting __le32 types to host byte-order types before comparison Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/fw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax/i2400m/fw.c b/drivers/staging/wimax

[PATCH 1/2] staging: wimax/i2400m: fix byte-order issue

2021-02-21 Thread karthik alapati
fix sparse byte-order warnings by converting host byte-order type to __le16 byte-order types before assigning to hdr.length Signed-off-by: karthik alapati --- drivers/staging/wimax/i2400m/op-rfkill.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wimax

[PATCH 0/2] fix sparse warnings

2021-02-21 Thread karthik alapati
drivers/staging//wimax/i2400m/op-rfkill.c:89:25:got unsigned long . drivers/staging//wimax/i2400m/fw.c:514:27: warning: restricted __le32 degrades to integer karthik alapati (2): staging: wimax/i2400m: fix byte-order issue staging: wimax/i2400m: convert __le32 type to host byte-order

[PATCH] staging: media/atomisp: don't compile unused code

2021-02-21 Thread karthik alapati
currently the functions defined in ibuf_ctrl_rmgr.c file are only used by isys_init.c when CONFIG_VIDEO_ATOMISP_ISP2401 is selected so dont't compile it when not needed, also fixes some sparse warnings Signed-off-by: karthik alapati --- drivers/staging/media/atomisp/Makefile | 5 +++-- 1

[PATCH] staging: et131x: remove unneccessary 'out of memory' message

2014-09-30 Thread Karthik Nayak
Removes the 'out of memory' warning issues by checkpatch Signed-off-by: Karthik Nayak --- drivers/staging/et131x/et131x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 41fcaa0..f1a9296 100644 --- a/drive

[PATCH] staging: bcm: multiple checkpatch fixes for InterfaceIdleMode.c

2014-09-30 Thread Karthik Nayak
1. WARNING: else is not generally useful after a break or return 2. WARNING: quoted string split across lines 3. WARNING: break quoted strings at a space character Signed-off-by: Karthik Nayak --- drivers/staging/bcm/InterfaceIdleMode.c | 11 +-- 1 file changed, 5 insertions(+), 6

Re: [PATCH] staging: et131x: remove unneccessary 'out of memory' message

2014-09-30 Thread karthik nayak
Oops, i sent the old commit again, sorry! Regards, Karthik Nayak On Tue, Sep 30, 2014 at 11:10 PM, Fabio Estevam wrote: > On Tue, Sep 30, 2014 at 2:19 PM, Karthik Nayak wrote: >> Removes the 'out of memory' warning issues by checkpatch >> >> Signed-off-by: Ka

[PATCH] staging: et131x: remove unneccessary 'out of memory' message

2014-09-30 Thread Karthik Nayak
Removes the 'out of memory' warning issued by checkpatch Signed-off-by: Karthik Nayak --- drivers/staging/et131x/et131x.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 41fcaa0..f92cceb 10

[PATCH] staging: bcm: remove unnecessary else statement

2014-09-30 Thread Karthik Nayak
Removed the else statement occurring after an if statement with a return value as per checkpatch warning. Signed-off-by: Karthik Nayak --- drivers/staging/bcm/InterfaceMisc.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/bcm/InterfaceMisc.c b

Re: [PATCH] Staging: rtl8192u: removed an unnecessary else statement

2014-12-18 Thread karthik nayak
On 12/19/2014 04:08 AM, Jeremiah Mahler wrote: Karthik, On Thu, Dec 18, 2014 at 02:50:11PM +0530, Karthik Nayak wrote: As per checkpatch warning, removed an unnecessary else statement proceeding an if statement with a return. Signed-off-by: Karthik Nayak --- drivers/staging/rtl8192u

[PATCH] Staging: rtl8712: removed an unnecessary else statement

2014-12-15 Thread Karthik Nayak
As per checkpatch warning, removed an unnecessary else statement proceeding an if statement with a return. Signed-off-by: Karthik Nayak --- drivers/staging/rtl8712/rtl8712_recv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b

[PATCH] Staging: rtl8192u: removed an unnecessary else statement

2014-12-18 Thread Karthik Nayak
As per checkpatch warning, removed an unnecessary else statement proceeding an if statement with a return. Signed-off-by: Karthik Nayak --- drivers/staging/rtl8192u/r8192U_dm.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/staging/rtl8192u

[PATCH] staging: gdm724x: remove prohibited space before the comma

2014-12-18 Thread Karthik Nayak
Remove checkpatch error of prohibited space before the comma. Signed-off-by: Karthik Nayak --- drivers/staging/gdm724x/gdm_mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gdm724x/gdm_mux.c b/drivers/staging/gdm724x/gdm_mux.c index b5b063a..d1ab996

[PATCH] Fix clang warning -Wpointer-bool-conversion by removing unnecessary check in regd.c

2018-09-21 Thread Karthik Nishanth
~ ~^~~~ 1 warning generated. Link: https://github.com/ClangBuiltLinux/linux/issues/170 Signed-off-by: Karthik Nishanth --- drivers/staging/rtlwifi/regd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtlwifi/regd.c b/drivers/staging/rtl

Re: [PATCH] Fix clang warning -Wpointer-bool-conversion by removing unnecessary check in regd.c

2018-09-22 Thread Karthik Nishanth
Do I submit a new patch with the modified subject? On Sat, Sep 22, 2018 at 03:44:32PM +0300, Dan Carpenter wrote: >The subject needs a subsystem prefix. It's also slightly long. > >[PATCH] Staging: rtlwifi: remove unnecessary NULL check > >regards, >dan carpenter > Karthik Nishanth

[PATCH v2] Staging: rtlwifi: remove unnecessary NULL check

2018-09-22 Thread Karthik Nishanth
~ ~^~~~ 1 warning generated. Link: https://github.com/ClangBuiltLinux/linux/issues/170 Signed-off-by: Karthik Nishanth --- v2: changed the subject drivers/staging/rtlwifi/regd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtlwifi/regd.c b/

[PATCH] staging: lustre: selftest: Make brw_inject_one_error() static

2016-12-23 Thread Karthik Nayak
Since the function brw_inject_one_error() is used only within brw_test.c, make it static. This was reported as a warning by sparse. Signed-off-by: Karthik Nayak --- drivers/staging/lustre/lnet/selftest/brw_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH] staging: comedi: Fix incorrect type assignment

2017-02-07 Thread Karthik Nayak
/drivers//ni_pcimio.c:1229:32:got unsigned int This is done by introducing a temporary variable which is of type '__be32' and converting the existing variable to type 'unsigned int'. Signed-off-by: Karthik Nayak --- drivers/staging/comedi/drivers/ni_pcimio.c | 5 +++-- dr

Re: [PATCH] staging: comedi: Fix incorrect type assignment

2017-02-08 Thread Karthik Nayak
Hello, On Wed, Feb 8, 2017 at 6:43 PM, Ian Abbott wrote: > On 07/02/17 19:06, Karthik Nayak wrote: >> >> This patch fixes the following sparse error: >> drivers/staging/comedi/drivers//ni_pcimio.c:1229:32: warning: incorrect >> type in assignment (different base types

Re: [PATCH] staging: comedi: Fix incorrect type assignment

2017-02-08 Thread Karthik Nayak
Hi, On Wed, Feb 8, 2017 at 7:18 PM, Ian Abbott wrote: > On 08/02/17 13:26, Karthik Nayak wrote: >> >> Hello, >> >> On Wed, Feb 8, 2017 at 6:43 PM, Ian Abbott wrote: >>> >>> On 07/02/17 19:06, Karthik Nayak wrote: >>>> >>>> &

[PATCH 2/2] staging: comedi: drop unused variable from struct 'ni_private'

2017-02-08 Thread Karthik Nayak
Drop the 'serial_number' variable from the struct 'ni_private' since its never used after assignment. Signed-off-by: Karthik Nayak --- This is to be based on top of "staging: comedi: Fix incorrect type assignment" to which this is replied to. drivers/staging/

Re: [PATCH 2/2] staging: comedi: drop unused variable from struct 'ni_private'

2017-02-08 Thread Karthik Nayak
Hey, On Wed, Feb 8, 2017 at 11:10 PM, Ian Abbott wrote: > On 08/02/2017 16:55, Karthik Nayak wrote: >> >> Drop the 'serial_number' variable from the struct 'ni_private' since >> its never used after assignment. >> >> Signed-off-by: Kart

[PATCH] staging: comedi: Fix incorrect type assignment

2017-02-08 Thread Karthik Nayak
/drivers//ni_pcimio.c:1229:32:got unsigned int This is done by removing the whole code block, since the variable 'serial_number' is only assigned but never used. Helped-by: Ian Abbott Signed-off-by: Karthik Nayak --- drivers/staging/comedi/drivers/ni_pcimio.c | 9 - drivers/stag

Re: [PATCH 2/2] staging: comedi: drop unused variable from struct 'ni_private'

2017-02-08 Thread Karthik Nayak
atch, thanks :) > Sorry for the multiple messages, now that we're deleting the whole block, I think it makes sense to squash the commits, so I'll send one single patch. -- Regards, Karthik Nayak

Re: [PATCH] staging: comedi: Fix incorrect type assignment

2017-02-09 Thread Karthik Nayak
Hello, On Thu, Feb 9, 2017 at 4:34 PM, Greg KH wrote: > On Thu, Feb 09, 2017 at 01:53:56AM +0530, Karthik Nayak wrote: >> This patch fixes the following sparse error: >> drivers/staging/comedi/drivers//ni_pcimio.c:1229:32: warning: incorrect type >> in assignment

[PATCH] staging: comedi: ni_pcimio: remove unused variable 'serial_number'

2017-02-09 Thread Karthik Nayak
ning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers//ni_pcimio.c:1229:32:expected restricted __be32 [usertype] serial_number drivers/staging/comedi/drivers//ni_pcimio.c:1229:32:got unsigned int Signed-off-by: Karthik Nayak --- drivers/staging/comedi

Re: [PATCH v2] tty: serial: fsl_lpuart.c: prevent a bad shift operation

2020-08-05 Thread B K Karthik
On Wed, Aug 5, 2020 at 6:26 PM Greg Kroah-Hartman wrote: > > On Wed, Aug 05, 2020 at 12:09:34PM +, Vabhav Sharma (OSS) wrote: > > > > > > > -Original Message- > > > From: Greg Kroah-Hartman > > > Sent: Wednesday, July 29, 2020 9:34 P

[PATCH] staging: greybus: audio_helper.c: Replace strlcpy() with strscpy()

2021-01-09 Thread B K Karthik
showing this warning. Signed-off-by: B K Karthik --- drivers/staging/greybus/audio_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_helper.c b/drivers/staging/greybus/audio_helper.c index 3011b8abce38..1ed4772d2771 100644 --- a/drivers

Re: [PATCH] staging: andriod: ashmem: Declared file operation with const keyword

2021-03-13 Thread B K Karthik
On Sat, Mar 13, 2021 at 10:57 PM namratajanawade wrote: > > Warning found by checkpatch.pl script. That doesn't tell what you did or why you did it. Please write an appropriate commit description and resend the patch. karthik

[PATCH] staging: rtl8188eu: include: rtl8188e_xmit.h: fixed multiple blank space coding style issues

2020-07-01 Thread B K Karthik
added blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtl8188e_xmit.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_xmit.h b/drivers/staging/rtl8188eu

[PATCH] staging: rtl8188eu: include: rtw_cmd.h: fixed a blank space coding style issue.

2020-07-01 Thread B K Karthik
add blank spaces for improved code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_cmd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_cmd.h b/drivers/staging/rtl8188eu/include/rtw_cmd.h index

[PATCH 1/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed multiple parentheses coding style issues.

2020-07-01 Thread B K Karthik
add parentheses since complex valued macros must be enclosed within parentheses. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers

[PATCH 2/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed a blank space coding style issue.

2020-07-01 Thread B K Karthik
add blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h

[PATCH] staging: rtl8188eu: include: rtw_recv.h: fixed a blank space coding style issue.

2020-07-01 Thread B K Karthik
added blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_recv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_recv.h b/drivers/staging/rtl8188eu/include/rtw_recv.h index

[PATCH] staging: rtl8188eu: include: rtl8188e_xmit.h: fixed multiple blank space coding style issues

2020-07-01 Thread B K Karthik
added blank spaces to improve code readability. (coding style issue) Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtl8188e_xmit.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_xmit.h b/drivers

[PATCH 1/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed multiple parentheses coding style issues.

2020-07-01 Thread B K Karthik
add parentheses since complex valued macros must be enclosed within parentheses. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers

[PATCH 2/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed a blank space coding style issue.

2020-07-01 Thread B K Karthik
add blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h

[PATCH] staging: rtl8188eu: include: rtl8188e_xmit.h: fixed multiple blank space coding style issues

2020-07-02 Thread B K Karthik
added blank spaces to improve code readability. (coding style issue) Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtl8188e_xmit.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_xmit.h b/drivers

[PATCH] staging: rtl8188eu: include: rtw_cmd.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
add blank spaces for improved code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_cmd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_cmd.h b/drivers/staging/rtl8188eu/include/rtw_cmd.h index

[PATCH] staging: rtl8188eu: include: rtw_recv.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
added blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_recv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_recv.h b/drivers/staging/rtl8188eu/include/rtw_recv.h index

[PATCH 1/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed multiple parentheses coding style issues.

2020-07-02 Thread B K Karthik
add parentheses since complex valued macros must be enclosed within parentheses. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers

[PATCH 2/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
add blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h

[PATCH] staging: rtl8188eu: include: odm.h: fixed a blank space coding style issue.

2020-06-29 Thread B K Karthik
added blank space and enclosed a complex valued macro within parentheses for improved code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/odm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/odm.h b/drivers

[PATCH] staging: rtl8188eu: include: hal8188e_rate_adaptive.h: fixed a blank space coding style issue

2020-06-29 Thread B K Karthik
added blank space around arithmetic operators to improve readability Signed-off-by: B K Karthik --- .../rtl8188eu/include/hal8188e_rate_adaptive.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/hal8188e_rate_adaptive.h b

[PATCH] staging: nvec: changed coding style, line should not end with a (

2020-06-25 Thread B K Karthik
Signed-off-by: B K Karthik --- drivers/staging/nvec/nvec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c index 360ec0407740..16afbe1dfdeb 100644 --- a/drivers/staging/nvec/nvec.c +++ b/drivers/staging/nvec/nvec.c

[PATCH 1/1] staging: media: soc_camera: Adding parentheses to macro defination at line 241, Clearing alignment issues at lines 410 and 1270, fixing return values at EPOLLERR

2020-06-25 Thread B K Karthik
yle problem "Alignment should match open parenthesis" by adding tab spaces in line 1270. Signed-off-by: B K Karthik --- drivers/staging/media/soc_camera/soc_camera.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/media/soc_camera/soc_camera.

[PATCH] fixing ERROR: Macros with complex values must be enclosed within parentheses.

2020-06-25 Thread B K Karthik
soc_camera.c: fixing ERROR: Macros with complex values must be enclused within parentheses. Signed-off-by: B K Karthik --- drivers/staging/media/soc_camera/soc_camera.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/media/soc_camera/soc_camera.c b

[PATCH] staging: media: usbvision: removing prohibited space before ',' (ctx:WxW)

2020-06-26 Thread B K Karthik
fixing ERROR: space prohibited before that ',' (ctx:WxW) Signed-off-by: B K Karthik --- drivers/staging/media/usbvision/usbvision-i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/usbvision/usbvision-i2c.c b/drivers/staging/media

[PATCH] drivers: staging: media: atomisp: pci: css_2401_system: host: csi_rx.c: fixed a sparse warning by making undeclared symbols static

2020-07-14 Thread B K Karthik
changed symbols N_SHORT_PACKET_LUT_ENTRIES, N_LONG_PACKET_ENTRIES, N_CSI_RX_FE_CTRL_DLANES, N_CSI_RX_BE_SID_WIDTH to static because they were not declared earlier. Signed-off-by: B K Karthik --- .../media/atomisp/pci/css_2401_system/host/csi_rx.c | 8 1 file changed, 4 insertions

[PATCH v2] staging: media: atomisp: mark some csi_rx.c symbols static

2020-07-14 Thread B K Karthik
changed symbols N_SHORT_PACKET_LUT_ENTRIES, N_LONG_PACKET_ENTRIES, N_CSI_RX_FE_CTRL_DLANES, N_CSI_RX_BE_SID_WIDTH to static because they were not declared earlier. Signed-off-by: B K Karthik --- .../media/atomisp/pci/css_2401_system/host/csi_rx.c | 8 1 file changed, 4 insertions

[PATCH] net: ipv6: fix slab-out-of-bounda Read in xfrm6_tunnel_alloc_spi

2020-07-25 Thread B K Karthik
com Signed-off-by: B K Karthik --- net/ipv6/xfrm6_tunnel.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/net/ipv6/xfrm6_tunnel.c b/net/ipv6/xfrm6_tunnel.c index 25b7ebda2fab..2d049244be81 100644 --- a/net/ipv6/xfrm6_tunnel.c +++ b/net/ipv6/xfrm6_tunnel.c @@ -121,8 +12

[PATCH] smack: fix slab-out-of-bounds by checking for overflow

2020-07-25 Thread B K Karthik
fc fc fc fc 888097d68380: 00 00 00 00 01 fc fc fc fc fc fc fc fc fc fc fc == Reported-and-testedby: syzbot+a22c6092d003d6fe1...@syzkaller.appspotmail.com Signed-off-by: B K Karthik --- security/smack/smackfs.c | 5 + 1 f

[PATCH] net: ipv6: fix slab-out-of-bounds Read in __xfrm6_tunnel_spi_check

2020-07-25 Thread B K Karthik
fc fc fc fc fc fc 8880a93a5f00: fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc fc == Reported-by: syzbot+7da3fdf292816554b...@syzkaller.appspotmail.com Signed-off-by: B K Karthik --- net/ipv6/xfrm6_tunnel.c | 4 ++-- 1 file changed

[PATCH] net: xfrm: xfrm_policy.c: remove some unnecessary cases in decode_session6

2020-07-25 Thread B K Karthik
remove some unnecessary cases in decode_session6 Signed-off-by: B K Karthik --- net/xfrm/xfrm_policy.c | 4 1 file changed, 4 deletions(-) diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index 19c5e0fa3f44..e1c988a89382 100644 --- a/net/xfrm/xfrm_policy.c +++ b/net/xfrm

[PATCH v2] net: ipv6: fix use-after-free Read in __xfrm6_tunnel_spi_lookup

2020-07-25 Thread B K Karthik
-by: syzbot+72ff2fa98097767b5...@syzkaller.appspotmail.com Reported-by: kernel test robot Signed-off-by: B K Karthik --- v1 -> v2: added cast in arguement from u32 to (const xfrm_address_t *) added Reported-by: kernel test robot removed Reported-by: syzbot+72ff2fa9809

Re: [PATCH v2] net: ipv6: fix use-after-free Read in __xfrm6_tunnel_spi_lookup

2020-07-25 Thread B K Karthik
On Sun, Jul 26, 2020 at 11:05 AM Cong Wang wrote: > > On Sat, Jul 25, 2020 at 8:09 PM B K Karthik wrote: > > @@ -103,10 +103,10 @@ static int __xfrm6_tunnel_spi_check(struct net *net, > > u32 spi) > > { > > struct xfrm6_tunnel_net *xfrm

Re: [PATCH v2] net: ipv6: fix use-after-free Read in __xfrm6_tunnel_spi_lookup

2020-07-26 Thread B K Karthik
On Mon, Jul 27, 2020 at 1:37 AM Cong Wang wrote: > > On Sat, Jul 25, 2020 at 11:12 PM B K Karthik wrote: > > > > On Sun, Jul 26, 2020 at 11:05 AM Cong Wang wrote: > > > > > > On Sat, Jul 25, 2020 at 8:09 PM B K Karthik > > > wr

  1   2   >