[PATCH] clk: fix return value check in bcm2835_init_clocks()

From: Wei Yongjun In case of error, the function clk_register_fixed_rate() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei

[PATCH] perf tools: remove duplicated include from trace-event-python.c

From: Wei Yongjun Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- tools/perf/util/scripting-engines/trace-event-python.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/perf/util/scripting

[PATCH] drivers:misc: ti-st: fix potential NULL pointer dereference in st_register()

From: Wei Yongjun The dereference should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/misc/ti-st/st_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a

[PATCH] staging: r8712u: fix potential NULL pointer dereference in r871x_wps_start()

From: Wei Yongjun The dereference should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH v2] drivers:misc: ti-st: fix potential NULL pointer dereference in st_register()

From: Wei Yongjun Remove the pointless NULL dereference above the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- v1 -> v2: drop the NULL dereference --- drivers/misc/ti-st/st_core.c | 1 - 1 file changed

[PATCH] hwmon: ina2xx: use module_i2c_driver to simplify the code

From: Wei Yongjun Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/hwmon/ina2xx.c | 13 + 1 file changed, 1 insertion(+), 12

[PATCH] powerpc/windfarm: use module_i2c_driver to simplify the code

From: Wei Yongjun Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/macintosh/windfarm_smu_sat.c| 13 + drivers

[PATCH] w1/ds2482: use module_i2c_driver to simplify the code

From: Wei Yongjun Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/w1/masters/ds2482.c | 13 + 1 file changed, 1 insertion

[PATCH] mfd: use module_i2c_driver to simplify the code

From: Wei Yongjun Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/mfd/adp5520.c | 12 +--- 1 file changed, 1 insertion(+), 11

[PATCH v2] gma500: medfield: drop bogus NULL check in mdfld_dsi_output_init()

From: Wei Yongjun Drop the NULL test for dev since it never be NULL. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] clk: fix return value check in bcm2835_init_clocks()

On 10/09/2012 10:31 AM, Stephen Warren wrote: > On 10/07/2012 08:02 AM, Wei Yongjun wrote: >> From: Wei Yongjun >> >> In case of error, the function clk_register_fixed_rate() returns >> ERR_PTR() and never returns NULL. The NULL test in the return value >> che

[PATCH] ubi: use list_move_tail instead of list_del/list_add_tail

From: Wei Yongjun Using list_move_tail() instead of list_del() + list_add_tail(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/mtd/ubi/fastmap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH] pinctrl: remove duplicated include from pinctrl-xway.c

From: Wei Yongjun Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/pinctrl/pinctrl-xway.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/pinctrl/pinctrl-xway.c b/drivers

[PATCH] block, sx8: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/block/sx8.c | 13 + 1 file

[PATCH] pch_dma: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/dma/pch_dma.c | 13 + 1 file

[PATCH] virtio-pci: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/virtio/virtio_pci.c | 14

[PATCH -next] drm: tegra: fix missing unlock on error

From: Wei Yongjun Add the missing unlock before return from function host1x_drm_init() and host1x_drm_exit() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/tegra/host1x.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/tegra/host1x.c b/drivers

[PATCH -next] xtensa: use for_each_compatible_node() macro

From: Wei Yongjun Use for_each_compatible_node() macro. Signed-off-by: Wei Yongjun --- arch/xtensa/platforms/xtfpga/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/xtensa/platforms/xtfpga/setup.c b/arch/xtensa/platforms/xtfpga/setup.c index 71d61ca6

[PATCH -next] f2fs: remove unused variable

From: Wei Yongjun The variables node_page and page_offset are initialized but never used otherwise, so remove those unused variables. Signed-off-by: Wei Yongjun --- fs/f2fs/file.c | 2 -- fs/f2fs/dir.c | 2 -- 2 files changed, 4 deletions(-) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c

[PATCH -next] mfd: sta2x11-mfd: fix potential NULL pointer dereference in __sta2x11_mfd_mask()

From: Wei Yongjun The dereference to 'mfd' should be moved below the NULL test. Signed-off-by: Wei Yongjun --- drivers/mfd/sta2x11-mfd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/sta2x11-mfd.c b/drivers/mfd/sta2x11-mfd.c index 012a521..1ecb

[PATCH -next] mfd: sta2x11-mfd: fix potential NULL pointer dereference in __sta2x11_mfd_mask()

From: Wei Yongjun The dereference to 'mfd' should be moved below the NULL test. Signed-off-by: Wei Yongjun --- drivers/mfd/sta2x11-mfd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/sta2x11-mfd.c b/drivers/mfd/sta2x11-mfd.c index 012a521..1ecb

[PATCH -next] firewire: net: remove unused variable in fwnet_receive_broadcast()

From: Wei Yongjun The variable card is initialized but never used otherwise, so remove the unused variable. Signed-off-by: Wei Yongjun --- drivers/firewire/net.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/firewire/net.c b/drivers/firewire/net.c index 638e1f7..e7a711f5 100644

[PATCH -next] tile/PCI: use for_each_pci_dev to simplify the code

From: Wei Yongjun Use for_each_pci_dev to simplify the code. Signed-off-by: Wei Yongjun --- arch/tile/kernel/pci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/tile/kernel/pci.c b/arch/tile/kernel/pci.c index 7598226..aac1cd5 100644 --- a/arch/tile/kernel/pci.c

[PATCH -next] mn10300: use for_each_pci_dev to simplify the code

From: Wei Yongjun Use for_each_pci_dev to simplify the code. Signed-off-by: Wei Yongjun --- arch/mn10300/unit-asb2305/pci-irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mn10300/unit-asb2305/pci-irq.c b/arch/mn10300/unit-asb2305/pci-irq.c index 91212ea..77439da

[PATCH -next] Drivers: hv: remove unused variable in vmbus_recvpacket_raw()

From: Wei Yongjun The variable userlen is initialized but never used otherwise, so remove the unused variable. Signed-off-by: Wei Yongjun --- drivers/hv/channel.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index 773a2f2..3148d80 100644

[PATCH -next] TTY: hvsi: use for_each_compatible_node() macro

From: Wei Yongjun Use for_each_compatible_node() macro instead of open coding it. Signed-off-by: Wei Yongjun --- drivers/tty/hvc/hvsi.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/tty/hvc/hvsi.c b/drivers/tty/hvc/hvsi.c index 5b95b4f..70e0ef7 100644 --- a

[PATCH -next] powerpc/82xx: use for_each_compatible_node() macro

From: Wei Yongjun Use for_each_compatible_node() macro instead of open coding it. Signed-off-by: Wei Yongjun --- arch/powerpc/platforms/82xx/pq2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/platforms/82xx/pq2.c b/arch/powerpc/platforms/82xx/pq2.c

[PATCH] CRIS: locking: fix the return value of arch_read_trylock()

From: Wei Yongjun arch_write_trylock() should return 'ret' instead of always return 1. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- arch/cris/include/arch-v32/arch/spinlock.h | 2 +- 1 file changed, 1 inser

Re: CONFIG_MODULE_SIG breaks out-of-tree modules in modpost.

On 10/15/2012 08:19 PM, David Howells wrote: > Valdis Kletnieks wrote: > >> /bin/sh: -c: line 0: syntax error near unexpected token `;' >> /bin/sh: -c: line 0: `set -e; ; echo >> 'cmd_/usr/src/valdis/NVIDIA-Linux-x86_64-304.51/kernel/nvidia.ko := ' > >> /usr/src/valdis/NVIDIA-Linux-x86_64-304.5

[PATCH] ALSA: snd-usb: remove unused variable in init_pitch_v2()

From: Wei Yongjun The variable ep is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- sound/usb/pcm.c | 3 --- 1 file changed, 3 deletions(-) diff --git a

[PATCH] ASoC: jack: remove unused variable in snd_soc_jack_report()

From: Wei Yongjun The variable oldstatus is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- sound/soc/soc-jack.c | 3 --- 1 file changed, 3 deletions

[PATCH] fuse: remove unused variable in fuse_try_move_page()

From: Wei Yongjun The variables mapping,index are initialized but never used otherwise, so remove the unused variables. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- fs/fuse/dev.c | 5 - 1 file changed, 5 deletions

[PATCH] dlm: remove unused variable in *dlm_lowcomms_get_buffer()

From: Wei Yongjun The variable users is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- fs/dlm/lowcomms.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH] vme: vme_tsi148.c: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/vme/bridges/vme_tsi148.c | 15

[PATCH] vme: vme_ca91cx42.c: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/vme/bridges/vme_ca91cx42.c | 15

[PATCH] vme: vme_vmivme7805.c: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/vme/boards/vme_vmivme7805.c | 15

[PATCH] Staging: ipack/bridges/tpci200: use module_pci_driver to simplify the code

From: Wei Yongjun Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/staging/ipack/bridges/tpci200.c | 12

[PATCH] dmaengine: imx-dma: fix missing unlock on error in imxdma_xfer_desc()

From: Wei Yongjun Add the missing unlock on the error handling path in function imxdma_xfer_desc(). Signed-off-by: Wei Yongjun --- drivers/dma/imx-dma.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/dma/imx-dma.c b/drivers/dma/imx-dma.c index f11b5b2..7d9554c

[PATCH] TTY: hvcs: fix missing unlock on error in hvcs_initialize()

From: Wei Yongjun Add the missing unlock on the error handling path in function hvcs_initialize(). Signed-off-by: Wei Yongjun --- drivers/tty/hvc/hvcs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/tty/hvc/hvcs.c b/drivers/tty/hvc/hvcs.c index cab5c7a

[PATCH] ARM: PCI: fix missing unlock on error

From: Wei Yongjun Add the missing unlock on the error handle path in function nanoengine_read_config() and nanoengine_write_config(). Signed-off-by: Wei Yongjun --- arch/arm/mach-sa1100/pci-nanoengine.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach

[PATCH] ARM: PCI: fix missing unlock on error

From: Wei Yongjun Add the missing unlock on the error handle path in function nanoengine_read_config() and nanoengine_write_config(). Signed-off-by: Wei Yongjun --- arch/arm/mach-sa1100/pci-nanoengine.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach

[PATCH] pinctrl: fix missing unlock on error in pinctrl_groups_show()

From: Wei Yongjun Add the missing unlock on the error handle path in function pinctrl_groups_show(). Signed-off-by: Wei Yongjun --- drivers/pinctrl/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c index 0f1ec9e

[PATCH] Staging: vt6655: fix missing unlock on error in vCommandTimer()

From: Wei Yongjun Add the missing unlock on the error handle path in function vCommandTimer. Signed-off-by: Wei Yongjun --- drivers/staging/vt6655/wcmd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/vt6655/wcmd.c b/drivers/staging/vt6655/wcmd.c index 94bd1fc..6d0b87a

[PATCH] Staging: ced1401: fix missing unlock on error in FreeCircBlock()

From: Wei Yongjun Add the missing unlock on the error handle path in function FreeCircBlock(). Signed-off-by: Wei Yongjun --- drivers/staging/ced1401/ced_ioc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401

[PATCH] ALSA: hda - remove unused variable in azx_position_ok()

From: Wei Yongjun The variable stream is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- sound/pci/hda/hda_intel.c | 2 -- 1 file changed, 2 deletions

[PATCH] aoe: remove unused variable in resend()

From: Wei Yongjun The variable ah is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/block/aoe/aoecmd.c | 2 -- 1 file changed, 2 deletions

[PATCH] aoe: remove unused variable in resend()

From: Wei Yongjun The variable ah is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/block/aoe/aoecmd.c | 2 -- 1 file changed, 2 deletions

[PATCH] Drivers: hv: remove unused variable from channel_mgmt.c

From: Wei Yongjun The variables guidtype, guidinstance and initiate are initialized but never used otherwise, so remove the unused variables. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/hv/channel_mgmt.c | 8

Re: [PATCH] ASoC: jack: remove unused variable in snd_soc_jack_report()

On 10/22/2012 09:50 PM, Mark Brown wrote: > On Thu, Oct 18, 2012 at 10:49:19PM +0800, Wei Yongjun wrote: >> From: Wei Yongjun >> >> The variable oldstatus is initialized but never used >> otherwise, so remove the unused variable. > Applied, thanks. However for chang

[PATCH -next] virtio: balloon: fix missing unlock on error in fill_balloon()

From: Wei Yongjun Add the missing unlock before return from function fill_balloon() in the error handling case. Introduced by 9864a8(virtio_balloon: introduce migration primitives to balloon pages) dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off

[PATCH -next] ACPI: fix missing unlock on error in acpi_memory_remove_memory()

From: Wei Yongjun Add the missing mutex_unlock() before return from function acpi_memory_remove_memory() in the error handling case. Introduce by commit 85fcb3758c10e063a2a30dfad75017097999deed 'ACPI / memory-hotplug: introduce a mutex lock to protect the list in acpi_memory_device

[PATCH -next] ASoC: cs42l52: fix the return value of cs42l52_set_fmt()

From: Wei Yongjun Fix the return value of cs42l52_set_fmt() when clock inversion is not allowed and also remove the useless variable ret. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- sound/soc/codecs/cs42l52.c | 3 +-- 1

Re: [PATCH -next] ACPI: fix missing unlock on error in acpi_memory_remove_memory()

Hi Wen, On 11/08/2012 09:31 AM, Wen Congyang wrote: > Hi, wei > > At 11/07/2012 08:38 PM, Wei Yongjun Wrote: >> From: Wei Yongjun >> >> Add the missing mutex_unlock() before return from function >> acpi_memory_remove_memory() in the error handlin

Re: [PATCH] rtc: avoid calling platform_device_put() twice in test_init()

On 11/07/2012 08:51 AM, Andrew Morton wrote: > On Tue, 23 Oct 2012 13:08:41 +0800 > Wei Yongjun wrote: > >> From: Wei Yongjun >> >> In case of error, the function test_init() need to call >> platform_device_del() instead of platform_device_unregi

[PATCH -next] staging: comedi: usbduxfast: remove unused variable in usbduxfastsub_ai_Irq()

From: Wei Yongjun The variable 'p' is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/staging/comedi/drivers/usbduxfast.c | 2 -- 1 file

[PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

From: Wei Yongjun The dereference to port should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/block/mtip32xx/mtip32xx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

Re: [PATCH] spi: omap2-mcspi: remove duplicated include from spi-omap2-mcspi.c

On 11/14/2012 11:54 PM, Grant Likely wrote: > On Sun, 7 Oct 2012 21:29:21 +0800, Wei Yongjun wrote: >> From: Wei Yongjun >> >> Remove duplicated include. > Applied, thanks. > >> dpatch engine is used to auto generate this patch. >> (https://github.com/w

[PATCH -next] ALSA: hda - fix error return code in patch_alc662()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun --- sound/pci/hda/patch_realtek.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda

[PATCH -next] spi: omap2-mcspi: fix error return code in omap2_mcspi_probe()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/spi/spi-omap2-mcspi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-omap2

[PATCH -next] mei: fix krealloc() misuse in in mei_cl_irq_read_msg()

From: Wei Yongjun If krealloc() returns NULL, it doesn't free the original. So any code of the form 'foo = krealloc(foo, ...);' is almost certainly a bug. Introduced by commit fcb136e1ac5774909e0d85189f721b8dfa800e0f(mei: fix reading large reposnes) Signed-off-by: Wei Yongju

[PATCH -next] iommu/amd: fix error return code in early_amd_iommu_init()

From: Wei Yongjun Fix to return -ENOMEM int the memory alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/iommu/amd_iommu_init.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iommu/amd_iommu_init.c b/drivers/iommu

[PATCH] pch_phub: fix error return code in pch_phub_probe()

From: Wei Yongjun Error code had been assigned to retval in some error handling case but never use it, 0 is always returned in those case, this patch fix them to return a negative error code. Signed-off-by: Wei Yongjun --- drivers/misc/pch_phub.c | 36 +--- 1

[PATCH -next] staging: octeon-usb: remove duplicated include from octeon-hcd.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/staging/octeon-usb/octeon-hcd.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index b78bd19..39dc13d 100644 --- a/drivers

[PATCH] staging: tidspbridge: fix error return code in omap34_xx_bridge_probe()

From: Wei Yongjun Fix to return a negative error code in the class_create() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/staging/tidspbridge/rmgr/drv_interface.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging

[PATCH] memstick: fix error return code in r592_probe()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/memstick/host/r592.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/memstick

[PATCH] pcmcia: pd6729: fix error return code in pd6729_pci_probe()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/pcmcia/pd6729.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/pcmcia/pd6729.c b/drivers/pcmcia/pd6729.c

[PATCH] UBIFS: fix error return code in ubifs_remount_rw() and ubifs_fill_super()

From: Wei Yongjun Fix to return -ENOMEM in the kmalloc() and d_make_root() error handling case instead of 0, as done elsewhere in those functions. Signed-off-by: Wei Yongjun --- fs/ubifs/super.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/fs/ubifs/super.c b/fs

[PATCH -next] pinctrl: core: fix missing unlock on error in pinctrl_find_gpio_range_from_pin()

From: Wei Yongjun Add the missing unlock before return from function pinctrl_find_gpio_range_from_pin() in the error handling case. Introduced by commit 2ff3477efd7086544b9e298fc63afab0645921b4. (pinctrl: add pin list based GPIO ranges) Signed-off-by: Wei Yongjun --- drivers/pinctrl/core.c

[PATCH -next] Staging: netlogic: fix missing free_netdev() on error in xlr_net_probe()

From: Wei Yongjun Fix missing free_netdev() before return from function xlr_net_probe() in the devm_ioremap_resource() error handling case. Signed-off-by: Wei Yongjun --- no compile test. --- drivers/staging/netlogic/xlr_net.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH] ALSA: firewire: fix error return code in scs_probe()

From: Wei Yongjun Fix to return -ENOMEM in the kmalloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- sound/firewire/scs1x.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/firewire/scs1x.c b/sound/firewire

[PATCH -next v2] Staging: netlogic: fix missing free_netdev() on error in xlr_net_probe()

Fix missing free_netdev() before return from function xlr_net_probe() in the devm_ioremap_resource() error handling case. Signed-off-by: Wei Yongjun --- v1 -> v2: remove redundant error message. --- drivers/staging/netlogic/xlr_net.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deleti

[PATCH] vme: vme_tsi148.c: fix error return code in tsi148_probe()

From: Wei Yongjun Fix to return a negative error code in the tsi148_crcsr_init() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/vme/bridges/vme_tsi148.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] mfd: htc-egpio: use devm_ioremap_nocache() instead of ioremap_nocache()

From: Wei Yongjun Replace probe-time ioremap_nocache() call with devm_ioremap_nocache() to avoid iounmap() missing and get rid of the corresponding iounmap() call on remove. Signed-off-by: Wei Yongjun --- drivers/mfd/htc-egpio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH -next] dlm: remove duplicated include from lowcomms.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- fs/dlm/lowcomms.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 4f539dd..d90909e 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c @@ -52,7 +52,6 @@ #include

[PATCH -next] arm: remove duplicated include from process.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/kernel/process.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c index 7591df5..f89ff80 100644 --- a/arch/arm/kernel/process.c +++ b/arch/arm/kernel

[PATCH -next] drm/i915: fix potential NULL pointer dereference in i915_gem_context_get_hang_stats()

From: Wei Yongjun The dereference should be moved below the NULL test. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_gem_context.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915

[PATCH -next] rsxx: fix error return code in rsxx_pci_probe()

From: Wei Yongjun Fix to return -ENOMEM in the create_singlethread_workqueue() error handling case instead of 0, as done elsewhere in this function. Introduced by commit a3299ab18591d36ad5622f5064619123c439b779. (rsxx: Individual workqueues for interruptible events.) Signed-off-by: Wei Yongjun

[PATCH -next v2] dmaengine: ste_dma40: fix error return code in d40_probe()

From: Wei Yongjun In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead in those error handling case. This patch fixed them, and also removed useless variable 'er

[PATCH] ALSA: sis7019: fix error return code in sis_chip_create()

From: Wei Yongjun Fix to return a negative error code in the pci_set_dma_mask() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- sound/pci/sis7019.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/sis7019.c b

Re: [PATCH -next v2] dmaengine: ste_dma40: fix error return code in d40_probe()

On 05/31/2013 02:29 AM, Andy Shevchenko wrote: > On Thu, May 30, 2013 at 7:32 AM, Wei Yongjun wrote: >> In many of the error handling case, the return value 'ret' not set >> and 0 will be return from d40_probe() even if error, but we should >> return a negative er

[PATCH -next v3] dmaengine: ste_dma40: fix error return code in d40_probe()

From: Wei Yongjun In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead in those error handling case. This patch fixed them, and also removed useless variable 'er

[PATCH] xen-pciback: fix error return code in pcistub_irq_handler_switch()

From: Wei Yongjun Fix to return -ENOENT in the pcistub_device_find() and pci_get_drvdata() error handling case instead of 0(overwrite to 0 by str_to_slot()), as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/xen/xen-pciback/pci_stub.c | 10 +++--- 1 file changed

[PATCH] pcie-gadget-spear: fix error return code in spear_pcie_gadget_probe()

From: Wei Yongjun Fix to return a negative error code in the clk_get_sys() and clk_enable() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/misc/spear13xx_pcie_gadget.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions

[PATCH -next] staging/lustre: fix return value check in libcfs_sock_ioctl()

From: Wei Yongjun In case of error, the function sock_alloc_file() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- no compile test --- drivers/staging/lustre/lustre/libcfs/linux/linux-tcpip.c

[PATCH -next] ASoC: mid-x86: Convert to use devm_* APIs

From: Wei Yongjun devm_* APIs are device managed and make code simpler. Signed-off-by: Wei Yongjun --- sound/soc/mid-x86/mfld_machine.c | 29 ++--- 1 file changed, 10 insertions(+), 19 deletions(-) diff --git a/sound/soc/mid-x86/mfld_machine.c b/sound/soc/mid-x86

[PATCH -next] pinctrl: vt8500: wmt: remove redundant dev_err call in wmt_pinctrl_probe()

From: Wei Yongjun There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/pinctrl/vt8500/pinctrl-wmt.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers

[PATCH -next] PM / AVS: SmartReflex: remove redundant dev_err call in omap_sr_probe()

From: Wei Yongjun There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/power/avs/smartreflex.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/power/avs

[PATCH -next] ARM: edma: remove duplicated include from edma.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/common/edma.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index a432e6c..39ad030 100644 --- a/arch/arm/common/edma.c +++ b/arch/arm/common/edma.c

[PATCH] ASoC: db1200: add .owner to struct snd_soc_card

From: Wei Yongjun Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun --- sound/soc/au1x/db1200.c | 4 1 file changed, 4 insertions(+) diff --git a/sound/soc/au1x/db1200.c b/sound/soc/au1x/db1200.c

[PATCH] ASoC: mop500: add .owner to struct snd_soc_card

From: Wei Yongjun Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun --- sound/soc/ux500/mop500.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/ux500/mop500.c b/sound/soc/ux500/mop500.c

[PATCH] ASoC: imx_mc13783: add .owner to struct snd_soc_card

From: Wei Yongjun Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun --- sound/soc/fsl/imx-mc13783.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/fsl/imx-mc13783.c b/sound/soc/fsl/imx

[PATCH] ASoC: brownstone: add .owner to struct snd_soc_card

From: Wei Yongjun Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun --- sound/soc/pxa/brownstone.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/pxa/brownstone.c b/sound/soc/pxa

[PATCH] ASoC: ttc_dkb: add .owner to struct snd_soc_card

From: Wei Yongjun Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun --- sound/soc/pxa/ttc-dkb.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/pxa/ttc-dkb.c b/sound/soc/pxa/ttc-dkb.c

[PATCH] c2port: fix return value check in duramar2150_c2port_init()

From: Wei Yongjun In case of error, the function c2port_device_register() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/misc/c2port/c2port-duramar2150.c | 4 ++-- 1 file changed, 2

[PATCH] ncpfs: fix error return code in ncp_parse_options()

From: Wei Yongjun Fix to return -EINVAL from the option parse error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- fs/ncpfs/inode.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/fs/ncpfs/inode.c b/fs/ncpfs

[PATCH] ARM: u300: fix return value check in __u300_init_boardpower()

From: Wei Yongjun In case of error, the function syscon_node_to_regmap() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- arch/arm/mach-u300/regulator.c | 4 ++-- 1 file changed, 2 insertions

[PATCH -next] pinctrl: st: fix return value check

From: Wei Yongjun In case of error, the function pinctrl_register() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. The function syscon_regmap_lookup_by_phandle() returns ERR_PTR() and never returns NULL. The NULL test in the return

[PATCH -next RESEND] pinctrl: st: fix return value check

From: Wei Yongjun In case of error, the function pinctrl_register() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. The function syscon_regmap_lookup_by_phandle() returns ERR_PTR() and never returns NULL. The NULL test in the return

[PATCH -next] loopdev: fix error return code in loop_add()

From: Wei Yongjun Fix to return a negative error code from the error handling case, as returned elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/block/loop.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/block/loop.c b/drivers/block/loop.c index 1e0cc5a

[PATCH -next] rpmsg: fix error return code in rpmsg_probe()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/rpmsg/virtio_rpmsg_bus.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/rpmsg

<    1   2   3   4   5   6   7   8   >