On Thu, Sep 23, 2021 at 01:28:21PM +0300, Jani Nikula wrote: > On Thu, 23 Sep 2021, Stanislav Lisovskiy <stanislav.lisovskiy@xxxxxxxxx> wrote: > > @@ -1941,6 +1951,10 @@ static bool gen12_plane_format_mod_supported(struct drm_plane *_plane, > > if (IS_ADLP_DISPLAY_STEP(dev_priv, STEP_A0, STEP_B0)) > > return false; > > break; > > + case I915_FORMAT_MOD_F_TILED: > > + if (!HAS_FTILE(dev_priv)) > > + return false; > > + fallthrough; > > default: > > return false; > > } > > Seems odd. I agree, however this wasn't even added by me. This patch got changed in ridiculous ways since last time, I ever touched it. Currently we have it internally exactly same way(wondering why) Unfortunately didn't pay attention to this, was assuming that if its r-bed and pushed - can trust it. Stan > > BR, > Jani. > > > -- > Jani Nikula, Intel Open Source Graphics Center
