[PATCH 0/6] code size reduction for xloader

[PATCH 0/6] code size reduction for xloader

[Jean-Christophe PLAGNIOL-VILLARD]

HI,

	this patch serie wrok on disble non needed code to optimize the size
	of barebox specially in the case of non interactive version suc as
	xloader for TI and late bootstrap for at91

	tis will also iclude a sync of the kconfig need for this


The following changes since commit 21d0fac1382cbe20dfc95c0a400350632f28cbdb:

  at91: add Calao TNY-A9260/TNY-A9263/TNY-A9G20 (2012-01-11 09:41:25 +0100)

are available in the git repository at:
   git://uboot.jcrosoft.org/barebox.git size-optimisation

Jean-Christophe PLAGNIOL-VILLARD (6):
      banner: add config to disable it
      uncompress: add config to disable it
      filetype: add config to disable it
      kconfig: sync to linux 3.2-rc4
      param: add config to disable it
      driver: switch driver_d name to const char*

 Makefile                                 |    3 +-
 commands/Kconfig                         |    3 +
 common/Kconfig                           |    9 +
 common/Makefile                          |    4 +-
 drivers/mtd/core.c                       |   18 +-
 include/common.h                         |    4 +
 include/driver.h                         |    2 +-
 include/linux/kconfig.h                  |   32 +
 include/param.h                          |   53 +
 include/usb/usb.h                        |    2 +-
 lib/Kconfig                              |    9 +
 lib/Makefile                             |    4 +-
 lib/lzo/Kconfig                          |    2 +-
 scripts/kconfig/Makefile                 |  113 +-
 scripts/kconfig/conf.c                   |   50 +-
 scripts/kconfig/confdata.c               |  332 +++--
 scripts/kconfig/expr.c                   |   13 +-
 scripts/kconfig/expr.h                   |    7 -
 scripts/kconfig/gconf.c                  |   50 +-
 scripts/kconfig/kxgettext.c              |    1 -
 scripts/kconfig/lkc.h                    |   18 +-
 scripts/kconfig/lkc_proto.h              |    1 +
 scripts/kconfig/lxdialog/BIG.FAT.WARNING |    2 +-
 scripts/kconfig/lxdialog/textbox.c       |    3 +-
 scripts/kconfig/mconf.c                  |   86 +-
 scripts/kconfig/menu.c                   |   18 +-
 scripts/kconfig/nconf.c                  |   65 +-
 scripts/kconfig/nconf.gui.c              |   59 +-
 scripts/kconfig/nconf.h                  |    2 +-
 scripts/kconfig/qconf.cc                 |   22 +-
 scripts/kconfig/qconf.h                  |    2 +-
 scripts/kconfig/streamline_config.pl     |   38 +-
 scripts/kconfig/symbol.c                 |   47 +-
 scripts/kconfig/util.c                   |    2 +
 scripts/kconfig/zconf.gperf              |    2 +-
 scripts/kconfig/zconf.hash.c_shipped     |  273 ++--
 scripts/kconfig/zconf.l                  |   44 +-
 scripts/kconfig/zconf.lex.c              | 2420 ++++++++++++++++++++++++++++++
 scripts/kconfig/zconf.lex.c_shipped      | 2420 ++++++++++++++++++++++++++++++
 scripts/kconfig/zconf.tab.c_shipped      |   69 +-
 scripts/kconfig/zconf.y                  |   19 +-
 41 files changed, 5722 insertions(+), 601 deletions(-)
 create mode 100644 include/linux/kconfig.h
 create mode 100644 scripts/kconfig/zconf.lex.c
 create mode 100644 scripts/kconfig/zconf.lex.c_shipped

Best Regards,
J.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 1/6] banner: add config to disable it

[Jean-Christophe PLAGNIOL-VILLARD]

this will allow to save 144 bytes

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 commands/Kconfig |    1 +
 common/Kconfig   |    4 ++++
 common/Makefile  |    2 +-
 include/common.h |    4 ++++
 4 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/commands/Kconfig b/commands/Kconfig
index 6f9ccb8..1d080e3 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -420,6 +420,7 @@ config CMD_REGINFO
 config CMD_VERSION
 	tristate
 	default y
+	depends on BANNER
 	prompt "version"
 
 config CMD_HELP
diff --git a/common/Kconfig b/common/Kconfig
index ca4f099..d184d9e 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -74,6 +74,10 @@ config LOCALVERSION_AUTO
 config BOARDINFO
 	string
 
+config BANNER
+	bool "display banner"
+	default y
+
 config ENVIRONMENT_VARIABLES
 	bool "environment variables support"
 
diff --git a/common/Makefile b/common/Makefile
index d1132c3..eaf74a6 100644
--- a/common/Makefile
+++ b/common/Makefile
@@ -17,7 +17,7 @@ obj-$(CONFIG_MALLOC_TLSF) += tlsf_malloc.o
 obj-$(CONFIG_MALLOC_TLSF) += tlsf.o
 obj-$(CONFIG_MALLOC_DUMMY) += dummy_malloc.o
 obj-y += clock.o
-obj-y += version.o
+obj-$(CONFIG_BANNER) += version.o
 obj-$(CONFIG_COMMAND_SUPPORT) += command.o
 obj-$(CONFIG_CONSOLE_FULL) += console.o
 obj-$(CONFIG_CONSOLE_SIMPLE) += console_simple.o
diff --git a/include/common.h b/include/common.h
index 2f37dd8..a89b3bd 100644
--- a/include/common.h
+++ b/include/common.h
@@ -198,7 +198,11 @@ int run_shell(void);
 int memory_display(char *addr, ulong offs, ulong nbytes, int size);
 
 extern const char version_string[];
+#ifdef CONFIG_BANNER
 void barebox_banner(void);
+#else
+static inline void barebox_banner(void) {}
+#endif
 
 #define IOMEM(addr)	((void __force __iomem *)(addr))
 
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 2/6] uncompress: add config to disable it

[Jean-Christophe PLAGNIOL-VILLARD]

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 commands/Kconfig |    1 +
 lib/Kconfig      |    5 +++++
 lib/Makefile     |    2 +-
 lib/lzo/Kconfig  |    2 +-
 4 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/commands/Kconfig b/commands/Kconfig
index 1d080e3..b003786 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -310,6 +310,7 @@ config CMD_BOOTM
 	tristate
 	default y
 	select CRC32
+	select UNCOMPRESS
 	prompt "bootm"
 
 config CMD_BOOTM_SHOW_TYPE
diff --git a/lib/Kconfig b/lib/Kconfig
index 3339a9a..0bbd206 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -1,9 +1,14 @@
 menu "Library routines"
+config UNCOMPRESS
+	bool
+
 config ZLIB
 	bool "include gzip uncompression support"
+	select UNCOMPRESS
 
 config BZLIB
 	bool "include bzip2 uncompression support"
+	select UNCOMPRESS
 
 config GENERIC_FIND_NEXT_BIT
 	def_bool n
diff --git a/lib/Makefile b/lib/Makefile
index 7799e69..c273c58 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -30,6 +30,6 @@ obj-y			+= show_progress.o
 obj-$(CONFIG_LZO_DECOMPRESS)		+= decompress_unlzo.o
 obj-$(CONFIG_PROCESS_ESCAPE_SEQUENCE)	+= process_escape_sequence.o
 obj-$(CONFIG_FDT)	+= fdt/
-obj-y			+= uncompress.o
+obj-$(CONFIG_UNCOMPRESS)	+= uncompress.o
 obj-$(CONFIG_BCH)	+= bch.o
 obj-$(CONFIG_BITREV)	+= bitrev.o
diff --git a/lib/lzo/Kconfig b/lib/lzo/Kconfig
index 03c3350..9276b21 100644
--- a/lib/lzo/Kconfig
+++ b/lib/lzo/Kconfig
@@ -1,6 +1,6 @@
 config LZO_DECOMPRESS
 	bool "include lzo uncompression support"
-	bool
+	select UNCOMPRESS
 
 config LZO_COMPRESS
 	bool
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 3/6] filetype: add config to disable it

[Jean-Christophe PLAGNIOL-VILLARD]

remove 160 bytes

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

diff --git a/commands/Kconfig b/commands/Kconfig
index b003786..fad3281 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -311,6 +311,7 @@ config CMD_BOOTM
 	default y
 	select CRC32
 	select UNCOMPRESS
+	select FILETYPE
 	prompt "bootm"
 
 config CMD_BOOTM_SHOW_TYPE
diff --git a/common/Kconfig b/common/Kconfig
index d184d9e..20e4050 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -40,6 +40,9 @@ config BLOCK_WRITE
 config HAVE_NOSHELL
 	bool
 
+config FILETYPE
+	bool
+
 menu "General Settings              "
 
 config LOCALVERSION
diff --git a/common/Makefile b/common/Makefile
index eaf74a6..76fe407 100644
--- a/common/Makefile
+++ b/common/Makefile
@@ -28,7 +28,7 @@ obj-$(CONFIG_CMD_BOOTM) += uimage.o
 obj-y += startup.o
 obj-y += misc.o
 obj-y += memsize.o
-obj-y += filetype.o
+obj-$(CONFIG_FILETYPE) += filetype.o
 obj-y += resource.o
 obj-$(CONFIG_MENU) += menu.o
 obj-$(CONFIG_PASSWORD) += password.o
diff --git a/lib/Kconfig b/lib/Kconfig
index 0bbd206..6b4063d 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -1,6 +1,7 @@
 menu "Library routines"
 config UNCOMPRESS
 	bool
+	select FILETYPE
 
 config ZLIB
 	bool "include gzip uncompression support"
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 4/6] kconfig: sync to linux 3.2-rc4

[Jean-Christophe PLAGNIOL-VILLARD]

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 Makefile                                 |    3 +-
 include/linux/kconfig.h                  |   32 +
 scripts/kconfig/Makefile                 |  113 +-
 scripts/kconfig/conf.c                   |   50 +-
 scripts/kconfig/confdata.c               |  332 +++--
 scripts/kconfig/expr.c                   |   13 +-
 scripts/kconfig/expr.h                   |    7 -
 scripts/kconfig/gconf.c                  |   50 +-
 scripts/kconfig/kxgettext.c              |    1 -
 scripts/kconfig/lkc.h                    |   18 +-
 scripts/kconfig/lkc_proto.h              |    1 +
 scripts/kconfig/lxdialog/BIG.FAT.WARNING |    2 +-
 scripts/kconfig/lxdialog/textbox.c       |    3 +-
 scripts/kconfig/mconf.c                  |   86 +-
 scripts/kconfig/menu.c                   |   18 +-
 scripts/kconfig/nconf.c                  |   65 +-
 scripts/kconfig/nconf.gui.c              |   59 +-
 scripts/kconfig/nconf.h                  |    2 +-
 scripts/kconfig/qconf.cc                 |   22 +-
 scripts/kconfig/qconf.h                  |    2 +-
 scripts/kconfig/streamline_config.pl     |   38 +-
 scripts/kconfig/symbol.c                 |   47 +-
 scripts/kconfig/util.c                   |    2 +
 scripts/kconfig/zconf.gperf              |    2 +-
 scripts/kconfig/zconf.hash.c_shipped     |  273 ++--
 scripts/kconfig/zconf.l                  |   44 +-
 scripts/kconfig/zconf.lex.c              | 2420 ++++++++++++++++++++++++++++++
 scripts/kconfig/zconf.lex.c_shipped      | 2420 ++++++++++++++++++++++++++++++
 scripts/kconfig/zconf.tab.c_shipped      |   69 +-
 scripts/kconfig/zconf.y                  |   19 +-
 30 files changed, 5627 insertions(+), 586 deletions(-)
 create mode 100644 include/linux/kconfig.h
 create mode 100644 scripts/kconfig/zconf.lex.c
 create mode 100644 scripts/kconfig/zconf.lex.c_shipped

http://git.jcrosoft.org/git?p=barebox.git;a=commitdiff;h=18e18dd13b5d756d9ac32458a7982d25056328fb;hp=507319e13bd5e0ad13138970e90e8426d08f5817

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 5/6] param: add config to disable it

[Jean-Christophe PLAGNIOL-VILLARD]

this will allow to save 992 Bytes for TI xlaoder or AT91 bootstrap

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 common/Kconfig     |    2 +
 drivers/mtd/core.c |   18 +++++++++-------
 include/param.h    |   53 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 lib/Kconfig        |    3 ++
 lib/Makefile       |    2 +-
 5 files changed, 69 insertions(+), 9 deletions(-)

diff --git a/common/Kconfig b/common/Kconfig
index 20e4050..382e591 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -258,6 +258,7 @@ choice
 		bool "hush parser"
 		select ENVIRONMENT_VARIABLES
 		select COMMAND_SUPPORT
+		select PARAMETER
 		help
 		  Enable hush support. This is the most advanced shell available
 		  for barebox.
@@ -266,6 +267,7 @@ choice
 		bool "Simple parser"
 		select ENVIRONMENT_VARIABLES
 		select COMMAND_SUPPORT
+		select PARAMETER
 		help
 		  simple shell. No if/then, no return values from commands, no loops
 
diff --git a/drivers/mtd/core.c b/drivers/mtd/core.c
index f25863d..491cc1f 100644
--- a/drivers/mtd/core.c
+++ b/drivers/mtd/core.c
@@ -204,14 +204,16 @@ int add_mtd_device(struct mtd_info *mtd, char *devname)
 	mtd->cdev.dev = &mtd->class_dev;
 	mtd->cdev.mtd = mtd;
 
-	sprintf(str, "%u", mtd->size);
-	dev_add_param_fixed(&mtd->class_dev, "size", str);
-	sprintf(str, "%u", mtd->erasesize);
-	dev_add_param_fixed(&mtd->class_dev, "erasesize", str);
-	sprintf(str, "%u", mtd->writesize);
-	dev_add_param_fixed(&mtd->class_dev, "writesize", str);
-	sprintf(str, "%u", mtd->oobsize);
-	dev_add_param_fixed(&mtd->class_dev, "oobsize", str);
+	if (IS_ENABLED(CONFIG_PARAMETER)) {
+		sprintf(str, "%u", mtd->size);
+		dev_add_param_fixed(&mtd->class_dev, "size", str);
+		sprintf(str, "%u", mtd->erasesize);
+		dev_add_param_fixed(&mtd->class_dev, "erasesize", str);
+		sprintf(str, "%u", mtd->writesize);
+		dev_add_param_fixed(&mtd->class_dev, "writesize", str);
+		sprintf(str, "%u", mtd->oobsize);
+		dev_add_param_fixed(&mtd->class_dev, "oobsize", str);
+	}
 
 	devfs_create(&mtd->cdev);
 
diff --git a/include/param.h b/include/param.h
index 207ad50..4a39dc7 100644
--- a/include/param.h
+++ b/include/param.h
@@ -19,6 +19,7 @@ struct param_d {
 	struct list_head list;
 };
 
+#ifdef CONFIG_PARAMETER
 const char *dev_get_param(struct device_d *dev, const char *name);
 int dev_set_param(struct device_d *dev, const char *name, const char *val);
 struct param_d *get_param_by_name(struct device_d *dev, const char *name);
@@ -40,6 +41,58 @@ int dev_set_param_ip(struct device_d *dev, char *name, IPaddr_t ip);
 IPaddr_t dev_get_param_ip(struct device_d *dev, char *name);
 
 int global_add_parameter(struct param_d *param);
+#else
+static inline const char *dev_get_param(struct device_d *dev, const char *name)
+{
+	return NULL;
+}
+static inline int dev_set_param(struct device_d *dev, const char *name,
+				const char *val)
+{
+	return 0;
+}
+static inline struct param_d *get_param_by_name(struct device_d *dev,
+						const char *name)
+{
+	return NULL;
+}
+
+static inline int dev_add_param(struct device_d *dev, char *name,
+		int (*set)(struct device_d *dev, struct param_d *p, const char *val),
+		char *(*get)(struct device_d *, struct param_d *p),
+		unsigned long flags)
+{
+	return 0;
+}
+
+static inline int dev_add_param_fixed(struct device_d *dev, char *name, char *value)
+{
+	return 0;
+}
+
+static inline void dev_remove_parameters(struct device_d *dev) {}
+
+static inline int dev_param_set_generic(struct device_d *dev, struct param_d *p,
+		const char *val)
+{
+	return 0;
+}
+
+/* Convenience functions to handle a parameter as an ip address */
+static inline int dev_set_param_ip(struct device_d *dev, char *name, IPaddr_t ip)
+{
+	return 0;
+}
+static inline IPaddr_t dev_get_param_ip(struct device_d *dev, char *name)
+{
+	return 0;
+}
+
+static inline int global_add_parameter(struct param_d *param)
+{
+	return 0;
+}
+#endif
 
 #endif /* PARAM_H */
 
diff --git a/lib/Kconfig b/lib/Kconfig
index 6b4063d..f886e6e 100644
--- a/lib/Kconfig
+++ b/lib/Kconfig
@@ -1,4 +1,7 @@
 menu "Library routines"
+config PARAMETER
+	bool
+
 config UNCOMPRESS
 	bool
 	select FILETYPE
diff --git a/lib/Makefile b/lib/Makefile
index c273c58..01a01b5 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -5,7 +5,7 @@ obj-y			+= string.o
 obj-y			+= vsprintf.o
 obj-y			+= div64.o
 obj-y			+= misc.o
-obj-y			+= parameter.o
+obj-$(CONFIG_PARAMETER)	+= parameter.o
 obj-y			+= xfuncs.o
 obj-y			+= getopt.o
 obj-y 			+= readkey.o
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 6/6] driver: switch driver_d name to const char*

[Jean-Christophe PLAGNIOL-VILLARD]

this will allow to save 200 bytes on a9g20 and 180 bytes on ti xlaoder

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 include/driver.h  |    2 +-
 include/usb/usb.h |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/driver.h b/include/driver.h
index 1751d3c..1b8b16d 100644
--- a/include/driver.h
+++ b/include/driver.h
@@ -105,7 +105,7 @@ struct device_d {
 struct driver_d {
 	/*! The name of this driver. Used to match to
 	 * the corresponding device. */
-	char name[MAX_DRIVER_NAME];
+	const char *name;
 
 	struct list_head list;
 
diff --git a/include/usb/usb.h b/include/usb/usb.h
index 08fd1a1..a61a008 100644
--- a/include/usb/usb.h
+++ b/include/usb/usb.h
@@ -187,7 +187,7 @@ struct usb_device {
 struct usb_device_id;
 
 struct usb_driver {
-	char	*name;
+	const char	*name;
 	int     (*probe) (struct usb_device *, const struct usb_device_id *);
 	void    (*disconnect)(struct usb_device *);
 
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 0/6] code size reduction for xloader

[Sascha Hauer]

Hi J,

On Wed, Jan 11, 2012 at 02:29:14PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> HI,
> 
> 	this patch serie wrok on disble non needed code to optimize the size
> 	of barebox specially in the case of non interactive version suc as
> 	xloader for TI and late bootstrap for at91
> 
> 	tis will also iclude a sync of the kconfig need for this
> 
> 
> The following changes since commit 21d0fac1382cbe20dfc95c0a400350632f28cbdb:
> 
>   at91: add Calao TNY-A9260/TNY-A9263/TNY-A9G20 (2012-01-11 09:41:25 +0100)
> 
> are available in the git repository at:
>    git://uboot.jcrosoft.org/barebox.git size-optimisation

Your git server does not work for me. A git pull just stalls. The web
frontend works well though.

Can you check this?

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 0/6] code size reduction for xloader

[Jean-Christophe PLAGNIOL-VILLARD]

On 09:50 Thu 12 Jan     , Sascha Hauer wrote:
> Hi J,
> 
> On Wed, Jan 11, 2012 at 02:29:14PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > HI,
> > 
> > 	this patch serie wrok on disble non needed code to optimize the size
> > 	of barebox specially in the case of non interactive version suc as
> > 	xloader for TI and late bootstrap for at91
> > 
> > 	tis will also iclude a sync of the kconfig need for this
> > 
> > 
> > The following changes since commit 21d0fac1382cbe20dfc95c0a400350632f28cbdb:
> > 
> >   at91: add Calao TNY-A9260/TNY-A9263/TNY-A9G20 (2012-01-11 09:41:25 +0100)
> > 
> > are available in the git repository at:
> >    git://uboot.jcrosoft.org/barebox.git size-optimisation
> 
> Your git server does not work for me. A git pull just stalls. The web
> frontend works well though.
> 
> Can you check this?
check ready

Best Regards,
J.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

RE: [PATCH 0/6] code size reduction for xloader

[Premi, Sanjeev]

> -----Original Message-----
> From: barebox-bounces@lists.infradead.org 
> [mailto:barebox-bounces@lists.infradead.org] On Behalf Of 
> Jean-Christophe PLAGNIOL-VILLARD
> Sent: Wednesday, January 11, 2012 6:59 PM
> To: barebox@lists.infradead.org
> Subject: [PATCH 0/6] code size reduction for xloader
> 
> HI,
> 
> 	this patch serie wrok on disble non needed code to 
> optimize the size
> 	of barebox specially in the case of non interactive 
> version suc as
> 	xloader for TI and late bootstrap for at91
> 
> 	tis will also iclude a sync of the kconfig need for this

I manually pulled all the patches in this series against latest
"master" except (didn't seemed related to size reduction):
      kconfig: sync to linux 3.2-rc4

I encountered compile error in this patch:
    param: add config to disable it

  CC      drivers/mtd/core.o
drivers/mtd/core.c: In function 'add_mtd_device':
drivers/mtd/core.c:230:2: warning: implicit declaration of function 'IS_ENABLED'
drivers/mtd/core.c:230:17: error: 'CONFIG_PARAMETER' undeclared (first use in this function)
drivers/mtd/core.c:230:17: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [drivers/mtd/core.o] Error 1
make[1]: *** [drivers/mtd] Error 2
make: *** [drivers] Error 2

I am not sure whether IS_ENABLED is implemented in yet-to-be merged patch;
so, continued testing after applying the temp patch at bottom of
this mail.

With the modification, tested all working fine on OMAP3EVM.

I had to apply these patches from the series I posted yesterday,
otherwise, barebox doesn't boot:
   [PATCH 3/5] ARM omap3evm: Allow building first stage
   [PATCH 4/5] ARM omap3evm: Enable HSMMC device

You may add Tested-by: Sanjeev Premi<premi@ti.com> to these:
      banner: add config to disable it
      uncompress: add config to disable it
      filetype: add config to disable it
      driver: switch driver_d name to const char*

~sanjeev

From bbe95b44869d63d991bb26a9098c5f123f3c757e Mon Sep 17 00:00:00 2001
From: Sanjeev Premi <premi@ti.com>
Date: Thu, 12 Jan 2012 20:16:15 +0530
Subject: [PATCH] temp: fix failure due to a portion of prev patch


Signed-off-by: Sanjeev Premi <premi@ti.com>
---
 drivers/mtd/core.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/mtd/core.c b/drivers/mtd/core.c
index ccc5c26..dc0d5b0 100644
--- a/drivers/mtd/core.c
+++ b/drivers/mtd/core.c
@@ -227,6 +227,7 @@ int add_mtd_device(struct mtd_info *mtd, char *devname)
        mtd->cdev.dev = &mtd->class_dev;
        mtd->cdev.mtd = mtd;

+#ifdef CONFIG_PARAMETER
        if (IS_ENABLED(CONFIG_PARAMETER)) {
                sprintf(str, "%u", mtd->size);
                dev_add_param_fixed(&mtd->class_dev, "size", str);
@@ -237,7 +238,7 @@ int add_mtd_device(struct mtd_info *mtd, char *devname)
                sprintf(str, "%u", mtd->oobsize);
                dev_add_param_fixed(&mtd->class_dev, "oobsize", str);
        }
-
+#endif
        devfs_create(&mtd->cdev);

        list_for_each_entry(hook, &mtd_register_hooks, hook)

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 0/6] code size reduction for xloader

[Jean-Christophe PLAGNIOL-VILLARD]

On 15:14 Thu 12 Jan     , Premi, Sanjeev wrote:
> > -----Original Message-----
> > From: barebox-bounces@lists.infradead.org 
> > [mailto:barebox-bounces@lists.infradead.org] On Behalf Of 
> > Jean-Christophe PLAGNIOL-VILLARD
> > Sent: Wednesday, January 11, 2012 6:59 PM
> > To: barebox@lists.infradead.org
> > Subject: [PATCH 0/6] code size reduction for xloader
> > 
> > HI,
> > 
> > 	this patch serie wrok on disble non needed code to 
> > optimize the size
> > 	of barebox specially in the case of non interactive 
> > version suc as
> > 	xloader for TI and late bootstrap for at91
> > 
> > 	tis will also iclude a sync of the kconfig need for this
> 
> I manually pulled all the patches in this series against latest
> "master" except (didn't seemed related to size reduction):
>       kconfig: sync to linux 3.2-rc4
because you do need this one

to be able to the IS_ENABLED macro
> 
> I encountered compile error in this patch:
>     param: add config to disable it
> 
>   CC      drivers/mtd/core.o
> drivers/mtd/core.c: In function 'add_mtd_device':
> drivers/mtd/core.c:230:2: warning: implicit declaration of function 'IS_ENABLED'
> drivers/mtd/core.c:230:17: error: 'CONFIG_PARAMETER' undeclared (first use in this function)
> drivers/mtd/core.c:230:17: note: each undeclared identifier is reported only once for each function it appears in
> make[2]: *** [drivers/mtd/core.o] Error 1
> make[1]: *** [drivers/mtd] Error 2
> make: *** [drivers] Error 2
> 
> I am not sure whether IS_ENABLED is implemented in yet-to-be merged patch;
> so, continued testing after applying the temp patch at bottom of
> this mail.

Best Regards,
J.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox