On Thu, Jul 14, 2016 at 12:10:46PM +0200, Christophe JAILLET wrote:
> In function 'xilinx_pcie_init_irq_domain', the pattern used to check and
> return error is:
>
>if (!var) {
> dev_err(...);
> return PTR_ERR(var);
>}
>
> So the returned value in case of error is always 0, wh
On Thu, 2016-07-14 at 12:10:46 +0200, Christophe JAILLET wrote:
> In function 'xilinx_pcie_init_irq_domain', the pattern used to check and
> return error is:
>
>if (!var) {
> dev_err(...);
> return PTR_ERR(var);
>}
>
> So the returned value in case of error is always 0, which
In function 'xilinx_pcie_init_irq_domain', the pattern used to check and
return error is:
if (!var) {
dev_err(...);
return PTR_ERR(var);
}
So the returned value in case of error is always 0, which means 'success'.
Change it to return -ENODEV instead.
Signed-off-by: Christophe J
3 matches
Mail list logo