Re: [U-Boot] [PATCH] Remove static type from gunzip_bmp()

2009-01-27 Thread Wolfgang Denk
Dear Mark Jackson,

In message <497ee024.9040...@mimc.co.uk> you wrote:
> THe current bmp routines support gzipped files, but compilation fails due to 
> the gunzip_bmp() function being defined as static.

I cannot confirm that. There are boards that use this feature, and
they build fine.

> This patch fixes this problem.
> 
> Signed-off-by: Mark Jackson 
> ---
>   common/cmd_bmp.c |2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/common/cmd_bmp.c b/common/cmd_bmp.c
> index bc08b26..2104e59 100644
> --- a/common/cmd_bmp.c
> +++ b/common/cmd_bmp.c
> @@ -46,7 +46,7 @@ int gunzip(void *, int, unsigned char *, unsigned long *);
>* didn't contain a valid BMP signature.
>*/
>   #ifdef CONFIG_VIDEO_BMP_GZIP
> -static bmp_image_t *gunzip_bmp(unsigned long addr, unsigned long *lenp)
> +bmp_image_t *gunzip_bmp(unsigned long addr, unsigned long *lenp)

There is another ocurrence of "static bmp_image_t *gunzip_bmp()" in
this file - you change one and leave the other unchanged.

If you change this, please change both places so they stay in sync,
and add a proper prototype declaration to the header file.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"A child is a person who can't understand why someone would give away
a perfectly good kitten."   - Doug Larson
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] Remove static type from gunzip_bmp()

2009-01-27 Thread Mark Jackson
THe current bmp routines support gzipped files, but compilation fails due to 
the gunzip_bmp() function being defined as static.

This patch fixes this problem.

Signed-off-by: Mark Jackson 
---
  common/cmd_bmp.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/common/cmd_bmp.c b/common/cmd_bmp.c
index bc08b26..2104e59 100644
--- a/common/cmd_bmp.c
+++ b/common/cmd_bmp.c
@@ -46,7 +46,7 @@ int gunzip(void *, int, unsigned char *, unsigned long *);
   * didn't contain a valid BMP signature.
   */
  #ifdef CONFIG_VIDEO_BMP_GZIP
-static bmp_image_t *gunzip_bmp(unsigned long addr, unsigned long *lenp)
+bmp_image_t *gunzip_bmp(unsigned long addr, unsigned long *lenp)
  {
void *dst;
unsigned long len;
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot