diff mbox

[1/7] stddev: make it selectable via Kconfig

Message ID 1342963142-30339-1-git-send-email-plagnioj@jcrosoft.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jean-Christophe PLAGNIOL-VILLARD July 22, 2012, 1:18 p.m. UTC
enable is by default for simple and hush
we do not need to NO_SHELL

this allow to save 1KiB

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 commands/Makefile |    2 +-
 common/Kconfig    |    5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

Comments

Baruch Siach July 22, 2012, 1:43 p.m. UTC | #1
Hi Jean-Christophe,

On Sun, Jul 22, 2012 at 03:18:56PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> enable is by default for simple and hush
> we do not need to NO_SHELL
> 
> this allow to save 1KiB
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

Wrong list. This series was intended for the Barebox list.

baruch
Thomas Petazzoni July 22, 2012, 1:50 p.m. UTC | #2
Le Sun, 22 Jul 2012 15:18:56 +0200,
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> a écrit :

> enable is by default for simple and hush
> we do not need to NO_SHELL
> 
> this allow to save 1KiB
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
>  commands/Makefile |    2 +-
>  common/Kconfig    |    5 +++++
>  2 files changed, 6 insertions(+), 1 deletion(-)

Hum, I guess this should have been sent to the barebox@ mailing list,
rather than the linux-arm-kernel@ mailing list, no?

Thomas
Jean-Christophe PLAGNIOL-VILLARD July 22, 2012, 2:04 p.m. UTC | #3
On 15:50 Sun 22 Jul     , Thomas Petazzoni wrote:
> Le Sun, 22 Jul 2012 15:18:56 +0200,
> Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> a écrit :
> 
> > enable is by default for simple and hush
> > we do not need to NO_SHELL
> > 
> > this allow to save 1KiB
> > 
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> > ---
> >  commands/Makefile |    2 +-
> >  common/Kconfig    |    5 +++++
> >  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> Hum, I guess this should have been sent to the barebox@ mailing list,
> rather than the linux-arm-kernel@ mailing list, no?
yeap I send it to the wrong ml

Best Regards,
diff mbox

Patch

diff --git a/commands/Makefile b/commands/Makefile
index 54191b4..e9157bf 100644
--- a/commands/Makefile
+++ b/commands/Makefile
@@ -1,4 +1,4 @@ 
-obj-y				+= stddev.o
+obj-$(CONFIG_STDDEV)		+= stddev.o
 obj-$(CONFIG_CMD_BOOTM)		+= bootm.o
 obj-$(CONFIG_CMD_UIMAGE)	+= uimage.o
 obj-$(CONFIG_CMD_LINUX16)	+= linux16.o
diff --git a/common/Kconfig b/common/Kconfig
index 1c996aa..c6f1afa 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -50,6 +50,9 @@  config BINFMT
 config GLOBALVAR
 	bool
 
+config STDDEV
+	bool
+
 menu "General Settings              "
 
 config LOCALVERSION
@@ -295,6 +298,7 @@  choice
 		select COMMAND_SUPPORT
 		select PARAMETER
 		select BINFMT
+		select STDDEV
 		help
 		  Enable hush support. This is the most advanced shell available
 		  for barebox.
@@ -304,6 +308,7 @@  choice
 		select ENVIRONMENT_VARIABLES
 		select COMMAND_SUPPORT
 		select PARAMETER
+		select STDDEV
 		help
 		  simple shell. No if/then, no return values from commands, no loops