[PATCH] spi: ep93xx: fix error return code in ep93xx_spi_probe()

2013-05-15 Thread Wei Yongjun
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

[PATCH] spi: coldfire-qspi: fix error return code in mcfqspi_probe()

2013-05-15 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn 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

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

2013-04-17 Thread 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 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

[PATCH -next] spi: tegra: slink: make symbol static

2013-04-05 Thread Wei Yongjun
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

[PATCH -next v2] spi: tegra: slink: make local symbols static

2013-04-05 Thread Wei Yongjun
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

[PATCH -next] mxs/spi: fix error return code in mxs_spi_probe()

2013-04-03 Thread 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 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

[PATCH -next] spi: remove unused variable in tegra_slink_read_rx_fifo_to_client_rxbuf()

2013-03-13 Thread Wei Yongjun
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

[PATCH -next] spi: fix return value check in ce4100_spi_probe()

2013-02-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function platform_device_register_full() 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

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

2012-11-15 Thread Wei Yongjun
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

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

2012-10-07 Thread Wei Yongjun
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/spi/spi-omap2-mcspi.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH] spi: sh-hspi: fix return value check in hspi_probe()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() and never returns NULL pointer. The NULL test in the error handling should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch

[PATCH] spi-topcliff-pch: fix to use list_for_each_entry_safe() when delete items

2012-08-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Since we will be removing items off the list using list_del() we need to use a safer version of the list_for_each_entry() macro aptly named list_for_each_entry_safe(). We should use the safe macro if the loop involves deletions of items. Signed-off