On Fri, Jan 15, 2016 at 03:22:08PM -0800, Matt Roper wrote:
> On Fri, Jan 15, 2016 at 08:46:53PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote:
> > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> >
> > On SKL+ plane scaling is mutually exclusive with color keying. The code
> > check for this, but during some refactoring the code got changed to
> > also reject primary plane windowing when color keying is used. There is
> > no such restriction in the hardware, so restore the original logic.
> >
> > Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> > Fixes: 061e4b8d650a ("drm/i915: clean up atomic plane check functions, v2.")
> > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> s/primayr/primary in your headline, but otherwise
>
> Reviewed-by: Matt Roper <matthew.d.roper@xxxxxxxxx>
>
> Still waiting for CI to run the BAT on this...
CI didn't scream. so pushed to dinq with the typo fixed.
Thanks for the reviews.
>
>
> Matt
>
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 11 ++++++-----
> > 1 file changed, 6 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index a726c5e91220..7bc56e217ecc 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -14019,11 +14019,12 @@ intel_check_primary_plane(struct drm_plane *plane,
> > int max_scale = DRM_PLANE_HELPER_NO_SCALING;
> > bool can_position = false;
> >
> > - /* use scaler when colorkey is not required */
> > - if (INTEL_INFO(plane->dev)->gen >= 9 &&
> > - state->ckey.flags == I915_SET_COLORKEY_NONE) {
> > - min_scale = 1;
> > - max_scale = skl_max_scale(to_intel_crtc(crtc), crtc_state);
> > + if (INTEL_INFO(plane->dev)->gen >= 9) {
> > + /* use scaler when colorkey is not required */
> > + if (state->ckey.flags == I915_SET_COLORKEY_NONE) {
> > + min_scale = 1;
> > + max_scale = skl_max_scale(to_intel_crtc(crtc), crtc_state);
> > + }
> > can_position = true;
> > }
> >
> > --
> > 2.4.10
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> --
> Matt Roper
> Graphics Software Engineer
> IoTG Platform Enabling & Development
> Intel Corporation
> (916) 356-2795
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
