[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[Jean-Christophe PLAGNIOL-VILLARD]

> +
> +config BR2_PACKAGE_LCDPROC_DRIVER_ULA200
> +	bool "ula200"
> +	help
> +	  Driver for ula200 devices
> +
> +config BR2_PACKAGE_LCDPROC_DRIVER_XOSD
> +	bool "xosd"
> +	help
> +	  Driver for xosd devices
> +
> +endmenu
> diff --git a/package/lcdproc/lcdproc.mk b/package/lcdproc/lcdproc.mk
> index e8919e9..e0964a5 100644
> --- a/package/lcdproc/lcdproc.mk
> +++ b/package/lcdproc/lcdproc.mk
> @@ -7,7 +7,156 @@ LCDPROC_VERSION = 0.5.5
>  LCDPROC_SOURCE = lcdproc-$(LCDPROC_VERSION).tar.gz
>  LCDPROC_SITE = http://$(BR2_SOURCEFORGE_MIRROR).dl.sourceforge.net/project/lcdproc/lcdproc/$(LCDPROC_VERSION)
>  
> -LCDPROC_CONF_OPT = --enable-drivers=all
> +LCDPROC_DRIVERS =
> +ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_BAYRAD),y)
> +	LCDPROC_DRIVERS += bayrad
> +endif
LCDPROC_DRIVERS-$(BR2_PACKAGE_LCDPROC_DRIVER_BAYRAD) += bayrad

and then use
LCDPROC_DRIVERS-y


so we can drop the ifeq blabla

Best Regards,
J.

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[spdawson at gmail.com]

From: Simon Dawson <spdawson@gmail.com>

Signed-off-by: Simon Dawson <spdawson@gmail.com>
---
 package/lcdproc/Config.in  |  246 ++++++++++++++++++++++++++++++++++++++++++++
 package/lcdproc/lcdproc.mk |  151 +++++++++++++++++++++++++++-
 2 files changed, 396 insertions(+), 1 deletions(-)

diff --git a/package/lcdproc/Config.in b/package/lcdproc/Config.in
index 6c8614d..1f0f185 100644
--- a/package/lcdproc/Config.in
+++ b/package/lcdproc/Config.in
@@ -5,3 +5,249 @@ config BR2_PACKAGE_LCDPROC
          LCD display driver daemon and clients
 
          http://lcdproc.org/
+
+menu "Drivers"
+	depends on BR2_PACKAGE_LCDPROC
+
+config BR2_PACKAGE_LCDPROC_DRIVER_BAYRAD
+	bool "bayrad"
+	help
+	  Driver for bayrad devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_CFONTZ
+	bool "CFontz"
+	help
+	  Driver for CFontz devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_CFONTZ633
+	bool "CFontz633"
+	help
+	  Driver for CFontz633 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_CFONTZPACKET
+	bool "CFontzPacket"
+	help
+	  Driver for CFontzPacket devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_CURSES
+	bool "curses"
+	default y
+	help
+	  Driver for curses devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_CWLNX
+	bool "CwLnx"
+	help
+	  Driver for CwLnx devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_EA65
+	bool "ea65"
+	help
+	  Driver for ea65 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_EYEBOXONE
+	bool "EyeboxOne"
+	help
+	  Driver for EyeboxOne devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_G15
+	bool "g15"
+	help
+	  Driver for g15 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_GLCDLIB
+	bool "glcdlib"
+	help
+	  Driver for glcdlib devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_GLK
+	bool "glk"
+	help
+	  Driver for glk devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_HD44780
+	bool "hd44780"
+	help
+	  Driver for hd44780 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_I2500VFD
+	bool "i2500vfd"
+	help
+	  Driver for i2500vfd devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_ICP_A106
+	bool "icp_a106"
+	help
+	  Driver for icp_a106 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_IMON
+	bool "imon"
+	help
+	  Driver for imon devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_IMONLCD
+	bool "imonlcd"
+	help
+	  Driver for imonlcd devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_IOWARRIOR
+	bool "IOWarrior"
+	help
+	  Driver for IOWarrior devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_IRMAN
+	bool "irman"
+	help
+	  Driver for irman devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_IRTRANS
+	bool "irtrans"
+	help
+	  Driver for irtrans devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_JOY
+	bool "joy"
+	help
+	  Driver for joy devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_LB216
+	bool "lb216"
+	help
+	  Driver for lb216 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_LCDM001
+	bool "lcdm001"
+	help
+	  Driver for lcdm001 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_LCTERM
+	bool "lcterm"
+	help
+	  Driver for lcterm devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_LIRC
+	bool "lirc"
+	help
+	  Driver for lirc devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_LIS
+	bool "lis"
+	help
+	  Driver for lis devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MD8800
+	bool "MD8800"
+	help
+	  Driver for MD8800 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MDM166A
+	bool "mdm166a"
+	help
+	  Driver for mdm166a devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MS6931
+	bool "ms6931"
+	help
+	  Driver for ms6931 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MTC_S16209X
+	bool "mtc_s16209x"
+	help
+	  Driver for mtc_s16209x devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MTXORB
+	bool "MtxOrb"
+	help
+	  Driver for MtxOrb devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_MX5000
+	bool "mx5000"
+	help
+	  Driver for mx5000 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_NORITAKEVFD
+	bool "NoritakeVFD"
+	help
+	  Driver for NoritakeVFD devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_PICOLCD
+	bool "picolcd"
+	help
+	  Driver for picolcd devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_PYRAMID
+	bool "pyramid"
+	help
+	  Driver for pyramid devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SED1330
+	bool "sed1330"
+	help
+	  Driver for sed1330 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SED1520
+	bool "sed1520"
+	help
+	  Driver for sed1520 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SERIALPOS
+	bool "serialPOS"
+	help
+	  Driver for serialPOS devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SERIALVFD
+	bool "serialVFD"
+	help
+	  Driver for serialVFD devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SHUTTLEVFD
+	bool "shuttleVFD"
+	help
+	  Driver for shuttleVFD devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SLI
+	bool "sli"
+	help
+	  Driver for sli devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_STV5730
+	bool "stv5730"
+	help
+	  Driver for stv5730 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SUREELEC
+	bool "SureElec"
+	help
+	  Driver for SureElec devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_SVGA
+	bool "svga"
+	help
+	  Driver for svga devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_T6963
+	bool "t6963"
+	help
+	  Driver for t6963 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_TEXT
+	bool "text"
+	help
+	  Driver for text devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_TYAN
+	bool "tyan"
+	help
+	  Driver for tyan devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_ULA200
+	bool "ula200"
+	help
+	  Driver for ula200 devices
+
+config BR2_PACKAGE_LCDPROC_DRIVER_XOSD
+	bool "xosd"
+	help
+	  Driver for xosd devices
+
+endmenu
diff --git a/package/lcdproc/lcdproc.mk b/package/lcdproc/lcdproc.mk
index e8919e9..e0964a5 100644
--- a/package/lcdproc/lcdproc.mk
+++ b/package/lcdproc/lcdproc.mk
@@ -7,7 +7,156 @@ LCDPROC_VERSION = 0.5.5
 LCDPROC_SOURCE = lcdproc-$(LCDPROC_VERSION).tar.gz
 LCDPROC_SITE = http://$(BR2_SOURCEFORGE_MIRROR).dl.sourceforge.net/project/lcdproc/lcdproc/$(LCDPROC_VERSION)
 
-LCDPROC_CONF_OPT = --enable-drivers=all
+LCDPROC_DRIVERS =
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_BAYRAD),y)
+	LCDPROC_DRIVERS += bayrad
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_CFONTZ),y)
+	LCDPROC_DRIVERS += CFontz
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_CFONTZ633),y)
+	LCDPROC_DRIVERS += CFontz633
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_CFONTZPACKET),y)
+	LCDPROC_DRIVERS += CFontzPacket
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_CURSES),y)
+	LCDPROC_DRIVERS += curses
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_CWLNX),y)
+	LCDPROC_DRIVERS += CwLnx
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_EA65),y)
+	LCDPROC_DRIVERS += ea65
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_EYEBOXONE),y)
+	LCDPROC_DRIVERS += EyeboxOne
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_G15),y)
+	LCDPROC_DRIVERS += g15
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_GLCDLIB),y)
+	LCDPROC_DRIVERS += glcdlib
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_GLK),y)
+	LCDPROC_DRIVERS += glk
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_HD44780),y)
+	LCDPROC_DRIVERS += hd44780
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_I2500VFD),y)
+	LCDPROC_DRIVERS += i2500vfd
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_ICP_A106),y)
+	LCDPROC_DRIVERS += icp_a106
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_IMON),y)
+	LCDPROC_DRIVERS += imon
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_IMONLCD),y)
+	LCDPROC_DRIVERS += imonlcd
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_IOWARRIOR),y)
+	LCDPROC_DRIVERS += IOWarrior
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_IRMAN),y)
+	LCDPROC_DRIVERS += irman
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_IRTRANS),y)
+	LCDPROC_DRIVERS += irtrans
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_JOY),y)
+	LCDPROC_DRIVERS += joy
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_LB216),y)
+	LCDPROC_DRIVERS += lb216
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_LCDM001),y)
+	LCDPROC_DRIVERS += lcdm001
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_LCTERM),y)
+	LCDPROC_DRIVERS += lcterm
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_LIRC),y)
+	LCDPROC_DRIVERS += lirc
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_LIS),y)
+	LCDPROC_DRIVERS += lis
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MD8800),y)
+	LCDPROC_DRIVERS += MD8800
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MDM166A),y)
+	LCDPROC_DRIVERS += mdm166a
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MS6931),y)
+	LCDPROC_DRIVERS += ms6931
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MTC_S16209X),y)
+	LCDPROC_DRIVERS += mtc_s16209x
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MTXORB),y)
+	LCDPROC_DRIVERS += MtxOrb
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_MX5000),y)
+	LCDPROC_DRIVERS += mx5000
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_NORITAKEVFD),y)
+	LCDPROC_DRIVERS += NoritakeVFD
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_PICOLCD),y)
+	LCDPROC_DRIVERS += picolcd
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_PYRAMID),y)
+	LCDPROC_DRIVERS += pyramid
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SED1330),y)
+	LCDPROC_DRIVERS += sed1330
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SED1520),y)
+	LCDPROC_DRIVERS += sed1520
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SERIALPOS),y)
+	LCDPROC_DRIVERS += serialPOS
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SERIALVFD),y)
+	LCDPROC_DRIVERS += serialVFD
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SHUTTLEVFD),y)
+	LCDPROC_DRIVERS += shuttleVFD
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SLI),y)
+	LCDPROC_DRIVERS += sli
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_STV5730),y)
+	LCDPROC_DRIVERS += stv5730
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SUREELEC),y)
+	LCDPROC_DRIVERS += SureElec
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_SVGA),y)
+	LCDPROC_DRIVERS += svga
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_T6963),y)
+	LCDPROC_DRIVERS += t6963
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_TEXT),y)
+	LCDPROC_DRIVERS += text
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_TYAN),y)
+	LCDPROC_DRIVERS += tyan
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_ULA200),y)
+	LCDPROC_DRIVERS += ula200
+endif
+ifeq ($(BR2_PACKAGE_LCDPROC_DRIVER_XOSD),y)
+	LCDPROC_DRIVERS += xosd
+endif
+
+ifeq ($(LCDPROC_DRIVERS),)
+	LCDPROC_DRIVERS = all
+endif
+LCDPROC_CONF_OPT = --enable-drivers=$(shell echo $(LCDPROC_DRIVERS) | sed -e 's/ /,/g')
 
 LCDPROC_DEPENDENCIES = ncurses
 
-- 
1.7.5.4

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[Thomas Petazzoni]

Hello Simon,

Le Wed, 11 Apr 2012 11:02:48 +0100,
spdawson at gmail.com a ?crit :

> From: Simon Dawson <spdawson@gmail.com>
> 
> Signed-off-by: Simon Dawson <spdawson@gmail.com>
> ---
>  package/lcdproc/Config.in  |  246 ++++++++++++++++++++++++++++++++++++++++++++
>  package/lcdproc/lcdproc.mk |  151 +++++++++++++++++++++++++++-
>  2 files changed, 396 insertions(+), 1 deletions(-)

Thanks for this patch. I am wondering whether we should have such a
long list of drivers as a menu, or whether we should just provide an
option to give a comma-separated list of drivers. Something like the
following choice:

 [ ] Build all drivers
 [ ] Build custom list of drivers

When the second choice is enabled, then it shows:

 () List of drivers to build

But I don't feel really strongly about this, so please wait for
comments from other Buildroot developers before working on such change.

> +LCDPROC_CONF_OPT = --enable-drivers=$(shell echo $(LCDPROC_DRIVERS) | sed -e 's/ /,/g')

This can be done in pure make:

LCDPROC_CONF_OPT = --enable-drivers=$(subst $(space),$(comma),$(LCDPROC_DRIVERS))

Regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[spdawson at gmail.com]

Add a configuration menu item to specify a list of lcdproc drivers to
be built. The list defaults to 'all'.

Signed-off-by: Simon Dawson <spdawson@gmail.com>
---
 package/lcdproc/Config.in  |    6 ++++++
 package/lcdproc/lcdproc.mk |    2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/package/lcdproc/Config.in b/package/lcdproc/Config.in
index 6c8614d..e997e91 100644
--- a/package/lcdproc/Config.in
+++ b/package/lcdproc/Config.in
@@ -5,3 +5,9 @@ config BR2_PACKAGE_LCDPROC
          LCD display driver daemon and clients
 
          http://lcdproc.org/
+
+config BR2_PACKAGE_LCDPROC_DRIVERS
+	string "List of drivers to build"
+	default "all"
+	help
+	  Specify a comma-separated list of lcdproc drivers to be built
diff --git a/package/lcdproc/lcdproc.mk b/package/lcdproc/lcdproc.mk
index e8919e9..745b386 100644
--- a/package/lcdproc/lcdproc.mk
+++ b/package/lcdproc/lcdproc.mk
@@ -7,7 +7,7 @@ LCDPROC_VERSION = 0.5.5
 LCDPROC_SOURCE = lcdproc-$(LCDPROC_VERSION).tar.gz
 LCDPROC_SITE = http://$(BR2_SOURCEFORGE_MIRROR).dl.sourceforge.net/project/lcdproc/lcdproc/$(LCDPROC_VERSION)
 
-LCDPROC_CONF_OPT = --enable-drivers=all
+LCDPROC_CONF_OPT = --enable-drivers=$(BR2_PACKAGE_LCDPROC_DRIVERS)
 
 LCDPROC_DEPENDENCIES = ncurses
 
-- 
1.7.5.4

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[spdawson at gmail.com]

Add a configuration menu item to specify a list of lcdproc drivers to
be built. The list defaults to 'all'.

Signed-off-by: Simon Dawson <spdawson@gmail.com>
---
 package/lcdproc/Config.in  |    7 +++++++
 package/lcdproc/lcdproc.mk |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/package/lcdproc/Config.in b/package/lcdproc/Config.in
index 6c8614d..70a2078 100644
--- a/package/lcdproc/Config.in
+++ b/package/lcdproc/Config.in
@@ -5,3 +5,10 @@ config BR2_PACKAGE_LCDPROC
          LCD display driver daemon and clients
 
          http://lcdproc.org/
+
+config BR2_PACKAGE_LCDPROC_DRIVERS
+	string "List of drivers to build"
+	default "all"
+	help
+	  Specify a comma-separated list of lcdproc drivers to be built
+	depends on BR2_PACKAGE_LCDPROC
diff --git a/package/lcdproc/lcdproc.mk b/package/lcdproc/lcdproc.mk
index e8919e9..745b386 100644
--- a/package/lcdproc/lcdproc.mk
+++ b/package/lcdproc/lcdproc.mk
@@ -7,7 +7,7 @@ LCDPROC_VERSION = 0.5.5
 LCDPROC_SOURCE = lcdproc-$(LCDPROC_VERSION).tar.gz
 LCDPROC_SITE = http://$(BR2_SOURCEFORGE_MIRROR).dl.sourceforge.net/project/lcdproc/lcdproc/$(LCDPROC_VERSION)
 
-LCDPROC_CONF_OPT = --enable-drivers=all
+LCDPROC_CONF_OPT = --enable-drivers=$(BR2_PACKAGE_LCDPROC_DRIVERS)
 
 LCDPROC_DEPENDENCIES = ncurses
 
-- 
1.7.5.4

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[Thomas Petazzoni]

Hello,

Le Wed, 11 Apr 2012 12:31:30 +0100,
spdawson at gmail.com a ?crit :

> Add a configuration menu item to specify a list of lcdproc drivers to
> be built. The list defaults to 'all'.

Ok, I told you to wait for others comments on whether we should have
the full list (as you did initially) or this solution. I didn't want to
waste your time on this. Some comments below, though.

> +config BR2_PACKAGE_LCDPROC_DRIVERS
> +	string "List of drivers to build"
> +	default "all"
> +	help
> +	  Specify a comma-separated list of lcdproc drivers to be built
> +	depends on BR2_PACKAGE_LCDPROC

The depends on should be before the help text. It's not mandatory by
the kconfig syntax, but that's the way we use kconfig everywhere in
Buildroot.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[spdawson at gmail.com]

Add a configuration menu item to specify a list of lcdproc drivers to
be built. The list defaults to 'all'.

Signed-off-by: Simon Dawson <spdawson@gmail.com>
---
 package/lcdproc/Config.in  |    7 +++++++
 package/lcdproc/lcdproc.mk |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/package/lcdproc/Config.in b/package/lcdproc/Config.in
index 6c8614d..7261105 100644
--- a/package/lcdproc/Config.in
+++ b/package/lcdproc/Config.in
@@ -5,3 +5,10 @@ config BR2_PACKAGE_LCDPROC
          LCD display driver daemon and clients
 
          http://lcdproc.org/
+
+config BR2_PACKAGE_LCDPROC_DRIVERS
+	string "List of drivers to build"
+	default "all"
+	depends on BR2_PACKAGE_LCDPROC
+	help
+	  Specify a comma-separated list of lcdproc drivers to be built
diff --git a/package/lcdproc/lcdproc.mk b/package/lcdproc/lcdproc.mk
index e8919e9..745b386 100644
--- a/package/lcdproc/lcdproc.mk
+++ b/package/lcdproc/lcdproc.mk
@@ -7,7 +7,7 @@ LCDPROC_VERSION = 0.5.5
 LCDPROC_SOURCE = lcdproc-$(LCDPROC_VERSION).tar.gz
 LCDPROC_SITE = http://$(BR2_SOURCEFORGE_MIRROR).dl.sourceforge.net/project/lcdproc/lcdproc/$(LCDPROC_VERSION)
 
-LCDPROC_CONF_OPT = --enable-drivers=all
+LCDPROC_CONF_OPT = --enable-drivers=$(BR2_PACKAGE_LCDPROC_DRIVERS)
 
 LCDPROC_DEPENDENCIES = ncurses
 
-- 
1.7.5.4

[Buildroot] [PATCH] Add lcdproc config options, for driver selection

[Peter Korsgaard]

>>>>> "spdawson" == spdawson  <spdawson@gmail.com> writes:

 spdawson> Add a configuration menu item to specify a list of lcdproc
 spdawson> drivers to be built. The list defaults to 'all'.

Committed, thanks.

-- 
Bye, Peter Korsgaard