[U-Boot] alpr U-Boot image too big

[U-Boot] alpr U-Boot image too big

[Stefan Roese]

Hi Pieter,

I just noticed that the alpr doesn't build anymore in the current U-Boot 
version (main and 4xx). The image size (256k) is exceeded because of general 
code growth. We either need to remove some features/commands from the alpr 
target or increase the image size. Please advise how to fix this. A patch 
would be most welcome.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

[U-Boot] alpr U-Boot image too big

[Stefan Roese]

Hi Pieter,

On Friday 21 November 2008, Stefan Roese wrote:
> I just noticed that the alpr doesn't build anymore in the current U-Boot
> version (main and 4xx). The image size (256k) is exceeded because of
> general code growth. We either need to remove some features/commands from
> the alpr target or increase the image size. Please advise how to fix this.
> A patch would be most welcome.

Any news on this?

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

[U-Boot] alpr U-Boot image too big

[Pieter Voorthuijsen]

Hi Stefan,

I think I missed the previous mail during my holiday. I quickly checked the header file, here what can be undefined from the CONFIG_CMD_xx's:

#define CONFIG_CMD_REGINFO
#define CONFIG_CMD_NFS
#define CONFIG_CMD_PCI

I also believe CMD_ELF is not needed for only booting a kernel, but not sure.

I'm sure that disabling these will save enough kilobytes to make it fit....

Best regards, Pieter


> -----Original Message-----
> From: Stefan Roese [mailto:sr at denx.de] 
> Sent: dinsdag 9 december 2008 17:16
> To: u-boot at lists.denx.de
> Cc: Pieter Voorthuijsen
> Subject: Re: [U-Boot] alpr U-Boot image too big
> 
> Hi Pieter,
> 
> On Friday 21 November 2008, Stefan Roese wrote:
> > I just noticed that the alpr doesn't build anymore in the current 
> > U-Boot version (main and 4xx). The image size (256k) is exceeded 
> > because of general code growth. We either need to remove some 
> > features/commands from the alpr target or increase the 
> image size. Please advise how to fix this.
> > A patch would be most welcome.
> 
> Any news on this?
> 
> Thanks.
> 
> Best regards,
> Stefan
> 
> =====================================================================
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: 
> office at denx.de 
> =====================================================================
> 

Disclaimer: The information contained in this email, including any attachments is 
confidential and is for the sole use of the intended recipient(s). Any unauthorized 
review, use, disclosure or distribution is prohibited. If you are not the intended 
recipient, please notify the sender immediately by replying to this message and 
destroy all copies of this message and any attachments.

[U-Boot] alpr U-Boot image too big

[Stefan Roese]

Hi Pieter,

On Tuesday 09 December 2008, Pieter Voorthuijsen wrote:
> I think I missed the previous mail during my holiday. I quickly checked the
> header file, here what can be undefined from the CONFIG_CMD_xx's:
>
> #define CONFIG_CMD_REGINFO
> #define CONFIG_CMD_NFS
> #define CONFIG_CMD_PCI

Are you really sure that you want to drop the PCI commands? IIRC then the ALPR 
has some PCI devices and it might be handy to access them from within U-Boot 
(for debugging etc).

> I also believe CMD_ELF is not needed for only booting a kernel, but not
> sure.

Yes, CMD_ELF is not needed for Linux kernel booting.

> I'm sure that disabling these will save enough kilobytes to make it fit....

I just checked and by disabling these features we fit into the 256k again:

#undef CONFIG_CMD_NFS
#undef CONFIG_CMD_REGINFO
#undef CONFIG_CMD_ELF

I'll submit a patch in a sec. Please send your ack if you think this is ok.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================