On Thu, 2018-06-28 at 08:48 +0200, Daniel Vetter wrote:
> On Wed, Jun 27, 2018 at 11:18:54PM -0700, Dhinakaran Pandiyan wrote:
> >
> > There is already a check to allow only RGB8888 formats with CCS
> > modifiers.
> >
> > Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@xxxxxxxxx>
> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
>
Thanks for the reviews.
> Aside: intel_framebuffer_init is horribly long function, and it would
> probably make a lot of sense to extract all the framebuffer related
> stuff
> into a new intel_framebuffer.c file. intel_display.c is a bit ... big
> :-)
Yup, it's become a pain to add new stuff. I'll add this to my todo
list.
> -Daniel
>
> >
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 5 -----
> > 1 file changed, 5 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c
> > b/drivers/gpu/drm/i915/intel_display.c
> > index eaa0663963a5..c5ec13e0f484 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -14521,11 +14521,6 @@ static int intel_framebuffer_init(struct
> > intel_framebuffer *intel_fb,
> > }
> > break;
> > case DRM_FORMAT_NV12:
> > - if (mode_cmd->modifier[0] ==
> > I915_FORMAT_MOD_Y_TILED_CCS ||
> > - mode_cmd->modifier[0] ==
> > I915_FORMAT_MOD_Yf_TILED_CCS) {
> > - DRM_DEBUG_KMS("RC not to be enabled with
> > NV12\n");
> > - goto err;
> > - }
> > if (INTEL_GEN(dev_priv) < 9 ||
> > IS_SKYLAKE(dev_priv) ||
> > IS_BROXTON(dev_priv)) {
> > DRM_DEBUG_KMS("unsupported pixel format:
> > %s\n",
> > --
> > 2.14.1
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
