[Mesa-dev] [PATCH v1 4/7] gbm: add XBGR8888 support for dumb buffers

Emil Velikov emil.l.velikov at gmail.com
Fri Jun 16 13:16:19 UTC 2017


On 15 June 2017 at 21:47, Robert Foss <robert.foss at collabora.com> wrote:
> From: Rob Herring <robh at kernel.org>
>
> Signed-off-by: Rob Herring <robh at 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.velikov at collabora.com>

Emil


More information about the mesa-dev mailing list