[PATCH] drm: Add missing break in the command line mode parsing code

Rodrigo Vivi rodrigo.vivi at gmail.com
Fri Oct 19 09:55:48 PDT 2012


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/dri-devel/attachments/20121019/9ecf5da0/attachment.html>


More information about the dri-devel mailing list