Zhong, Thank you for the patch. zhong jiang <zhongjiang@xxxxxxxxxx> wrote on Sun [2018-Sep-16 21:16:51 +0800]: > of_node_put has taken the null pinter check into account. So it is s/pinter/pointer/ > safe to remove the duplicated check before of_node_put. > > Signed-off-by: zhong jiang <zhongjiang@xxxxxxxxxx> with the above change, Reviewed-by: Benoit Parrot <bparrot@xxxxxx> > --- > drivers/media/platform/ti-vpe/cal.c | 12 ++++-------- > 1 file changed, 4 insertions(+), 8 deletions(-) > > diff --git a/drivers/media/platform/ti-vpe/cal.c b/drivers/media/platform/ti-vpe/cal.c > index cc052b2..5f9d4e0 100644 > --- a/drivers/media/platform/ti-vpe/cal.c > +++ b/drivers/media/platform/ti-vpe/cal.c > @@ -1747,14 +1747,10 @@ static int of_cal_create_instance(struct cal_ctx *ctx, int inst) > } > > cleanup_exit: > - if (remote_ep) > - of_node_put(remote_ep); > - if (sensor_node) > - of_node_put(sensor_node); > - if (ep_node) > - of_node_put(ep_node); > - if (port) > - of_node_put(port); > + of_node_put(remote_ep); > + of_node_put(sensor_node); > + of_node_put(ep_node); > + of_node_put(port); > > return ret; > } > -- > 1.7.12.4 >
