Re: [PATCH] sata_rcar: fix error return code in sata_rcar_probe()

From: Gustavo A. R. Silva
Date: Fri Jun 30 2017 - 15:37:26 EST


Hi Tejun, Sergei,

Quoting Tejun Heo <tj@xxxxxxxxxx>:

On Fri, Jun 30, 2017 at 12:42:38PM +0300, Sergei Shtylyov wrote:
Hello!

On 6/30/2017 8:22 AM, Gustavo A. R. Silva wrote:

> Print error message and propagate the return value of
> platform_get_irq on failure.

You should have probably mentioned that this function no longer returns 0
on error.

Yeah, the patches looks good to me but I'd really appreciate more
context in the changelogs. Gustavo, can you please respin the
patches?


Absolutely.

What do you think about the following changelog:

platform_get_irq() returns an error code, but the sata_rcar driver
ignores it and always returns -EINVAL. This is not correct, and
prevents -EPROBE_DEFER from being propagated properly.

Print error message and propagate the return value of platform_get_irq
on failure. Also, with this change function sata_rcar_probe() no longer
returns 0 on error.


Thank you!
--
Gustavo A. R. Silva