> -----Original Message----- > From: Wei Yongjun [mailto:[email protected]] > Sent: Tuesday, December 4, 2018 2:26 PM > To: Wolfgang Grandegger <[email protected]>; Marc Kleine-Budde > <[email protected]>; Aisheng DONG <[email protected]> > Cc: Wei Yongjun <[email protected]>; [email protected]; > [email protected]; [email protected] > Subject: [PATCH net-next] can: flexcan: flexcan_chip_start(): fix the error > return > code in flexcan_setup_stop_mode() > > The error return code PTR_ERR(gpr_np) is always 0 since gpr_np is equal to > NULL in this error handling case. Fix it by return -ENOENT. > > Fixes: de3578c198c6 ("can: flexcan: add self wakeup support") > Signed-off-by: Wei Yongjun <[email protected]> > --- > drivers/net/can/flexcan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c index > 0f36eaf..f412d84 100644 > --- a/drivers/net/can/flexcan.c > +++ b/drivers/net/can/flexcan.c > @@ -1432,7 +1432,7 @@ static int flexcan_setup_stop_mode(struct > platform_device *pdev) > gpr_np = of_find_node_by_phandle(phandle); > if (!gpr_np) { > dev_dbg(&pdev->dev, "could not find gpr node by phandle\n"); > - return PTR_ERR(gpr_np); > + return -ENOENT;
Good catch. Reviewed-by: Dong Aisheng <[email protected]> Regards Dong Aisheng > } > > priv = netdev_priv(dev); > >
