Reviewed-by: Rodrigo Vivi <rodrigo.vivi at gmail.com> On Tue, Oct 9, 2012 at 8:26 AM, Damien Lespiau <damien.lespiau at gmail.com>wrote: > From: Damien Lespiau <damien.lespiau at intel.com> > > As we parse the string given on the command line one char at a time, it > seems that we do want a break at every case. > > Signed-off-by: Damien Lespiau <damien.lespiau at intel.com> > --- > drivers/gpu/drm/drm_modes.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c > index 28637c1..50bd5c1 100644 > --- a/drivers/gpu/drm/drm_modes.c > +++ b/drivers/gpu/drm/drm_modes.c > @@ -1054,6 +1054,7 @@ bool drm_mode_parse_command_line_for_connector(const > char *mode_option, > was_digit = false; > } else > goto done; > + break; > case '0' ... '9': > was_digit = true; > break; > -- > 1.7.7.5 > > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel > -- Rodrigo Vivi Blog: http://blog.vivi.eng.br -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20121019/9ecf5da0/attachment.html>