[meta-xilinx] [RFC] machine-xilinx-default.inc: Add XSERVER

[Nathan Rossi]

On Fri, May 6, 2016 at 2:42 AM, Manjukumar Harthikote Matha
<manjukumar.harthikote-matha at xilinx.com> wrote:
> Hi Nathan
>
> On 05/05/2016 04:58 AM, Nathan Rossi wrote:
>>
>> On Thu, May 5, 2016 at 6:06 PM, Manjukumar Matha
>> <manjukumar.harthikote-matha at xilinx.com> wrote:
>>>
>>> Add X org X server configurations for ZynqMP.
>>
>>
> [....]
>>>
>>> +XSERVER = " \
>>
>>
>> ?= (since this is an include)
>
> Ok
>>
>>
>>> +           xserver-xorg \
>>> +           xf86-input-evdev \
>>> +           xf86-input-mouse \
>>> +           xf86-input-keyboard \
>>
>>
>> Add 'xf86-video-fbdev' to match the default (set in
>> packagegroup-core-x11-xserver).
>
> The reason was we need xf86-video-armsoc for ZynqMP. Maybe I need to
> decouple for other architectures and ZynqMP

It is fine, in general it is common to install multiple xf86-video-*
packages, and zynqmp can use fbdev. The xf86-video-armsoc can be added
to the zynqmp specific additions.

The "Driver" setting from your other patch for xf86-video-armsoc which
adds the "xf86-video-armsoc/20-zynqmp.conf" config will set xserver to
use armsoc.

>
>>
>>> +           ${@bb.utils.contains('DISTRO_FEATURES', 'opengl',
>>> 'xserver-xorg-extension-glx', '', d)} \
>>
>>
>> Not sure about this one specifically for Zynq and MicroBlaze, since
>> neither have GL compatible hw. Probably worth setting this via another
>> variable that can be expanded later for soc/arch/machine specifics,
>> something like:
>
>
> Mesa libraries work for Zynq but I am not sure about MicroBlaze

As a software implementation sure, but I am not sure if it is worth
including by default on those targets. For this patch, make it
specific for zynqmp only, can add it for other targets later if it
makes sense.

>>
>>
>> XSERVER_EXT ?= ""
>> XSERVER_EXT_zynqmp ?= "${@bb.utils.contains('DISTRO_FEATURES',
>> 'opengl', 'xserver-xorg-extension-glx', '', d)}"
>>
> Ok thanks, I was thinking on similar lines to add xf86-video-armsoc in
> tune-zynqmp. I will rework to add it in machine-xilinx-default

Adding the 'xf86-video-armsoc' in machine-xilinx-default.inc is
probably the best place to put it. The tune include is not really
intended for these sorts of configs.

Regards,
Nathan