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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/misc
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
fs/ncpfs/inode.c | 12 +---
1 file changed, 9 insertions(+), 3
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
arch/arm/mach
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
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
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
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add missing .owner of struct snd_soc_card. This prevents the
module from being removed from underneath its users.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/au1x/db1200.c | 4
1 file changed, 4 insertions(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add missing .owner of struct snd_soc_card. This prevents the
module from being removed from underneath its users.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/ux500/mop500.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add missing .owner of struct snd_soc_card. This prevents the
module from being removed from underneath its users.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/fsl/imx-mc13783.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add missing .owner of struct snd_soc_card. This prevents the
module from being removed from underneath its users.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/pxa/brownstone.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add missing .owner of struct snd_soc_card. This prevents the
module from being removed from underneath its users.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/pxa/ttc-dkb.c | 1 +
1 file changed, 1 insertion(+)
diff
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
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
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/pinctrl/vt8500/pinctrl-wmt.c | 4 +---
1 file changed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/power/avs/smartreflex.c | 4 +---
1 file changed, 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
devm_* APIs are device managed and make code simpler.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/mid-x86/mfld_machine.c | 29 ++---
1 file changed, 10 insertions(+), 19 deletions(-)
diff --git
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The dereference should be moved below the NULL test.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/i915/i915_gem_context.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915
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
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
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
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
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix missing free_netdev() before return from function xlr_net_probe()
in the devm_ioremap_resource() error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
no compile test.
---
drivers/staging/netlogic/xlr_net.c | 6
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the kmalloc() error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/firewire/scs1x.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion
Fix missing free_netdev() before return from function xlr_net_probe()
in the devm_ioremap_resource() error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: remove redundant error message.
---
drivers/staging/netlogic/xlr_net.c | 5 ++---
1 file changed, 2
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/vme/bridges/vme_tsi148.c | 3 ++-
1 file
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/mfd/htc-egpio.c | 4
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
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
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
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/staging/tidspbridge/rmgr/drv_interface.c | 1 +
1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/memstick/host/r592.c | 11 +++
1 file changed, 7
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/pcmcia/pd6729.c | 2 ++
1 file changed, 2 insertions(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
fs/ubifs/super.c | 8 ++--
1 file changed, 6
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/misc
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
no compile test
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
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/misc/spear13xx_pcie_gadget.c | 8
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 'err'.
Signed-off
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 error code i
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
sound/pci/sis7019.c | 3 ++-
1 file changed, 2
On 05/31/2013 02:29 AM, Andy Shevchenko wrote:
On Thu, May 30, 2013 at 7:32 AM, Wei Yongjun weiyj...@gmail.com 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 error code instead
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
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 'err'.
Signed-off
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
From: Wei Yongjun
Fix to return -ENOMEM in the kmap() error handling case
instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915
From: Wei Yongjun
Add the missing iounmap() before return from d40_probe()
in the error handling case.
Signed-off-by: Wei Yongjun
---
drivers/dma/ste_dma40.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 71bf4ec..018c2f7 100644
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 'err'.
Signed-off
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing iounmap() before return from d40_probe()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/dma/ste_dma40.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/dma/ste_dma40.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the kmap() error handling case
instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 6 --
1 file changed, 4 insertions
From: Wei Yongjun
Fix to return a negative error code from the devm_clk_get() error
handling case instead of 0, as done elsewhere in this function.
Introduced by commit 950707c0eb5c7aeaa2c446a04c824f4be686d2f6
(pinctrl: sunxi: add clock support)
Signed-off-by: Wei Yongjun
---
drivers/pinctrl
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the devm_clk_get() error
handling case instead of 0, as done elsewhere in this function.
Introduced by commit 950707c0eb5c7aeaa2c446a04c824f4be686d2f6
(pinctrl: sunxi: add clock support)
Signed-off-by: Wei
From: Wei Yongjun
Fix to return a negative error code in the irq descs alloc error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/gpio/gpio-ml-ioh.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpio/gpio-ml-ioh.c b/drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code in the irq descs alloc error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpio/gpio-ml-ioh.c | 1 +
1 file changed, 1
From: Wei Yongjun
Remove including that don't need it.
Signed-off-by: Wei Yongjun
---
drivers/staging/btmtk_usb/btmtk_usb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/btmtk_usb/btmtk_usb.c
b/drivers/staging/btmtk_usb/btmtk_usb.c
index 6b01f61..5d0a952 100644
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove including linux/version.h that don't need it.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/btmtk_usb/btmtk_usb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/btmtk_usb/btmtk_usb.c
b
From: Wei Yongjun
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Change function exynos_plane_init() to return ERR_PTR(), so
the caller can use the error code from it.
Signed-off-by: Wei Yongjun
---
v1 -> v2: cha
From: Wei Yongjun
When return from wl_adapter_insert() in the register_netdev() error
handling case, 'ret' which is 0 is returned, but we should return a
negative error code instead, so fix to return the return value of
register_netdev().
Introduce by commit
From: Wei Yongjun
Fix to return -ENOMEM in the kmem_cache_create() error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/mtd/ubi/build.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/mtd/ubi/build.c b
From: Wei Yongjun
Since we will remove items off the list using list_del_init() we need
to use a safe version of the list_for_each_entry() macro aptly named
list_for_each_entry_safe().
Signed-off-by: Wei Yongjun
---
drivers/staging/lustre/lustre/osc/osc_cache.c | 4 ++--
1 file changed, 2
From: Wei Yongjun
Add the missing misc_deregister() on probe error and remove.
Signed-off-by: Wei Yongjun
---
arch/arm/kernel/etm.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/kernel/etm.c b/arch/arm/kernel/etm.c
index 8ff0ecd..d0b7426 100644
--- a/arch
From: Wei Yongjun
Add the missing platform_device_put() before return from
try_smi_init() in the platform_device_add() fail case.
Signed-off-by: Wei Yongjun
---
drivers/char/ipmi/ipmi_si_intf.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing platform_device_put() before return from
try_smi_init() in the platform_device_add() fail case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/char/ipmi/ipmi_si_intf.c | 1 +
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing misc_deregister() on probe error and remove.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/arm/kernel/etm.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/kernel/etm.c b/arch
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Since we will remove items off the list using list_del_init() we need
to use a safe version of the list_for_each_entry() macro aptly named
list_for_each_entry_safe().
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the kmem_cache_create() error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mtd/ubi/build.c | 4 +++-
1 file changed, 3 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
When return from wl_adapter_insert() in the register_netdev() error
handling case, 'ret' which is 0 is returned, but we should return a
negative error code instead, so fix to return the return value of
register_netdev().
Introduce by commit
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Change function exynos_plane_init() to return ERR_PTR(), so
the caller can use the error code from it.
Signed-off-by: Wei Yongjun
From: Wei Yongjun
If pdev->dev.platform_data is not set, mcfqspi_probe() will return 0
and release all the resources, in this case, we should return a error
code instead of 0.
This patch fix to return -ENOENT in this case and move the check for
pdev->dev.platform_data to the
From: Wei Yongjun
Fix to return -ENOMEM in the workqueue create error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/spi/spi-ep93xx.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/spi/spi-ep93xx.c b/drivers/spi/spi-ep93xx.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the workqueue create error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-ep93xx.c | 1 +
1 file changed, 1 insertion(+)
diff
801 - 900 of 1371 matches
Mail list logo