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
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
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
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
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
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
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
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
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
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
12 matches
Mail list logo