From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the skd construct error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/block/skd_main.c | 4 +++-
1 file changed, 3 insertions(+), 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mtd/mtdcore.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c
index 7189089..92311a5 100644
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return ENODEV in the pci ioremap error handling case
instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/misc/tifm_7xx1.c | 4 +++-
1 file changed, 3 insertions(+), 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/arm/mach-ixp4xx/dsmg600-setup.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/mach-ixp4xx/dsmg600-setup.c
b/arch/arm/mach-ixp4xx/dsmg600
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/arm/mach-imx/clk-imx51-imx53.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c
b/arch/arm/mach-imx/clk-imx51-imx53
From: Wei Yongjun
In case of error, the function clk_get_sys() 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-spear/time.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
From: Wei Yongjun
In case of error, the function iget_locked() returns NULL pointer
not ERR_PTR(). The IS_ERR() test in the return value check should
be replaced with NULL test.
Signed-off-by: Wei Yongjun
---
fs/befs/linuxvfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function iget_locked() returns NULL pointer
not ERR_PTR(). The IS_ERR() test in the return value check should
be replaced with NULL test.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
fs/befs/linuxvfs.c | 4
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function clk_get_sys() 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-spear
From: Wei Yongjun
In case of error, the function platform_device_register_simple() 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/staging/imx-drm/imx-drm-core.c | 4 ++--
1 file changed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function platform_device_register_simple() 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
From: Wei Yongjun
Add the missing misc_deregister() before return from
sep_register_driver_with_fs() in the error handling case.
Signed-off-by: Wei Yongjun
---
drivers/staging/sep/sep_main.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
fs/xfs/xfs_bmap.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index e6b4202..92b97f7 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -34,7 +34,6 @@
#include
From: Wei Yongjun
module_platform_driver() makes the code simpler by eliminating
boilerplate code.
Signed-off-by: Wei Yongjun
---
drivers/pinctrl/pinctrl-tb10x.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-tb10x.c b/drivers/pinctrl
From: Wei Yongjun
This patch fix the error handling in tb10x_pinctrl_probe():
- devm_ioremap_resource() return ERR_PTR() and never return NULL
- remove the dev_err call to avoid redundant error message
- pinctrl_register() returns NULL not ERR_PTR()
Signed-off-by: Wei Yongjun
---
drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
module_platform_driver() makes the code simpler by eliminating
boilerplate code.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/pinctrl/pinctrl-tb10x.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
This patch fix the error handling in tb10x_pinctrl_probe():
- devm_ioremap_resource() return ERR_PTR() and never return NULL
- remove the dev_err call to avoid redundant error message
- pinctrl_register() returns NULL not ERR_PTR()
Signed-off
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
fs/xfs/xfs_bmap.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index e6b4202..92b97f7 100644
--- a/fs/xfs
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing misc_deregister() before return from
sep_register_driver_with_fs() in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/sep/sep_main.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun
Use free_bch() instead of kfree() to free init_bch()
allocated data.
Signed-off-by: Wei Yongjun
---
drivers/mtd/devices/docg3.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/devices/docg3.c b/drivers/mtd/devices/docg3.c
index 3e1b0a0
From: Wei Yongjun
Use vfree() instead of kfree() to free vmalloc()
allocated data.
Signed-off-by: Wei Yongjun
---
drivers/staging/crystalhd/crystalhd_lnx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/crystalhd/crystalhd_lnx.c
b/drivers/staging
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use vfree() instead of kfree() to free vmalloc()
allocated data.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/crystalhd/crystalhd_lnx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use free_bch() instead of kfree() to free init_bch()
allocated data.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mtd/devices/docg3.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/devices
From: Wei Yongjun
It's not necessary to free regulator consumers allocated with
devm_regulator_bulk_get.
Signed-off-by: Wei Yongjun
---
sound/soc/codecs/wm8993.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/codecs/wm8993.c b/sound/soc/codecs/wm8993.c
index 433d59a..2ee23a3
From: Wei Yongjun
It's not necessary to free regulator consumers allocated with
devm_regulator_bulk_get.
Signed-off-by: Wei Yongjun
---
sound/soc/codecs/wm8993.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/codecs/wm8993.c b/sound/soc/codecs/wm8993.c
index 433d59a..2ee23a3
From: Wei Yongjun yongjun_...@trendmicro.com.cn
It's not necessary to free regulator consumers allocated with
devm_regulator_bulk_get.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/codecs/wm8993.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/codecs
From: Wei Yongjun yongjun_...@trendmicro.com.cn
It's not necessary to free regulator consumers allocated with
devm_regulator_bulk_get.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/codecs/wm8993.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/codecs
From: Wei Yongjun
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun
---
drivers/mfd/ezx-pcap.c | 7 +--
1 file changed, 1 insertion(+), 6
From: Wei Yongjun
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun
---
drivers/mfd/twl6040.c | 10 ++
1 file changed, 2 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mfd
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mfd
From: Wei Yongjun
irq allocated with devm_request_irq should not be freed using
free_irq, because doing so causes a dangling pointer, and a
subsequent double free.
Signed-off-by: Wei Yongjun
---
drivers/misc/cb710/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Wei Yongjun
usb_free_urb(NULL) is safe and this check is not required.
Signed-off-by: Wei Yongjun
---
drivers/staging/rtl8188eu/os_dep/recv_linux.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/os_dep/recv_linux.c
b/drivers/staging
From: Wei Yongjun
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun
---
drivers/power/tps65090-charger.c | 11 ---
1 file changed, 4
From: Wei Yongjun
Add the missing destroy_workqueue() before return from
i2o_driver_register() in the error handling case.
Signed-off-by: Wei Yongjun
---
drivers/message/i2o/driver.c | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/drivers/message/i2o
From: Wei Yongjun
Using list_move() instead of list_del() + list_add().
Signed-off-by: Wei Yongjun
---
drivers/dma/mmp_pdma.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma/mmp_pdma.c
index ff8d7827..558994f 100644
--- a/drivers/dma
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Using list_move() instead of list_del() + list_add().
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/dma/mmp_pdma.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing destroy_workqueue() before return from
i2o_driver_register() in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/message/i2o/driver.c | 16 ++--
1 file changed, 10 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/power
From: Wei Yongjun yongjun_...@trendmicro.com.cn
usb_free_urb(NULL) is safe and this check is not required.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/rtl8188eu/os_dep/recv_linux.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
irq allocated with devm_request_irq should not be freed using
free_irq, because doing so causes a dangling pointer, and a
subsequent double free.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/misc/cb710/core.c | 2 +-
1 file
From: Wei Yongjun
This patch fix the error handle of function cci_pmu_probe():
- using IS_ERR() instead of NULL test for the return value of
devm_ioremap_resource() since it nerver return NULL.
- remove kfree() for devm_kzalloc allocated memory
- remove dev_warn() since devm_ioremap_resource
From: Wei Yongjun yongjun_...@trendmicro.com.cn
This patch fix the error handle of function cci_pmu_probe():
- using IS_ERR() instead of NULL test for the return value of
devm_ioremap_resource() since it nerver return NULL.
- remove kfree() for devm_kzalloc allocated memory
- remove dev_warn
On 09/12/2013 04:03 AM, KY Srinivasan wrote:
>
>> -Original Message-
>> From: Wei Yongjun [mailto:weiyj...@gmail.com]
>> Sent: Wednesday, September 11, 2013 4:20 AM
>> To: KY Srinivasan; Haiyang Zhang
>> Cc: yongjun_...@trendmicro.com.cn; de...@linu
From: Wei Yongjun
The dereference to 'pdata' should be moved below the NULL test.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/msm/msm_gpu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
index e1e1ec9
From: Wei Yongjun
Fix to return a negative error code in the irq alloc error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/mfd/ucb1x00-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mfd/ucb1x00-core.c b/drivers/mfd
From: Wei Yongjun
Fix to return -EINVAL in the version check error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/hv/connection.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/hv/connection.c b/drivers/hv
From: Wei Yongjun
Fix to return -ENOMEM in the refill memory alloc error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/omapdrm
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the refill memory alloc error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 1 +
1 file changed, 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code in the irq alloc error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mfd/ucb1x00-core.c | 1 +
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -EINVAL in the version check error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/hv/connection.c | 4 +++-
1 file changed, 3 insertions(+), 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The dereference to 'pdata' should be moved below the NULL test.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/msm/msm_gpu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/msm
On 09/12/2013 04:03 AM, KY Srinivasan wrote:
-Original Message-
From: Wei Yongjun [mailto:weiyj...@gmail.com]
Sent: Wednesday, September 11, 2013 4:20 AM
To: KY Srinivasan; Haiyang Zhang
Cc: yongjun_...@trendmicro.com.cn; de...@linuxdriverproject.org; linux-
ker...@vger.kernel.org
From: Wei Yongjun
In case of error, the function drm_prime_pages_to_sg() 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/gpu/drm/msm/msm_gem.c | 4 ++--
1 file changed, 2 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function drm_prime_pages_to_sg() 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/gpu/drm
From: Wei Yongjun
free_irq() expects the same device identity that was passed to
corresponding request_irq(), otherwise the IRQ is not freed.
Signed-off-by: Wei Yongjun
---
drivers/misc/vmw_vmci/vmci_guest.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/misc
From: Wei Yongjun yongjun_...@trendmicro.com.cn
free_irq() expects the same device identity that was passed to
corresponding request_irq(), otherwise the IRQ is not freed.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/misc/vmw_vmci/vmci_guest.c | 2 +-
1 file changed, 1
From: Wei Yongjun
Fix to return -ENOMEM in the fence manager init error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the fence manager init error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 4 +++-
1 file changed, 3
From: Wei Yongjun
free_irq() expects the same device identity that was passed to
corresponding request_irq(), otherwise the IRQ is not freed.
Signed-off-by: Wei Yongjun
---
drivers/vme/bridges/vme_ca91cx42.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/vme
From: Wei Yongjun yongjun_...@trendmicro.com.cn
free_irq() expects the same device identity that was passed to
corresponding request_irq(), otherwise the IRQ is not freed.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/vme/bridges/vme_ca91cx42.c | 6 +-
1 file changed
From: Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(). Also remove the dev_err call to avoid redundant
error message and check for return value
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(). Also remove the dev_err call to avoid redundant
error message and check
From: Wei Yongjun
Fix to return -EINVAL in the irq parse and map error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/dma/cppi41.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/dma/cppi41.c b/drivers/dma
From: Wei Yongjun
Fix to return -EINVAL in the parts parse error handling case instead
of 0(may overwrite to 0 by parse_subpart()), as done elsewhere in this
function.
Signed-off-by: Wei Yongjun
---
block/cmdline-parser.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/cmdline
From: Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(). Also remove the dev_err call to avoid redundant
error message.
Signed-off-by: Wei Yongjun
---
drivers
From: Wei Yongjun
Fix to return -ENOMEM in the kset create and add error handling
case instead of 0, as done elsewhere in this function.
Introduced by commit b59d0bae6ca30c496f298881616258f9cde0d9c6.
(f2fs: add sysfs support for controlling the gc_thread)
Signed-off-by: Wei Yongjun
---
fs
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -EINVAL in the parts parse error handling case instead
of 0(may overwrite to 0 by parse_subpart()), as done elsewhere in this
function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
block/cmdline-parser.c | 1 +
1 file
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -EINVAL in the irq parse and map error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/dma/cppi41.c | 4 +++-
1 file changed, 3 insertions(+), 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in the kset create and add error handling
case instead of 0, as done elsewhere in this function.
Introduced by commit b59d0bae6ca30c496f298881616258f9cde0d9c6.
(f2fs: add sysfs support for controlling the gc_thread)
Signed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(). Also remove the dev_err call to avoid redundant
error message.
Signed-off
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
mm/page_alloc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index efb2ffa..4751901 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -60,7 +60,6 @@
#include
#include
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
sound/soc/pxa/mmp-pcm.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/pxa/mmp-pcm.c b/sound/soc/pxa/mmp-pcm.c
index 9a97843..8235e23 100644
--- a/sound/soc/pxa/mmp-pcm.c
+++ b/sound/soc/pxa/mmp-pcm.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/pxa/mmp-pcm.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/sound/soc/pxa/mmp-pcm.c b/sound/soc/pxa/mmp-pcm.c
index 9a97843..8235e23 100644
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
mm/page_alloc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index efb2ffa..4751901 100644
--- a/mm/page_alloc.c
+++ b
From: Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(), and also remove the dev_err call to avoid redundant
error message.
Signed-off-by: Wei Yongjun
---
sound
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function devm_ioremap_resource() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR(), and also remove the dev_err call to avoid redundant
error message.
Signed-off
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
drivers/staging/xillybus/xillybus_core.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/xillybus/xillybus_core.c
b/drivers/staging/xillybus/xillybus_core.c
index dd0a71c..4b216d4 100644
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
drivers/staging/gdm724x/gdm_lte.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/gdm724x/gdm_lte.c
b/drivers/staging/gdm724x/gdm_lte.c
index 68ebc7d..7165d3a 100644
--- a/drivers/staging/gdm724x
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
drivers/staging/comedi/drivers/dt9812.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/dt9812.c
b/drivers/staging/comedi/drivers/dt9812.c
index ffb7572..b865367 100644
--- a/drivers
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
arch/x86/platform/ce4100/ce4100.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/platform/ce4100/ce4100.c
b/arch/x86/platform/ce4100/ce4100.c
index 8244f5e..d62e74d 100644
--- a/arch/x86/platform/ce4100
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/x86/platform/ce4100/ce4100.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/platform/ce4100/ce4100.c
b/arch/x86/platform/ce4100/ce4100.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/comedi/drivers/dt9812.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/dt9812.c
b/drivers/staging/comedi
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/gdm724x/gdm_lte.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/gdm724x/gdm_lte.c
b/drivers/staging/gdm724x/gdm_lte.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/xillybus/xillybus_core.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/xillybus/xillybus_core.c
b/drivers/staging
From: Wei Yongjun
Fix the error handling in function cirrus_device_init() to avoid resources
leak in the error handling case.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/cirrus/cirrus_main.c | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix the error handling in function cirrus_device_init() to avoid resources
leak in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/cirrus/cirrus_main.c | 19 ---
1 file changed
From: Wei Yongjun
A spin lock is taken here so we should use GFP_ATOMIC.
Signed-off-by: Wei Yongjun
---
drivers/staging/gdm724x/gdm_mux.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/gdm724x/gdm_mux.c
b/drivers/staging/gdm724x/gdm_mux.c
index f570bc0
From: Wei Yongjun yongjun_...@trendmicro.com.cn
A spin lock is taken here so we should use GFP_ATOMIC.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/gdm724x/gdm_mux.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/gdm724x
From: Wei Yongjun
Convert to use devm_* APIs to avoid resources leak on error handling case.
Signed-off-by: Wei Yongjun
---
drivers/mtd/nand/orion_nand.c | 29 +
1 file changed, 9 insertions(+), 20 deletions(-)
diff --git a/drivers/mtd/nand/orion_nand.c b/drivers
From: Wei Yongjun
Fix to return -ENODEV when no proper base address found error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/dma/pch_dma.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/dma/pch_dma.c b/drivers/dma
From: Wei Yongjun
In case of error, the function clk_register_composite() and
clk_register_factors() returns ERR_PTR() not NULL. The NULL test
in the return value check should be replaced with IS_ERR().
Signed-off-by: Wei Yongjun
---
drivers/clk/sunxi/clk-sunxi.c | 4 ++--
1 file changed, 2
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
---
sound/soc/fsl/imx-sgtl5000.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/sound/soc/fsl/imx-sgtl5000.c
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
---
sound/soc/fsl/imx-sgtl5000.c | 4 +++-
1 file changed, 3 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function clk_register_composite() and
clk_register_factors() returns ERR_PTR() not NULL. The NULL test
in the return value check should be replaced with IS_ERR().
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENODEV when no proper base address found error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/dma/pch_dma.c | 1 +
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Convert to use devm_* APIs to avoid resources leak on error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mtd/nand/orion_nand.c | 29 +
1 file changed, 9 insertions(+), 20 deletions
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
Acked-by: Nishanth Menon
---
cc to linux-pm with Nishanth Menon's ACK
---
drivers/power/avs/smartreflex.c | 4 +---
1 file changed, 1 insertion
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
Acked-by: Nishanth Menon n...@ti.com
---
cc to linux-pm with Nishanth Menon's ACK
---
drivers/power/avs
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
701 - 800 of 1371 matches
Mail list logo