| Message ID | 20201116141058.2365043-1-chengzhihao1@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | e9acf0298c664f825e6f1158f2a97341bf9e03ca |
| Headers | show |
| Series | i2c: qup: Fix error return code in qup_i2c_bam_schedule_desc() | expand |
On Mon 16 Nov 08:10 CST 2020, Zhihao Cheng wrote: > Fix to return the error code from qup_i2c_change_state() > instaed of 0 in qup_i2c_bam_schedule_desc(). > > Fixes: fbf9921f8b35d9b2 ("i2c: qup: Fix error handling") > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com> > --- > drivers/i2c/busses/i2c-qup.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c > index fbc04b60cfd1..5a47915869ae 100644 > --- a/drivers/i2c/busses/i2c-qup.c > +++ b/drivers/i2c/busses/i2c-qup.c > @@ -801,7 +801,8 @@ static int qup_i2c_bam_schedule_desc(struct qup_i2c_dev *qup) > if (ret || qup->bus_err || qup->qup_err) { > reinit_completion(&qup->xfer); > > - if (qup_i2c_change_state(qup, QUP_RUN_STATE)) { > + ret = qup_i2c_change_state(qup, QUP_RUN_STATE); In the case that we entered this block because ret was -ETIMEDOUT then this will overwrite this and the function will return -EIO. But in the other paths out of this block ret is being overwritten anyways, so I think it's fine. Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Regards, Bjorn > + if (ret) { > dev_err(qup->dev, "change to run state timed out"); > goto desc_err; > } > -- > 2.25.4 >
On Mon, Nov 16, 2020 at 10:10:58PM +0800, Zhihao Cheng wrote: > Fix to return the error code from qup_i2c_change_state() > instaed of 0 in qup_i2c_bam_schedule_desc(). > > Fixes: fbf9921f8b35d9b2 ("i2c: qup: Fix error handling") > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com> Applied to for-current, thanks!
Hello: This patch was applied to qcom/linux.git (refs/heads/for-next): On Mon, 16 Nov 2020 22:10:58 +0800 you wrote: > Fix to return the error code from qup_i2c_change_state() > instaed of 0 in qup_i2c_bam_schedule_desc(). > > Fixes: fbf9921f8b35d9b2 ("i2c: qup: Fix error handling") > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com> > > [...] Here is the summary with links: - i2c: qup: Fix error return code in qup_i2c_bam_schedule_desc() https://git.kernel.org/qcom/c/e9acf0298c66 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c index fbc04b60cfd1..5a47915869ae 100644 --- a/drivers/i2c/busses/i2c-qup.c +++ b/drivers/i2c/busses/i2c-qup.c @@ -801,7 +801,8 @@ static int qup_i2c_bam_schedule_desc(struct qup_i2c_dev *qup) if (ret || qup->bus_err || qup->qup_err) { reinit_completion(&qup->xfer); - if (qup_i2c_change_state(qup, QUP_RUN_STATE)) { + ret = qup_i2c_change_state(qup, QUP_RUN_STATE); + if (ret) { dev_err(qup->dev, "change to run state timed out"); goto desc_err; }
Fix to return the error code from qup_i2c_change_state() instaed of 0 in qup_i2c_bam_schedule_desc(). Fixes: fbf9921f8b35d9b2 ("i2c: qup: Fix error handling") Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com> --- drivers/i2c/busses/i2c-qup.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)