[meta-xilinx] [meta-xilinx at yoctoproject.org] machine-xilinx-default.inc: Add XSERVER

[Nathan Rossi]

On 23 November 2016 at 07:41, Manjukumar Matha
<manjukumar.harthikote-matha at xilinx.com> wrote:
> Add X org X server configurations
>
> Signed-off-by: Manjukumar Matha <manjukumar.harthikote-matha at xilinx.com>
> ---
>  conf/machine/include/machine-xilinx-default.inc | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/conf/machine/include/machine-xilinx-default.inc b/conf/machine/include/machine-xilinx-default.inc
> index 1dfef48..25e38f7 100644
> --- a/conf/machine/include/machine-xilinx-default.inc
> +++ b/conf/machine/include/machine-xilinx-default.inc
> @@ -39,4 +39,13 @@ UBOOT_ELF_aarch64 ?= "u-boot.elf"
>  MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS_append_zynqmp = " \
>                 ${@bb.utils.contains('MACHINE_FEATURES', 'mali', bb.utils.contains('PREFERRED_PROVIDER_virtual/kernel', 'linux-xlnx', 'kernel-module-mali', '', d), '', d)} \
>                 "
> +XSERVER ?= " \
> +        xserver-xorg \
> +        xf86-input-evdev \
> +        xf86-input-mouse \
> +        xf86-input-keyboard \
> +        xf86-video-fbdev \

Need to add the variable below to XSERVER. Also consistency, tabs not spaces ;).

${XSERVER_EXT} \

> +        "
> +

Default it to "" for completeness.

XSERVER_EXT ?= ""

> +XSERVER_EXT_zynqmp ?= "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'xserver-xorg-extension-glx', '', d)}"

Should this also have "xf86-video-armsoc"?
Also just curious since ZynqMP "CG" devices don't have mali, should
the opengl support be skipped in that case, aka should the above only
be added if both MACHINE_FEATURES has mali and DISTRO_FEATURES has
opengl?

Regards,
Nathan

>
> --
> 2.7.4
>
> --
> _______________________________________________
> meta-xilinx mailing list
> meta-xilinx at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx