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
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
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/input/serio/olpc_apsp.c | 4 +---
1 file changed, 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing iounmap() before return from msm_request_port()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/tty/serial/msm_serial.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff
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 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
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/spi/spi-tegra114.c | 1 -
1 file changed, 1 deletion
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/scsi/ufs/ufshcd-pltfrm.c | 1 -
1 file changed, 1
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/net/ethernet/ti/davinci_emac.c | 1 -
1 file changed
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
On 06/26/2013 05:46 PM, Thierry Reding wrote:
On Wed, Jun 26, 2013 at 06:55:18AM +0800, Wei Yongjun wrote:
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
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().
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/pwm/pwm
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/video/mxsfb.c | 1 -
1 file changed, 1 deletion
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/input/keyboard/nspire-keypad.c | 4 +---
1 file
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 yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/usb/phy/phy-tegra-usb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/phy/phy-tegra-usb.c b/drivers/usb/phy/phy-tegra-usb.c
index
On 06/18/2013 06:03 PM, Philipp Zabel wrote:
Am Dienstag, den 18.06.2013, 13:00 +0800 schrieb Wei Yongjun:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing unlock before return from function coda_stop_streaming()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_
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/infiniband/hw/ehca/ehca_main.c | 1 +
1 file changed, 1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing unlock before return from function coda_stop_streaming()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/media/platform/coda.c | 1 +
1 file changed, 1 insertion(+)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The dereference to 'up' should be moved below the NULL test.
Introduced by commit ddd85e225c8885b5e4419b0499ab27100e7c366a
(serial: omap: prevent runtime PM for no_console_suspend)
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Just fixed a typo in emac_probe().
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/net/ethernet/allwinner/sun4i-emac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/allwinner/sun4i
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
Hi
On 05/29/2013 02:58 PM, Guennadi Liakhovetski wrote:
Hi
On Tue, 28 May 2013, Wei Yongjun wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code instead of 0 when we cannot get
IRQ source by platform_get_irq(), as done elsewhere in this function
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
Fix to return a negative error code instead of 0 when we cannot get
IRQ source by platform_get_irq(), as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/sh_mobile_sdhi.c | 4 +++-
1
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, function of_clk_get_by_name() returns
ERR_PTR() 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/pci/host/pci
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 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
Fix to return -ENODEV in the dma channel request error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mtd/nand/fsmc_nand.c | 1 +
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENODEV in the dma channel request error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: set ret for error cases only
---
drivers/mtd/nand
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code in the init error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/omap_hsmmc.c | 3 ++-
1 file changed, 2 insertions
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.
Introduced by commit 9dddb4df90d136429b6d6ddefceb49a9b93f6cd1
(pinctrl: single: support generic pinconf)
Signed-off-by: Wei Yongjun
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/net/ethernet/toshiba/spider_net.c | 3 ++-
1 file changed, 2
On 05/07/2013 08:11 PM, Haojian Zhuang wrote:
On 7 May 2013 20:06, Wei Yongjun weiyj...@gmail.com wrote:
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.
Introduced by commit
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return a negative error code in the MSI bitmap alloc error
handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/powerpc/sysdev/fsl_msi.c | 5 ++---
1 file changed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/arm/xen/enlighten.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
index 9a0a917..d30042e 100644
From: Wei Yongjun yongjun_...@trendmicro.com.cn
We have registered platform driver when module init, and
need unregister it when module exit.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
no compile test.
---
drivers/tty/serial/sunsu.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
We have registered platform driver when module init, and
need unregister it when module exit.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
no compile test.
---
drivers/tty/serial/samsung.c | 1 +
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
We have registered platform driver when module init, and
need unregister it when module exit.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
no compile test.
---
drivers/tty/ehv_bytechan.c | 1 +
1 file changed, 1 insertion(+)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing platform_device_put() before return from
omap2430_probe() in the error handling case.
Introduced by commit ca784be36cc725bca9b526eba342de7550329731
(usb: start using the control module driver)
Signed-off-by: Wei Yongjun yongjun_
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing uart_unregister_driver() and uninit before return
from mpc52xx_uart_init() in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/tty/serial/mpc52xx_uart.c | 11 ---
1 file changed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOENT in the host1x_bo lookup error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: fix the patch subject
---
drivers/gpu/host1x/drm/gr2d.c | 8
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/gpu/host1x/drm/gr2d.c | 8 ++--
1 file changed, 6
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -EINVAL in the host1x_bo lookup error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: fix the patch subject
---
drivers/gpu/host1x/drm/gr2d.c | 8
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENOMEM in alloc workqueue error case instead
of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/dw_mmc.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix to return -ENODEV in the request dma error case instead
of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/mxs-mmc.c | 1 +
1 file changed, 1 insertion(+)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-omap2-mcspi.c | 3 ++-
1 file changed, 2 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/media/platform/s5p-mfc/s5p_mfc.c | 3 ++-
1 file changed, 2
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/net/ethernet/cadence/at91_ether.c | 3 ++-
1 file changed
From: Wei Yongjun yongjun_...@trendmicro.com.cn
symbol '' was not declared. It should be static.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-tegra20-slink.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/spi/spi-tegra20-slink.c b
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Neither tegra20_spi_cdata nor tegra30_spi_cdata are used outside this
file so they can, and should, be static.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-tegra20-slink.c | 4 ++--
1 file changed, 2 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-mxs.c | 1 +
1 file changed, 1 insertion(+)
diff
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/ssbi/ssbi.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
emac_poll_controller() was not declared. It should be static.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/net/ethernet/ti/davinci_emac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net
From: Wei Yongjun yongjun_...@trendmicro.com.cn
fec_poll_controller() was not declared. It should be static.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/net/ethernet/freescale/fec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Fix the '' vs '' typo.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/staging/nvec/nvec.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
From: Wei Yongjun yongjun_...@trendmicro.com.cn
sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/pinctrl/pinctrl-exynos5440.c | 2 +-
1 file changed, 1 insertion
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The variable bits_per_word is initialized but never used
otherwise, so remove the unused variable.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/spi/spi-tegra20-slink.c | 3 ---
1 file changed, 3 deletions(-)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use the module_i2c_driver() macro to make the code smaller
and a bit simpler.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/codecs/tas5086.c | 12 +---
1 file changed, 1 insertion(+), 11 deletions(-)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use resource_size function on resource object
instead of explicit computation.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/wmt-sdmmc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/microblaze/pci/pci-common.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/microblaze/pci/pci-common.c b/arch/microblaze/pci/pci-common.c
index
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Using for_each_set_bit() to simplify the code.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/pinctrl/pinctrl-sirf.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/pinctrl/pinctrl
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use for_each_compatible_node() macro instead of open coding it.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/powerpc/include/asm/parport.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/powerpc
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The variable 'mmc' is initialized but never used
otherwise, so remove the unused variable.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/wmt-sdmmc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/mmc
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use the module_i2c_driver() macro to make the code smaller
and a bit simpler.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/usb/misc/usb3503.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
'data' is malloced in snd_soc_bytes_put() and should be freed
before leaving from the error handling cases, otherwise it will cause
memory leak.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
sound/soc/soc-core.c | 6 --
1 file
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing unlock before return from function rcar_thermal_update_temp()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/thermal/rcar_thermal.c | 9 +
1 file changed, 5 insertions(+), 4
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Add the missing unlock before return from function rcar_thermal_update_temp()
in the error handling case.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/thermal/rcar_thermal.c | 1 +
1 file changed, 1 insertion(+)
diff --git
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use for_each_compatible_node() macro instead of open coding it.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
arch/powerpc/platforms/85xx/mpc85xx_mds.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Use for_each_compatible_node() macro instead of open coding it.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/tty/hvc/hvsi.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/tty/hvc/hvsi.c b
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, function of_clk_get() 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/tty/serial
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, function of_clk_get() 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/tty/serial
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/gpu/drm/tegra/host1x.c | 2 ++
1 file changed, 2 insertions
From: Wei Yongjun yongjun_...@trendmicro.com.cn
The dereference to 'of_id' should be moved below the NULL test.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/wmt-sdmmc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/host/wmt
On 11/14/2012 11:54 PM, Grant Likely wrote:
On Sun, 7 Oct 2012 21:29:21 +0800, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
Applied, thanks.
dpatch engine is used to auto generate this patch.
(https://github.com/weiyj
On 11/15/2012 08:08 PM, Grant Likely wrote:
On Thu, 1 Nov 2012 13:27:34 +0800, Wei Yongjun weiyj...@gmail.com wrote:
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function clk_get() returns ERR_PTR()
and never returns NULL. The NULL test in the return value
check
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Test for tegra and hcd in tegra_ehci_remove() look like potential
NULL pointer dereference, but in fact those tests are not needed,
so remove these pointless tests entirely.
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2
From: Wei Yongjun yongjun_...@trendmicro.com.cn
We had assigned the return value to 'ret' but ignored it when
return from isl29018_write_raw(), it's better to return 'ret'
instead of 0.
dpatch engine is used to auto generate this patch.
(https://github.com/weiyj/dpatch)
Signed-off-by: Wei
From: Wei Yongjun yongjun_...@trendmicro.com.cn
In case of error, the function clk_get() 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
From: Wei Yongjun yongjun_...@trendmicro.com.cn
Remove duplicated include.
dpatch engine is used to auto generate this patch.
(https://github.com/weiyj/dpatch)
Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
drivers/mmc/host/sdhci-dove.c | 1 -
1 file changed, 1 deletion(-)
diff
From: Wei Yongjun yongjun_...@trendmicro.com.cn
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 yongjun_...@trendmicro.com.cn
---
drivers/usb/host/ehci-tegra.c | 8 ++--
1
82 matches
Mail list logo