[meta-intel] [PATCH 5/6] meta-valleyisland: add graphics support for valleyisland bsp

[Kamble, Nitin A]

On 2/5/2014 1:25 AM, rebecca.swee.fun.chang at intel.com wrote:
> From: "Chang, Rebecca Swee Fun" <rebecca.swee.fun.chang at intel.com>
>
> Provide graphics support for valleyisland bsp.
>
> Signed-off-by: Chang, Rebecca Swee Fun <rebecca.swee.fun.chang at intel.com>
> ---
>   .../xserver-xf86-config/valleyisland-32/xorg.conf  |   26 ++++++++++++++++++++
>   .../xserver-xf86-config/valleyisland-64/xorg.conf  |   26 ++++++++++++++++++++
>   .../xorg-xserver/xserver-xf86-config_0.1.bbappend  |    1 +
>   3 files changed, 53 insertions(+)
>   create mode 100644 meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-32/xorg.conf
>   create mode 100644 meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-64/xorg.conf
>   create mode 100644 meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bbappend
>
> diff --git a/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-32/xorg.conf b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-32/xorg.conf
> new file mode 100644
> index 0000000..01d2cb5
> --- /dev/null
> +++ b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-32/xorg.conf
> @@ -0,0 +1,26 @@
> +Section "Device"
> +    Identifier    "Generic FrameBuffer"
> +    Driver        "fbdev"
> +EndSection
> +
> +Section "Monitor"
> +    Identifier    "Generic Monitor"
> +    Option        "DPMS"
> +EndSection
> +
> +Section "Screen"
> +    Identifier    "Default Screen"
> +    Device        "Generic FrameBuffer"
> +    Monitor       "Generic Monitor"
> +    DefaultDepth  24
> +EndSection
> +
> +Section "ServerLayout"
> +    Identifier    "Default Layout"
> +    Screen        "Default Screen"
> +EndSection
> +
> +Section "InputClass"
> +    Identifier    "default"
> +    Driver        "evdev"
> +EndSection
> diff --git a/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-64/xorg.conf b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-64/xorg.conf
> new file mode 100644
> index 0000000..01d2cb5
> --- /dev/null
> +++ b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config/valleyisland-64/xorg.conf
> @@ -0,0 +1,26 @@
> +Section "Device"
> +    Identifier    "Generic FrameBuffer"
> +    Driver        "fbdev"
> +EndSection
> +
> +Section "Monitor"
> +    Identifier    "Generic Monitor"
> +    Option        "DPMS"
> +EndSection
> +
> +Section "Screen"
> +    Identifier    "Default Screen"
> +    Device        "Generic FrameBuffer"
> +    Monitor       "Generic Monitor"
> +    DefaultDepth  24
> +EndSection
> +
> +Section "ServerLayout"
> +    Identifier    "Default Layout"
> +    Screen        "Default Screen"
> +EndSection
> +
> +Section "InputClass"
> +    Identifier    "default"
> +    Driver        "evdev"
> +EndSection
> diff --git a/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bbappend b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bbappend
> new file mode 100644
> index 0000000..72d991c
> --- /dev/null
> +++ b/meta-isg/meta-valleyisland/recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bbappend
> @@ -0,0 +1 @@
> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
Darren also pointed this out.Are these X config files really needed? 
Most of the BSPs are doing fine without such X config files. Can you 
check booting the BSP images without the xorg.conf files? And if they 
are not necessary, then drop this commit entirely.

Thanks,
Nitin