On Mon, 17 Apr 2017, Olimpiu Dejeu wrote: > On Tue, Mar 21, 2017 at 11:07 AM, Daniel Thompson < > daniel.thompson@xxxxxxxxxx> wrote: > > > On 21/03/17 14:58, Olimpiu Dejeu wrote: > > > >> On Fri, Mar 17, 2017 at 10:48 AM, kbuild test robot <lkp@xxxxxxxxx> > >> wrote: > >> > >>> drivers/video/backlight/arcxcnn_bl.c:183:1-3: WARNING: PTR_ERR_OR_ZERO > >>> can be used > >>> > >>> > >>> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR > >>> > >>> Generated by: scripts/coccinelle/api/ptr_ret.cocci > >>> > >>> CC: Olimpiu Dejeu <olimpiu@xxxxxxxxxxxxxx> > >>> Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx> > >>> --- > >>> > >>> arcxcnn_bl.c | 5 +---- > >>> 1 file changed, 1 insertion(+), 4 deletions(-) > >>> > >>> --- a/drivers/video/backlight/arcxcnn_bl.c > >>> +++ b/drivers/video/backlight/arcxcnn_bl.c > >>> @@ -180,10 +180,7 @@ static int arcxcnn_backlight_register(st > >>> > >>> lp->bl = devm_backlight_device_register(lp->dev, name, lp->dev, > >>> lp, > >>> &arcxcnn_bl_ops, props); > >>> - if (IS_ERR(lp->bl)) > >>> - return PTR_ERR(lp->bl); > >>> - > >>> - return 0; > >>> + return PTR_ERR_OR_ZERO(lp->bl); > >>> } > >>> > >>> static void arcxcnn_parse_dt(struct arcxcnn *lp) > >>> > >> > >> Acked-by: Olimpiu Dejeu <olimpiu@xxxxxxxxxxxxxx> > >> > > > > Glad you approve! Could you apply this change and re-post the patch > > series? Thanks. > > > > > > Daniel. > > > > > Applied change, re-posted series on March 21st. No sure what next. Please > advise. Thanks. Yes you did. Daniel, do you see v9? FYI, if this happens again, it's best to wait a couple of weeks then send the patch again like "[RESEND v9] ..." to gain attention. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
