Re: [PATCH 4/5] drm/fb: small cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Mar 7, 2015 at 5:23 AM, Rob Clark <robdclark@xxxxxxxxx> wrote:
> Flip conditional to reduce indentation level of rest of fxn, and use
> min/max to make the code clearer.
>
> Signed-off-by: Rob Clark <robdclark@xxxxxxxxx>
> ---
>  drivers/gpu/drm/drm_fb_helper.c | 28 +++++++++++++++-------------
>  1 file changed, 15 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
> index 1e6a0c7..5ec3849 100644
> --- a/drivers/gpu/drm/drm_fb_helper.c
> +++ b/drivers/gpu/drm/drm_fb_helper.c
> @@ -1035,22 +1035,24 @@ static int drm_fb_helper_single_fb_probe(struct drm_fb_helper *fb_helper,
>         for (i = 0; i < fb_helper->crtc_count; i++) {
>                 struct drm_display_mode *desired_mode;
>                 int x, y;
> +
>                 desired_mode = fb_helper->crtc_info[i].desired_mode;
> +
> +               if (!desired_mode)
> +                       continue;
> +
> +               crtc_count++;
> +
>                 x = fb_helper->crtc_info[i].x;
>                 y = fb_helper->crtc_info[i].y;
> -               if (desired_mode) {
> -                       if (gamma_size == 0)
> -                               gamma_size = fb_helper->crtc_info[i].mode_set.crtc->gamma_size;
> -                       if (desired_mode->hdisplay + x < sizes.fb_width)
> -                               sizes.fb_width = desired_mode->hdisplay + x;
> -                       if (desired_mode->vdisplay + y < sizes.fb_height)
> -                               sizes.fb_height = desired_mode->vdisplay + y;
> -                       if (desired_mode->hdisplay + x > sizes.surface_width)
> -                               sizes.surface_width = desired_mode->hdisplay + x;
> -                       if (desired_mode->vdisplay + y > sizes.surface_height)
> -                               sizes.surface_height = desired_mode->vdisplay + y;
> -                       crtc_count++;
> -               }
> +
> +               if (gamma_size == 0)
> +                       gamma_size = fb_helper->crtc_info[i].mode_set.crtc->gamma_size;
> +
> +               sizes.surface_width  = max_t(u32, desired_mode->hdisplay + x, sizes.surface_width);
> +               sizes.surface_height = max_t(u32, desired_mode->vdisplay + y, sizes.surface_width);

This should be:

 sizes.surface_height = max_t(u32, desired_mode->vdisplay + y,
sizes.surface_height);

Other than that, this one is:

Reviewed-by: Daniel Kurtz <djkurtz@xxxxxxxxxxxx>


> +               sizes.fb_width  = min_t(u32, desired_mode->hdisplay + x, sizes.fb_width);
> +               sizes.fb_height = min_t(u32, desired_mode->vdisplay + y, sizes.fb_height);
>         }
>
>         if (crtc_count == 0 || sizes.fb_width == -1 || sizes.fb_height == -1) {
> --
> 2.1.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux