Re: [PATCH 04/11] drm/i915: don't keep reassigning FBC_UNSUPPORTED

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

 



On Mon, Dec 8, 2014 at 8:09 AM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote:
> From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
>
> This may save a few picoseconds on !HAS_FBC platforms. And it also
> satisfies my OCD.
>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> index 4647d57..f3d5764 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -516,10 +516,8 @@ void intel_fbc_update(struct drm_device *dev)
>         const struct drm_display_mode *adjusted_mode;
>         unsigned int max_width, max_height;
>
> -       if (!HAS_FBC(dev)) {
> -               set_no_fbc_reason(dev_priv, FBC_UNSUPPORTED);

I really don't like this fbc_reasons... I believe a drm_debug here is enough.
but I can try to kill that later.

For this patch: Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

> +       if (!HAS_FBC(dev))
>                 return;
> -       }
>
>         if (i915.enable_fbc < 0) {
>                 if (set_no_fbc_reason(dev_priv, FBC_CHIP_DEFAULT))
> @@ -684,6 +682,7 @@ void intel_fbc_init(struct drm_i915_private *dev_priv)
>  {
>         if (!HAS_FBC(dev_priv)) {
>                 dev_priv->fbc.enabled = false;
> +               dev_priv->fbc.no_fbc_reason = FBC_UNSUPPORTED;
>                 return;
>         }
>
> --
> 2.1.3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux