[Buildroot] [PATCH 2/8] package/opengl/libgbm: introduce api features
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Tue Sep 21 14:10:41 UTC 2021
Hello Kamel,
On Tue, 21 Sep 2021 15:03:19 +0200
Kamel Bouhara <kamel.bouhara at bootlin.com> wrote:
> We introduce two feature that shall help user choosing the version
> implemented by a libgbm provider. This foresightly avoid building package
> without having the required libgbm version (e.g. kmscube, qt5, sdl2
> etc.)
>
> Signed-off-by: Kamel Bouhara <kamel.bouhara at bootlin.com>
I think this should be squashed into PATCH 1.
> diff --git a/package/opengl/libgbm/Config.in b/package/opengl/libgbm/Config.in
> index ee18c4e6cb..f6d68e847a 100644
> --- a/package/opengl/libgbm/Config.in
> +++ b/package/opengl/libgbm/Config.in
> @@ -4,3 +4,15 @@ config BR2_PACKAGE_HAS_LIBGBM
> config BR2_PACKAGE_PROVIDES_LIBGBM
> string
> depends on BR2_PACKAGE_HAS_LIBGBM
> +
> +config BR2_PACKAGE_LIBGBM_HAS_FEATURE_FORMAT_MODIFIER_PLANE_COUNT
> + bool
> + depends on BR2_PACKAGE_HAS_LIBGBM
> +
> +comment "feature implemented in mesa gbm api version 17 or later"
I'm not sure this makes sense as a comment visible in menuconfig, as
the option itself is not even visible. I would rather suggest something
like this:
# gbm implementations should select this option if they provide the
# format modifier plane count feature. This API was initially introduced
# in mesa3d version 17. A gbm implementation provides this feature if it
# is implement function A, B, C.
> +config BR2_PACKAGE_LIBGBM_HAS_FEATURE_DMA_BUF
> + bool
> + depends on BR2_PACKAGE_HAS_LIBGBM
> +
> +comment "feature implemented in mesa gbm api 10 or later"
Same here.
Best regards,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
More information about the buildroot
mailing list