[PATCH] mmc: usdhi6rol0: Handle return value of platform_get_irq_byname for card detect

platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/mmc/host/usdhi6rol0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/host/usdhi6rol0.c b/drivers/mmc/host/usdhi6

[PATCH] mmc: usdhi6rol0: Handle return value of platform_get_irq_byname for card detect

platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/mmc/host/usdhi6rol0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/host/usdhi6rol0.c b/drivers/mmc/host/usdhi6rol0.c index cdfeb15..5a896de

[PATCH] vlynq : Handle return value of platform_get_irq_byname and vlynq_probe

platform_get_irq_byname() can fail here and we must check its return value. device_register() can fail here, initialised 'result' variable beforen'goto fail_register' otherwise vlynq_probe will return uninitialised 'result' variable. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.

[PATCH] vlynq : Handle return value of platform_get_irq_byname and vlynq_probe

platform_get_irq_byname() can fail here and we must check its return value. device_register() can fail here, initialised 'result' variable beforen'goto fail_register' otherwise vlynq_probe will return uninitialised 'result' variable. Signed-off-by: Arvind Yadav --- drivers/vlynq/vlynq.c | 8

[PATCH v2] phy: cpcap-usb: Fix platform_get_irq_byname's error checking.

The platform_get_irq_byname() function returns negative if an error occurs. zero or positive number on success. platform_get_irq_byname() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v1 : Patch descr

[PATCH v2] phy: cpcap-usb: Fix platform_get_irq_byname's error checking.

The platform_get_irq_byname() function returns negative if an error occurs. zero or positive number on success. platform_get_irq_byname() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- changes in v1 : Patch description was not correct. drivers/phy

[PATCH] phy: cpcap-usb: Fix platform_get_irq_byname's error checking

The platform_get_irq_byname() function returns -1 if an error occurs. zero or positive number on success. platform_get_irq_byname() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/phy/motorola/phy-cpcap-usb.c | 2 +- 1 file chan

[PATCH] phy: cpcap-usb: Fix platform_get_irq_byname's error checking

The platform_get_irq_byname() function returns -1 if an error occurs. zero or positive number on success. platform_get_irq_byname() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/phy/motorola/phy-cpcap-usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] drm/sti: Handle return value of platform_get_irq_byname

platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/gpu/drm/sti/sti_hdmi.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c

[PATCH] drm/sti: Handle return value of platform_get_irq_byname

platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/gpu/drm/sti/sti_hdmi.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c index 30f02d2..adabd41 100644

[PATCH] [media] winbond-cir: Fix pnp_irq's error checking for wbcir_probe

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-c

[PATCH] [media] winbond-cir: Fix pnp_irq's error checking for wbcir_probe

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond

[PATCH 08/12] isdn: hisax: Fix pnp_irq's error checking for setup_isurf

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/isurf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/isurf.c b/d

[PATCH] staging: comedi: ni_atmio: Handle return value of pnp_*

pnp_irq() and pnp_port_start() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/comedi/drivers/ni_atmio.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/comedi/drivers/ni_atmio.c b/d

[PATCH 08/12] isdn: hisax: Fix pnp_irq's error checking for setup_isurf

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/isurf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/isurf.c b/drivers/isdn/hisax/isurf.c index

[PATCH] staging: comedi: ni_atmio: Handle return value of pnp_*

pnp_irq() and pnp_port_start() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/staging/comedi/drivers/ni_atmio.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/comedi/drivers/ni_atmio.c b/drivers/staging/comedi/drivers

[PATCH 07/12] isdn: hisax: Handle return value of pnp_irq and pnp_port_start

pnp_irq() and pnp_port_start() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hisax_fcpcipnp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/isdn/hisax/hisax_fcpcipnp.c b/drivers/isdn

[PATCH 07/12] isdn: hisax: Handle return value of pnp_irq and pnp_port_start

pnp_irq() and pnp_port_start() can fail here and we must check its return value. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/hisax_fcpcipnp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/isdn/hisax/hisax_fcpcipnp.c b/drivers/isdn/hisax/hisax_fcpcipnp.c index e4f7573

[PATCH 12/12] isdn: hisax: Fix pnp_irq's error checking for setup_teles3

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/teles3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/tele

[PATCH 12/12] isdn: hisax: Fix pnp_irq's error checking for setup_teles3

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/teles3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/teles3.c b/drivers/isdn/hisax/teles3.c index

[PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/avm_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/avm_p

[PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/avm_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/avm_pci.c b/drivers/isdn/hisax/avm_pci.c

[PATCH 04/12] isdn: hisax: Fix pnp_irq's error checking for setup_elsa_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/elsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/elsa.c b/d

[PATCH 04/12] isdn: hisax: Fix pnp_irq's error checking for setup_elsa_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/elsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/elsa.c b/drivers/isdn/hisax/elsa.c index

[PATCH 10/12] isdn: hisax: Fix pnp_irq's error checking for setup_niccy

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/niccy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/niccy.c b/d

[PATCH 10/12] isdn: hisax: Fix pnp_irq's error checking for setup_niccy

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/niccy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/niccy.c b/drivers/isdn/hisax/niccy.c index

[PATCH 11/12] isdn: hisax: Fix pnp_irq's error checking for setup_sedlbauer_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/sedlbauer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/sedlbau

[PATCH 11/12] isdn: hisax: Fix pnp_irq's error checking for setup_sedlbauer_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/sedlbauer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/sedlbauer.c b/drivers/isdn/hisax

[PATCH 09/12] isdn: hisax: Fix pnp_irq's error checking for setup_ix1micro

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/ix1_micro.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/ix1_mic

[PATCH 09/12] isdn: hisax: Fix pnp_irq's error checking for setup_ix1micro

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/ix1_micro.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/ix1_micro.c b/drivers/isdn/hisax

[PATCH 06/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcs

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hfcscard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfcsca

[PATCH 06/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcs

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/hfcscard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfcscard.c b/drivers/isdn/hisax/hfcscard.c

[PATCH 05/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcsx

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hfc_sx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfc_

[PATCH 05/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcsx

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/hfc_sx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfc_sx.c b/drivers/isdn/hisax/hfc_sx.c index

[PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/asuscom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/asusc

[PATCH 03/12] isdn: hisax: Fix pnp_irq's error checking for setup_diva_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/diva.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/diva.c b/d

[PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/asuscom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/asuscom.c b/drivers/isdn/hisax/asuscom.c

[PATCH 03/12] isdn: hisax: Fix pnp_irq's error checking for setup_diva_isapnp

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav --- drivers/isdn/hisax/diva.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/diva.c b/drivers/isdn/hisax/diva.c index

[PATCH 00/12] isdn: hisax: Fix pnp_irq's error checking

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Arvind Yadav (12): [PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom [PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup [PATCH 03/12] isdn

[PATCH 00/12] isdn: hisax: Fix pnp_irq's error checking

The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Arvind Yadav (12): [PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom [PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup [PATCH 03/12] isdn

[PATCH] Input: davinci_keyscan: Free memory occupied by davinci key scan device

Here, key_dev is NULL in input_free_device() and It will not free memory occupied by davinci key scan device. Move "key_dev = NULL" after input_free_device(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/input/keyboard/davinci_keyscan.c | 2 +- 1 file chan

[PATCH] Input: davinci_keyscan: Free memory occupied by davinci key scan device

Here, key_dev is NULL in input_free_device() and It will not free memory occupied by davinci key scan device. Move "key_dev = NULL" after input_free_device(). Signed-off-by: Arvind Yadav --- drivers/input/keyboard/davinci_keyscan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletio

[PATCH] RDMA/bnxt_re: Remove redundant bnxt_qplib_disable_nq() call

The bnxt_qplib_disable_nq() call is redundant as it occurs after 'goto fail' and hence it called twice. Remove it. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/infiniband/hw/bnxt_re/qplib_fp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/infiniband/hw/b

[PATCH] RDMA/bnxt_re: Remove redundant bnxt_qplib_disable_nq() call

The bnxt_qplib_disable_nq() call is redundant as it occurs after 'goto fail' and hence it called twice. Remove it. Signed-off-by: Arvind Yadav --- drivers/infiniband/hw/bnxt_re/qplib_fp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/infiniband/hw/bnxt_re/qplib_fp.c b/drivers

[PATCH] atm: horizon: Fix irq release error

atm_dev_register() can fail here and passed parameters to free irq which is not initialised. Initialization of 'dev->irq' happened after the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in free_irq(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/a

[PATCH] atm: horizon: Fix irq release error

atm_dev_register() can fail here and passed parameters to free irq which is not initialised. Initialization of 'dev->irq' happened after the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in free_irq(). Signed-off-by: Arvind Yadav --- drivers/atm/horizon.c | 2 +- 1 file chan

[PATCH 3/3] irqchip/gic-v3: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/irqchip/irq-gic-v3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqch

[PATCH 3/3] irqchip/gic-v3: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/irqchip/irq-gic-v3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c index b5df99c

[PATCH 1/3] irqchip/mips-gic: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/irqchip/irq-mips-gic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-mips-gic.c b/d

[PATCH 1/3] irqchip/mips-gic: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/irqchip/irq-mips-gic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-mips-gic.c b/drivers/irqchip/irq-mips-gic.c index

[PATCH 0/3] irqchip: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Arvind Yadav (3): [PATCH 1/3] irqchip/mips-gic: pr_err() strings should end with newlines [PATCH 2/3] irqchip/s3c24xx: pr_err() strings should end with newlines [PATCH 3/3] irqchip/gic-v3: pr_err

[PATCH 0/3] irqchip: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Arvind Yadav (3): [PATCH 1/3] irqchip/mips-gic: pr_err() strings should end with newlines [PATCH 2/3] irqchip/s3c24xx: pr_err() strings should end with newlines [PATCH 3/3] irqchip/gic-v3: pr_err

[PATCH 2/3] irqchip/s3c24xx: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/irqchip/irq-s3c24xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-s3c24xx.c b/d

[PATCH 2/3] irqchip/s3c24xx: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/irqchip/irq-s3c24xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-s3c24xx.c b/drivers/irqchip/irq-s3c24xx.c index

[PATCH] mailbox/omap: unregister mbox class

platform_driver_register() can fail here and we must unregister mbox class. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/mailbox/omap-mailbox.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/mailbox/omap-mailbox.c b/drivers/mailbo

[PATCH] mailbox/omap: unregister mbox class

platform_driver_register() can fail here and we must unregister mbox class. Signed-off-by: Arvind Yadav --- drivers/mailbox/omap-mailbox.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/mailbox/omap-mailbox.c b/drivers/mailbox/omap-mailbox.c index c5e8b9c

[PATCH] powercap: Remove reassignment

It is not necessary to reassignment of 'result'. Here, result is being initialized zero and then updated with seed_constraint_attributes(). class_register is enough to return successful and error value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/powercap/powercap

[PATCH] powercap: Remove reassignment

It is not necessary to reassignment of 'result'. Here, result is being initialized zero and then updated with seed_constraint_attributes(). class_register is enough to return successful and error value. Signed-off-by: Arvind Yadav --- drivers/powercap/powercap_sys.c | 6 ++ 1 file changed

[PATCH 0/2] scsi: ses: Fix space before '[' and remove error check

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. It is not necessary to check return value of class_register. enclosure_init returns both successful and error value. Arvind Yadav (2): [PATCH 1/2] scsi: ses: Fix space before '[' error. [PATCH 2/2] scsi: ses

[PATCH 0/2] scsi: ses: Fix space before '[' and remove error check

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. It is not necessary to check return value of class_register. enclosure_init returns both successful and error value. Arvind Yadav (2): [PATCH 1/2] scsi: ses: Fix space before '[' error. [PATCH 2/2] scsi: ses

[PATCH 1/2] scsi: ses: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/misc/enclosure.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/misc/enclosure.c b/drivers/misc/enclosure.c

[PATCH 1/2] scsi: ses: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav --- drivers/misc/enclosure.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/misc/enclosure.c b/drivers/misc/enclosure.c index eb29113..c4c5552 100644

[PATCH 2/2] scsi: ses: Remove unnecessary error check

It is not necessary to check return value of class_register. enclosure_init returns both successful and error value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/misc/enclosure.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/driver

[PATCH 2/2] scsi: ses: Remove unnecessary error check

It is not necessary to check return value of class_register. enclosure_init returns both successful and error value. Signed-off-by: Arvind Yadav --- drivers/misc/enclosure.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/misc/enclosure.c b/drivers/misc

[RFT] [media] em28xx: Fix use-after-free in v4l2_fh_init

Here, em28xx_free_v4l2 is release "v4l2->dev->v4l2" Which is allready release by em28xx_v4l2_init. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov "net/media/em28xx: use-after-free in v4l2_fh_init" drivers/med

[RFT] [media] em28xx: Fix use-after-free in v4l2_fh_init

Here, em28xx_free_v4l2 is release "v4l2->dev->v4l2" Which is allready release by em28xx_v4l2_init. Signed-off-by: Arvind Yadav --- This bug report by Andrey Konovalov "net/media/em28xx: use-after-free in v4l2_fh_init" drivers/media/usb/em28xx/em28xx-video.c | 2 +- 1

[PATCH] ata: pata_pdc2027x: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/ata/pata_pdc2027x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/ata/pata_pdc2027x.c b/drive

[PATCH] ata: pata_pdc2027x: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav --- drivers/ata/pata_pdc2027x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/ata/pata_pdc2027x.c b/drivers/ata/pata_pdc2027x.c index 82bfd51..ffd8d33

[PATCH] atm: iphase: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/atm/iphase.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/atm/iphase.c b/drivers/atm/iphase.c index fc

[PATCH] atm: iphase: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav --- drivers/atm/iphase.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/atm/iphase.c b/drivers/atm/iphase.c index fc72b763..a785c6e 100644

[PATCH] tty: ehv_bytechan: fix spelling mistake

Trivial fix to spelling mistakes "interupt" -> "interrupt". Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/tty/ehv_bytechan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/ehv_bytechan.c b/drivers/tty

[PATCH] tty: ehv_bytechan: fix spelling mistake

Trivial fix to spelling mistakes "interupt" -> "interrupt". Signed-off-by: Arvind Yadav --- drivers/tty/ehv_bytechan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/ehv_bytechan.c b/drivers/tty/ehv_bytechan.c index a1c7125..ec0989d 1

[PATCH] mtd: mtdswap: fix spelling mistake: 'TRESHOLD' -> 'THRESHOLD'

Trivial fix to spelling mistakes. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/mtd/mtdswap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/mtdswap.c b/drivers/mtd/mtdswap.c index 7d9080e..f07492c 100644 --- a/drivers/mtd/mtd

[PATCH] mtd: mtdswap: fix spelling mistake: 'TRESHOLD' -> 'THRESHOLD'

Trivial fix to spelling mistakes. Signed-off-by: Arvind Yadav --- drivers/mtd/mtdswap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/mtdswap.c b/drivers/mtd/mtdswap.c index 7d9080e..f07492c 100644 --- a/drivers/mtd/mtdswap.c +++ b/drivers/mtd/mtdswap.c

[PATCH] scsi: scsi_transport_iscsi: fix spelling mistake: 'Cound' -> 'Could'

Trivial fix to spelling mistakes in 'iscsi_get_host_stats'. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/scsi/scsi_transport_iscsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_transport_iscsi.c b/driver

[PATCH] scsi: scsi_transport_iscsi: fix spelling mistake: 'Cound' -> 'Could'

Trivial fix to spelling mistakes in 'iscsi_get_host_stats'. Signed-off-by: Arvind Yadav --- drivers/scsi/scsi_transport_iscsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_transport_iscsi.c b/drivers/scsi/scsi_transport_iscsi.c index 8934f19..590a56c

[PATCH] mfd: pcf50633: fix spelling mistake: 'Falied' -> 'Failed'

Trivial fix to spelling mistakes in 'pcf50633_client_dev_register'. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/mfd/pcf50633-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/pcf50633-core.c b/drivers/mfd/pcf50633-core.c index 6

[PATCH] mfd: pcf50633: fix spelling mistake: 'Falied' -> 'Failed'

Trivial fix to spelling mistakes in 'pcf50633_client_dev_register'. Signed-off-by: Arvind Yadav --- drivers/mfd/pcf50633-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/pcf50633-core.c b/drivers/mfd/pcf50633-core.c index 6155d12..f952dff 100644

[PATCH] leds: lp55xx: fix spelling mistake: 'cound' -> 'could'

Trivial fix to spelling mistakes in 'lp5523_init_program_engine'. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/leds/leds-lp5523.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c index 9

[PATCH] leds: lp55xx: fix spelling mistake: 'cound' -> 'could'

Trivial fix to spelling mistakes in 'lp5523_init_program_engine'. Signed-off-by: Arvind Yadav --- drivers/leds/leds-lp5523.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c index 924e50a..52b6f529 100644 --- a/drivers

[PATCH] libata: fix spelling mistake: 'ambigious' -> 'ambiguous'

Trivial fix to spelling mistakes in ata_parse_force_one(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/ata/libata-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 1945a8e..b3

[PATCH] libata: fix spelling mistake: 'ambigious' -> 'ambiguous'

Trivial fix to spelling mistakes in ata_parse_force_one(). Signed-off-by: Arvind Yadav --- drivers/ata/libata-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 1945a8e..b38df2db 100644 --- a/drivers/ata/libata

[PATCH] staging: rtl8192e: set priv->irq as 0 after the irq is freed

_rtl92e_init can fail here, we must set priv->irq as 0 after free_irq. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c b/driver

[PATCH] staging: rtl8192e: set priv->irq as 0 after the irq is freed

_rtl92e_init can fail here, we must set priv->irq as 0 after free_irq. Signed-off-by: Arvind Yadav --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c b/drivers/staging/rtl8192e/rtl8192e/rtl_cor

[PATCH] staging: rtl8723bs: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/

[PATCH] staging: rtl8723bs: Fix space before '[' error.

Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav --- drivers/staging/rtl8723bs/os_dep/rtw_proc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/rtw_proc.c b/drivers/staging

[RFT] media: dvb_frontend: Fix use-after-free in __dvb_frontend_free

Here, dvb_free_device will free dvb_device. dvb_frontend_invoke_release is using dvb_device after free. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov (usb/media/dtt200u: use-after-free in __dvb_frontend_free). drivers/media/dv

[RFT] media: dvb_frontend: Fix use-after-free in __dvb_frontend_free

Here, dvb_free_device will free dvb_device. dvb_frontend_invoke_release is using dvb_device after free. Signed-off-by: Arvind Yadav --- This bug report by Andrey Konovalov (usb/media/dtt200u: use-after-free in __dvb_frontend_free). drivers/media/dvb-core/dvb_frontend.c | 4 ++-- 1 file

[PATCH] thermal: cpu_cooling: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/t

[PATCH] thermal: cpu_cooling: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 908a801

[PATCH] media: imon: Fix null-ptr-deref in imon_probe

It seems that the return value of usb_ifnum_to_if() can be NULL and needs to be checked. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov usb/media/imon: null-ptr-deref in imon_probe drivers/media/rc/imon.c | 5 + 1 file changed, 5 inse

[PATCH] media: imon: Fix null-ptr-deref in imon_probe

It seems that the return value of usb_ifnum_to_if() can be NULL and needs to be checked. Signed-off-by: Arvind Yadav --- This bug report by Andrey Konovalov usb/media/imon: null-ptr-deref in imon_probe drivers/media/rc/imon.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers

[PATCH] sh: clkfwk: use '"%s:", __func__' instead of function name.

The script "checkpatch.pl" pointed information out like the following. WARNING: Prefer using '"%s...", __func__' to using 'sh_clk_init_parent', this function's name, in a string. pr_err() strings should end with newlines. Add '\n' to fix this. Signed-off-by: Arvind

[PATCH] sh: clkfwk: use '"%s:", __func__' instead of function name.

The script "checkpatch.pl" pointed information out like the following. WARNING: Prefer using '"%s...", __func__' to using 'sh_clk_init_parent', this function's name, in a string. pr_err() strings should end with newlines. Add '\n' to fix this. Signed-off-by: Arvind Yadav

Re: [PATCH] tty: n_gsm: pr_err() strings should end with newlines

Hi, On Monday 09 October 2017 05:30 PM, Joe Perches wrote: On Mon, 2017-10-09 at 17:14 +0530, Arvind Yadav wrote: pr_err() messages should end with a new-line to avoid other messages being concatenated. [] diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c [] @@ -2788,7 +2788,7

Re: [PATCH] tty: n_gsm: pr_err() strings should end with newlines

Hi, On Monday 09 October 2017 05:30 PM, Joe Perches wrote: On Mon, 2017-10-09 at 17:14 +0530, Arvind Yadav wrote: pr_err() messages should end with a new-line to avoid other messages being concatenated. [] diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c [] @@ -2788,7 +2788,7

[PATCH] thermal/intel_powerclamp: pr_err()/pr_info() strings should end with newlines

pr_err()/pr_info() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/thermal/intel_powerclamp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/t

[PATCH] thermal/intel_powerclamp: pr_err()/pr_info() strings should end with newlines

pr_err()/pr_info() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/thermal/intel_powerclamp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal

[PATCH] tty: n_gsm: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/tty/n_gsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c index 0

[PATCH] tty: n_gsm: pr_err() strings should end with newlines

pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav --- drivers/tty/n_gsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c index 0a3c966..b27986f 100644 --- a/drivers

<    1   2   3   4   5   6   7   8   9   10   >