All of lore.kernel.org
 help / color / mirror / Atom feed
From: Damien Lespiau <damien.lespiau@intel.com>
To: akash.goel@intel.com
Cc: shobhit.kumar@intel.com, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/6] New drm crtc property for varying the Crtc size
Date: Thu, 14 Aug 2014 16:32:07 +0100	[thread overview]
Message-ID: <20140814153207.GG18039@strange.config> (raw)
In-Reply-To: <1408008267-11443-1-git-send-email-akash.goel@intel.com>

Hi,

Do we have some (possiblly CRC based) tests coming for this? it'd be
super good to be able to test the work indepedently of a compositor when
reviewing. At the end of the day, we do really want tests covering that
code.

Thanks,

-- 
Damien

On Thu, Aug 14, 2014 at 02:54:21PM +0530, akash.goel@intel.com wrote:
> From: Akash Goel <akash.goel@intel.com>
> 
> Added a new drm crtc property which provides control
> to vary the Pipe Src or Crtc size. 
> With this, User can flip the frame buffers of resolution, which
> is different from the currently selected mode. For this Driver will
> appropriately enable the Panel fitter internally, for the required
> scaling.
> For this also removed the check of change in frame buffer pitch,
> from the crtc page flip function.  
> Added a return type to panel fitter config & crtc retsore mode
> functions, so that an error could be returned to User space for an 
> invalid configuration. 
> Also added Max downscale ratio checks when enbaling Panel fitter.
> 
> Akash Goel (6):
>   drm/i915: Added a return type for panel fitter config functions
>   drm/i915: Added a return type for the restore crtc mode function
>   drm/i915: Added Max down-scale ratio checks when enabling Panel fitter
>   drm/i915: Added support to allow change in FB pitch across flips
>   Documentation/drm: Describing crtc size property
>   drm/i915: New drm crtc property for varying the Crtc size
> 
>  Documentation/DocBook/drm.tmpl       |  13 +++-
>  drivers/gpu/drm/drm_crtc.c           |   7 ++
>  drivers/gpu/drm/drm_fb_helper.c      |   7 ++
>  drivers/gpu/drm/i915/intel_display.c | 128 +++++++++++++++++++++++++++++++++--
>  drivers/gpu/drm/i915/intel_dp.c      |  45 +++++++++---
>  drivers/gpu/drm/i915/intel_drv.h     |  11 ++-
>  drivers/gpu/drm/i915/intel_hdmi.c    |  28 ++++++--
>  drivers/gpu/drm/i915/intel_lvds.c    |  18 +++--
>  drivers/gpu/drm/i915/intel_panel.c   | 123 ++++++++++++++++++++++++++-------
>  drivers/gpu/drm/i915/intel_sdvo.c    |  57 ++++++++++++++--
>  drivers/gpu/drm/i915/intel_sprite.c  |   4 +-
>  drivers/gpu/drm/i915/intel_tv.c      |  26 ++++++-
>  include/drm/drm_crtc.h               |   7 ++
>  13 files changed, 410 insertions(+), 64 deletions(-)
> 
> -- 
> 1.9.2
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  parent reply	other threads:[~2014-08-14 15:32 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-14  9:24 [PATCH 0/6] New drm crtc property for varying the Crtc size akash.goel
2014-08-14  9:24 ` [PATCH 1/6] drm/i915: Added a return type for panel fitter config functions akash.goel
2014-08-14  9:24 ` [PATCH 2/6] drm/i915: Added a return type for the restore crtc mode function akash.goel
2014-08-14  9:24 ` [PATCH 3/6] drm/i915: Added Max down-scale ratio checks when enabling Panel fitter akash.goel
2014-08-14  9:24 ` [PATCH 4/6] drm/i915: Added support to allow change in FB pitch across flips akash.goel
2014-08-14 14:27   ` Daniel Vetter
2014-08-14  9:24 ` [PATCH 5/6] Documentation/drm: Describing crtc size property akash.goel
2014-08-14  9:24 ` [PATCH 6/6] drm/i915: New drm crtc property for varying the Crtc size akash.goel
2014-08-14 14:42   ` Daniel Vetter
2014-08-14 15:06     ` Ville Syrjälä
2014-08-14 15:26       ` Daniel Vetter
2014-08-14 15:32         ` Ville Syrjälä
2014-08-14 15:45         ` Ville Syrjälä
2014-08-14 16:07           ` Daniel Vetter
2014-08-14 16:45             ` Ville Syrjälä
2014-08-14 17:36               ` Daniel Vetter
2014-08-14 17:58                 ` Ville Syrjälä
2014-08-14 18:33                   ` Daniel Vetter
2014-08-14 18:51                     ` Ville Syrjälä
2014-08-14 15:32 ` Damien Lespiau [this message]
  -- strict thread matches above, loose matches on Subject: below --
2014-08-27  7:20 [PATCH 0/6] " akash.goel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140814153207.GG18039@strange.config \
    --to=damien.lespiau@intel.com \
    --cc=akash.goel@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=shobhit.kumar@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.