[Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package

[Peter Korsgaard]

On Fri, Jul 31, 2015 at 12:59 AM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> From: Scott Fan <fancp2007 at gmail.com>
>
> Add the fbturbo video driver, which is based on xf86-video-fbdev (with
> none of the original features stripped), primarily optimized for the
> devices powered by the Allwinner SoC (A10, A13, A20).
>
> https://github.com/ssvb/xf86-video-fbturbo/
>
> Signed-off-by: Scott Fan <fancp2007 at gmail.com>
> [yann.morin.1998 at free.fr: strip help text to the basics only]
> Signed-off-by: Yann E. MORIN <yann.morin.1998 at free.fr>
> ---
>  package/x11r7/Config.in                             |  1 +
>  package/x11r7/xdriver_xf86-video-fbturbo/Config.in  | 16 ++++++++++++++++
>  .../xdriver_xf86-video-fbturbo.mk                   | 21 +++++++++++++++++++++
>  3 files changed, 38 insertions(+)
>  create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/Config.in
>  create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
>
> +++ b/package/x11r7/xdriver_xf86-video-fbturbo/Config.in
> @@ -0,0 +1,16 @@
> +config BR2_PACKAGE_XDRIVER_XF86_VIDEO_FBTURBO
> +       bool "xf86-video-fbturbo"
> +       select BR2_PACKAGE_LIBDRI2

It actually doesn't use libdri2 for anything (it does contain a stray
#include "dri2.h" but it isn't needed and I've added a patch to drop
it).

The configure script also checks for libdrm and pixman, so I've added
that as well.

> +       select BR2_PACKAGE_XPROTO_FONTSPROTO
> +       select BR2_PACKAGE_XPROTO_RANDRPROTO
> +       select BR2_PACKAGE_XPROTO_RENDERPROTO
> +       select BR2_PACKAGE_XPROTO_VIDEOPROTO
> +       select BR2_PACKAGE_XPROTO_XPROTO
> +       select BR2_PACKAGE_XPROTO_XF86DRIPROTO
> +       help
> +         video driver, primarily optimized for the devices powered
> +         by the Allwinner SoC (A10, A13, A20). It can use some of
> +         the 2D/3D hardware acceleration features.
> +
> +         https://github.com/ssvb/xf86-video-fbturbo
> +
> diff --git a/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
> new file mode 100644
> index 0000000..18f80d5
> --- /dev/null
> +++ b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# xdriver_xf86-video-fbturbo
> +#
> +################################################################################
> +
> +XDRIVER_XF86_VIDEO_FBTURBO_VERSION = 0.4.0
> +XDRIVER_XF86_VIDEO_FBTURBO_SITE = $(call github,ssvb,xf86-video-fbturbo,$(XDRIVER_XF86_VIDEO_FBTURBO_VERSION))
> +XDRIVER_XF86_VIDEO_FBTURBO_LICENSE = MIT
> +XDRIVER_XF86_VIDEO_FBTURBO_LICENSE_FILES = COPYING
> +XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES = xserver_xorg-server xproto_fontsproto xproto_randrproto xproto_renderproto xproto_videoproto xproto_xproto
> +
> +XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES += xproto_xf86driproto libdri2

There's no need for the += here, it can all be handled on a single
_DEPENDENCIES assignment. I've fixed the overlong line as well.

The package also has optional support for libpciaccess, so I've added that.

Committed with these fixes, thanks.

-- 
Bye, Peter Korsgaard