Hi Wei, On Tue, Oct 30, 2012 at 7:19 PM, Wei Yongjun <weiyj.lk@xxxxxxxxx> wrote: > From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> > > The pattern E == C1 && E == C2 is always false. This patch > fix this according to the assumption that && should be ||. > > dpatch engine is used to auto generate this patch. > (https://github.com/weiyj/dpatch) > > Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> > --- > drivers/media/platform/davinci/vpif_display.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied to my tree with following commit message, davinci: vpif_display: fix return type check for v4l2_subdev_call() The v4l2_subdev_call() call returns -ENODEV when subdev is null and -ENOIOCTLCMD wnen no icotl is present. This patch fixes the return type check for v4l2_subdev_call(). The pattern E == C1 && E == C2 is always false. This patch fix this according to the assumption that && should be ||. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Regards, --Prabhakar Lad > diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c > index b716fbd..977ee43 100644 > --- a/drivers/media/platform/davinci/vpif_display.c > +++ b/drivers/media/platform/davinci/vpif_display.c > @@ -1380,7 +1380,7 @@ vpif_enum_dv_timings(struct file *file, void *priv, > int ret; > > ret = v4l2_subdev_call(ch->sd, video, enum_dv_timings, timings); > - if (ret == -ENOIOCTLCMD && ret == -ENODEV) > + if (ret == -ENOIOCTLCMD || ret == -ENODEV) > return -EINVAL; > return ret; > } > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
