On Tue, Nov 13, 2018 at 06:49:38PM +0000, Chris Wilson wrote: > Quoting Stanislav Lisovskiy (2018-11-13 07:45:02) > > @@ -408,6 +424,9 @@ void sna_video_textured_setup(struct sna *sna, ScreenPtr screen) > > } else if (sna->kgem.gen < 040) { > > adaptor->nImages = ARRAY_SIZE(gen3_Images); > > adaptor->pImages = (XvImageRec *)gen3_Images; > > + } else if (sna->kgem.gen >= 0110) { > > + adaptor->nImages = ARRAY_SIZE(gen9_Images); > > + adaptor->pImages = (XvImageRec *)gen9_Images; > > } else { > > adaptor->nImages = ARRAY_SIZE(gen4_Images); > > adaptor->pImages = (XvImageRec *)gen4_Images; > > Grr. Aside from the minor nits, ok. Ville? Yeah. Seems good enough. For both Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel