Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>><br><br><div class="gmail_quote">On Tue, Oct 9, 2012 at 8:26 AM, Damien Lespiau <span dir="ltr"><<a href="mailto:damien.lespiau@gmail.com" target="_blank">damien.lespiau@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Damien Lespiau <<a href="mailto:damien.lespiau@intel.com">damien.lespiau@intel.com</a>><br>
<br>
As we parse the string given on the command line one char at a time, it<br>
seems that we do want a break at every case.<br>
<br>
Signed-off-by: Damien Lespiau <<a href="mailto:damien.lespiau@intel.com">damien.lespiau@intel.com</a>><br>
---<br>
drivers/gpu/drm/drm_modes.c | 1 +<br>
1 files changed, 1 insertions(+), 0 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c<br>
index 28637c1..50bd5c1 100644<br>
--- a/drivers/gpu/drm/drm_modes.c<br>
+++ b/drivers/gpu/drm/drm_modes.c<br>
@@ -1054,6 +1054,7 @@ bool drm_mode_parse_command_line_for_connector(const char *mode_option,<br>
was_digit = false;<br>
} else<br>
goto done;<br>
+ break;<br>
case '0' ... '9':<br>
was_digit = true;<br>
break;<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.7.5<br>
<br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div><br>