On Thu, Feb 21, 2013 at 3:49 PM, Ville Syrj?l? <ville.syrjala at linux.intel.com> wrote: >> + if (fb->depth > 24 && !HAS_PCH_SPLIT(dev)) { >> + DRM_DEBUG_KMS("high depth not supported on gmch platforms\n"); >> + return -EINVAL; >> + } > > AFAIK Gen4 does support 2:10:10:10 formats. This would fail there, no? I've noticed that, but failed to properly fix that up in the next patch where I add the correct check, but forgot to kill this bogus one here. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch