On 15 June 2017 at 21:47, Robert Foss <robert.f...@collabora.com> wrote: > From: Rob Herring <r...@kernel.org> > > Signed-off-by: Rob Herring <r...@kernel.org> > --- > src/gbm/backends/dri/gbm_dri.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c > index 19be440d48..58b62ac361 100644 > --- a/src/gbm/backends/dri/gbm_dri.c > +++ b/src/gbm/backends/dri/gbm_dri.c > @@ -1067,7 +1067,7 @@ create_dumb(struct gbm_device *gbm, > is_cursor = (usage & GBM_BO_USE_CURSOR) != 0 && > format == GBM_FORMAT_ARGB8888; > is_scanout = (usage & GBM_BO_USE_SCANOUT) != 0 && > - format == GBM_FORMAT_XRGB8888; > + (format == GBM_FORMAT_XRGB8888 || format == GBM_FORMAT_XBGR8888); AFAICT there's no other parts that need updating.
Still, the commit message should mention, an actual use case for the extra format. With that Reviewed-by: Emil Velikov <emil.veli...@collabora.com> Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
