On Wednesday, 2018-03-14 14:48:37 +0900, Seung-Woo Kim wrote: > There is already condition checking input values between 2 and 4096 > so condition checking 0 is always false. Remove the dead condition. > > Signed-off-by: Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx> Indeed, good catch :) Reviewed-by: Eric Engestrom <eric.engestrom@xxxxxxxxxx> ... and pushed, thanks! > --- > tests/exynos/exynos_fimg2d_perf.c | 7 ------- > 1 files changed, 0 insertions(+), 7 deletions(-) > > diff --git a/tests/exynos/exynos_fimg2d_perf.c b/tests/exynos/exynos_fimg2d_perf.c > index a2d5c19..97691a7 100644 > --- a/tests/exynos/exynos_fimg2d_perf.c > +++ b/tests/exynos/exynos_fimg2d_perf.c > @@ -274,13 +274,6 @@ int main(int argc, char **argv) > goto out; > } > > - if (bufw == 0 || bufh == 0) { > - fprintf(stderr, "error: buffer width/height should be non-zero.\n"); > - ret = -1; > - > - goto out; > - } > - > fd = drmOpen("exynos", NULL); > if (fd < 0) { > fprintf(stderr, "error: failed to open drm\n"); > -- > 1.7.4.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel