On Mon, 3 Jun 2019 at 01:40, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote: > > Signed-off-by: Ilia Mirkin <imirkin@xxxxxxxxxxxx> > --- > tests/modetest/modetest.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c > index 9c85c07b..a1c81f6c 100644 > --- a/tests/modetest/modetest.c > +++ b/tests/modetest/modetest.c > @@ -1335,8 +1335,8 @@ static void atomic_set_mode(struct device *dev, struct pipe_arg *pipes, unsigned > if (pipe->mode == NULL) > continue; > > - printf("setting mode %s-%dHz@%s on connectors ", > - pipe->mode_str, pipe->mode->vrefresh, pipe->format_str); > + printf("setting mode %s-%dHz on connectors ", > + pipe->mode_str, pipe->mode->vrefresh); AFAICT we can drop the format on modeset all together. I cannot see anything that would require it - regardless if the modeset is atomic or not. Plus we can remove the --help string and argument parsing code. Can I interest you in doing that? -Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel