Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1557488 > unrolled thread
| Started by | Wei Yongjun <weiyj.lk@gmail.com> |
|---|---|
| First post | 2017-01-12 16:00 +0100 |
| Last post | 2017-01-12 22:40 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH -next] soc: ti: wkup_m3_ipc: Fix error return code in wkup_m3_ipc_probe() Wei Yongjun <weiyj.lk@gmail.com> - 2017-01-12 16:00 +0100
Re: [PATCH -next] soc: ti: wkup_m3_ipc: Fix error return code in wkup_m3_ipc_probe() Tony Lindgren <tony@atomide.com> - 2017-01-12 22:40 +0100
| From | Wei Yongjun <weiyj.lk@gmail.com> |
|---|---|
| Date | 2017-01-12 16:00 +0100 |
| Subject | [PATCH -next] soc: ti: wkup_m3_ipc: Fix error return code in wkup_m3_ipc_probe() |
| Message-ID | <sYP17-5bP-7@gated-at.bofh.it> |
From: Wei Yongjun <weiyongjun1@huawei.com>
Fix to return a negative error code from the kthread_run() error
handling case instead of 0, as done elsewhere in this function.
Fixes: cdd5de500b2c ("soc: ti: Add wkup_m3_ipc driver")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
drivers/soc/ti/wkup_m3_ipc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/soc/ti/wkup_m3_ipc.c b/drivers/soc/ti/wkup_m3_ipc.c
index 8bfa44b..369aef5 100644
--- a/drivers/soc/ti/wkup_m3_ipc.c
+++ b/drivers/soc/ti/wkup_m3_ipc.c
@@ -457,6 +457,7 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev)
if (IS_ERR(task)) {
dev_err(dev, "can't create rproc_boot thread\n");
+ ret = PTR_ERR(task);
goto err_put_rproc;
}
[toc] | [next] | [standalone]
| From | Tony Lindgren <tony@atomide.com> |
|---|---|
| Date | 2017-01-12 22:40 +0100 |
| Subject | Re: [PATCH -next] soc: ti: wkup_m3_ipc: Fix error return code in wkup_m3_ipc_probe() |
| Message-ID | <sYVgf-Ak-29@gated-at.bofh.it> |
| In reply to | #1557488 |
* Wei Yongjun <weiyj.lk@gmail.com> [170112 06:55]:
> From: Wei Yongjun <weiyongjun1@huawei.com>
>
> Fix to return a negative error code from the kthread_run() error
> handling case instead of 0, as done elsewhere in this function.
>
> Fixes: cdd5de500b2c ("soc: ti: Add wkup_m3_ipc driver")
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---
> drivers/soc/ti/wkup_m3_ipc.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/soc/ti/wkup_m3_ipc.c b/drivers/soc/ti/wkup_m3_ipc.c
> index 8bfa44b..369aef5 100644
> --- a/drivers/soc/ti/wkup_m3_ipc.c
> +++ b/drivers/soc/ti/wkup_m3_ipc.c
> @@ -457,6 +457,7 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev)
>
> if (IS_ERR(task)) {
> dev_err(dev, "can't create rproc_boot thread\n");
> + ret = PTR_ERR(task);
> goto err_put_rproc;
> }
>
>
Applying into omap-for-v4.10/fixes thanks.
Tony
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web