[meta-freescale] [PATCH 25/32] imx-dpu-g2d: Add new G2D implementation based on DPU

[Gary Bisson]

Hi Otavio, Tom,

On Mon, Jun 25, 2018 at 04:30:25PM -0300, Otavio Salvador wrote:
> From: Tom Hochstein <tom.hochstein at nxp.com>
> 
> Signed-off-by: Tom Hochstein <tom.hochstein at nxp.com>
> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> ---
> 
>  .../imx-dpu-g2d/imx-dpu-g2d_1.4.2.bb          | 37 +++++++++++++++++++
>  1 file changed, 37 insertions(+)
>  create mode 100644 recipes-graphics/imx-dpu-g2d/imx-dpu-g2d_1.4.2.bb
> 
> diff --git a/recipes-graphics/imx-dpu-g2d/imx-dpu-g2d_1.4.2.bb b/recipes-graphics/imx-dpu-g2d/imx-dpu-g2d_1.4.2.bb
> new file mode 100644
> index 00000000..8abbfad8
> --- /dev/null
> +++ b/recipes-graphics/imx-dpu-g2d/imx-dpu-g2d_1.4.2.bb
> @@ -0,0 +1,37 @@
> +# Copyright (C) 2016 Freescale Semiconductor
> +# Copyright 2017-2018 NXP
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +
> +DESCRIPTION = "G2D graphics library and apps for i.MX with DRM"
> +LICENSE = "Proprietary"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=75abe2fa1d16ca79f87cde926f05f72d"
> +
> +RDEPENDS_${PN} = "virtual/libgal-x11 libdrm"
> +
> +PROVIDES += "virtual/libg2d"
> +
> +SRC_URI = "${FSL_MIRROR}/${PN}-${PV}.bin;fsl-eula=true"
> +
> +S="${WORKDIR}/${PN}-${PV}"
> +
> +inherit fsl-eula-unpack
> +
> +SRC_URI[md5sum] = "9beeb2c66f1bad4e82a77487486a31c8"
> +SRC_URI[sha256sum] = "905e797d7fb573b5fea616940b07dc15d817fe2f5e94668f883e9faaa516989a"
> +
> +do_install () {
> +
> +    install -d ${D}${libdir}
> +    install -d ${D}${includedir}
> +
> +    cp -r ${S}/g2d/usr/lib/*.so* ${D}${libdir}
> +    cp -Pr ${S}/g2d/usr/include/* ${D}${includedir}
> +    cp -r ${S}/gpu-demos/opt ${D}
> +}
> +
> +INSANE_SKIP_${PN} += "ldflags"
> +
> +FILES_${PN} = "${libdir}/libg2d* /opt"
> +FILES_${PN}-dev = "${libdir}/libg2d${SOLIBSDEV} ${includedir}"
> +
> +COMPATIBLE_MACHINE = "(mx8)"

Shouldn't the compatible be more specific here to only match platforms
that actually have a DPU? mx8m for instance should be excluded.

Regards,
Gary