On 15 June 2017 at 21:47, Robert Foss <[email protected]> wrote: > From: Rob Herring <[email protected]> > > Signed-off-by: Rob Herring <[email protected]> > --- > 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 <[email protected]> Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
