[Buildroot] [PATCH 06/28] xserver_xorg-server: option to build KDrive input drivers

[Arnout Vandecappelle]

On 01/06/13 22:29, Thomas Petazzoni wrote:
> When the Kdrive variant of the X server is selected, the drivers are
> directly built into the X server. The X server therefore provides
> options to enable or disable certain drivers, especially input
> drivers.
>
> This patch adds options to be able to enable or disable the evdev, kbd
> and mouse drivers of Kdrive.
>
> Signed-off-by: Thomas Petazzoni<thomas.petazzoni at free-electrons.com>
> ---
>   package/x11r7/xserver_xorg-server/Config.in        |   29 ++++++++++++++++++++
>   .../xserver_xorg-server/xserver_xorg-server.mk     |   19 +++++++++++++
>   2 files changed, 48 insertions(+)
>
> diff --git a/package/x11r7/xserver_xorg-server/Config.in b/package/x11r7/xserver_xorg-server/Config.in
> index 4ac0ec7..2382786 100644
> --- a/package/x11r7/xserver_xorg-server/Config.in
> +++ b/package/x11r7/xserver_xorg-server/Config.in
> @@ -87,6 +87,35 @@ config BR2_PACKAGE_XSERVER_XORG_SERVER_AIGLX
>       help
>           Enable/Use AIGLX extension.
>
> +if BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE
> +
> +config BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_EVDEV
> +       bool "Enable KDrive/TinyX evdev input driver"
> +       help
> +         Enable evdev input driver in KDrive. This allows KDrive to
> +         use input devices that are integrated in the Linux input
> +         subsystem, i.e have a /dev/input/eventX device file. This is
> +         most likely the input driver you want to use.
> +
> +	 This driver can then be enabled by running the X server
> +         using:

  Indentation seems to be screwed up a bit here... Actually, there are 
not tabs at all!

> +
> +           Xfbdev -keybd evdev,,device=/dev/input/eventX -mouse evdev,,device=/dev/input/eventY.
> +
> +config BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_KBD
> +       bool "Enable KDrive/TinyX kbd input driver"
> +       help
> +         Enable kbd input driver in KDrive. It uses the console
> +         keyboard as input device.
> +
> +config BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_MOUSE
> +       bool "Enable KDrive/TinyX mouse input driver"
> +       help
> +         Enable mouse input driver in KDrive. It supports PS/2 mice
> +         and serial port mice.
> +
> +endif
> +
>   comment "Optional Servers"
>
>   config BR2_PACKAGE_XSERVER_XORG_SERVER_XVFB
> diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> index 623b25d..552d8e8 100644
> --- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> +++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> @@ -76,6 +76,25 @@ define XSERVER_CREATE_X_SYMLINK
>    ln -f -s Xfbdev $(TARGET_DIR)/usr/bin/X
>   endef
>   XSERVER_XORG_SERVER_POST_INSTALL_TARGET_HOOKS += XSERVER_CREATE_X_SYMLINK
> +
> +ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_EVDEV),y)
> +XSERVER_XORG_SERVER_CONF_OPT += --enable-kdrive-evdev
> +else
> +XSERVER_XORG_SERVER_CONF_OPT += --disable-kdrive-evdev
> +endif
> +
> +ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_KBD),y)
> +XSERVER_XORG_SERVER_CONF_OPT += --enable-kdrive-kbd
> +else
> +XSERVER_XORG_SERVER_CONF_OPT += --disable-kdrive-kbd
> +endif
> +
> +ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_KDRIVE_MOUSE),y)
> +XSERVER_XORG_SERVER_CONF_OPT += --enable-kdrive-mouse
> +else
> +XSERVER_XORG_SERVER_CONF_OPT += --disable-kdrive-mouse
> +endif
> +
>   else

  With your addition this else is moving very far from the if, so a 
comment would be useful.

  Regards,
  Arnout


>   XSERVER_XORG_SERVER_CONF_OPT += --disable-kdrive --disable-xfbdev
>   endif


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F