Coding Style cleanup, update CHANGELOG

Signed-off-by: Wolfgang Denk <wd@denx.de>

CHANGELOG

@@ -1,3 +1,111 @@
+commit fc6414eca55f1fc108fb12fc8cdc43bd8b4463f9
+Author: Mike Frysinger <vapier@gentoo.org>
+Date:	Tue Dec 18 04:29:55 2007 -0500
+
+    fix easylogo on big endian dev systems
+
+    didnt realize how out of shape easylogo actually was until i tried using it.
+    this patch does byte swapping as need be on the input tga header since the tga
+    is in little endian but the host could just as well be big endian.	i didnt
+    bother using bswap macros or such stuff from system headers as nothing in
+    POSIX dictates byte swapping functionality.
+
+    Signed-Off-By: Mike Frysinger <vapier@gentoo.org>
+
+commit 38d299c2db81bd889c601b5dfc12c4e83ef83333
+Author: Mike Frysinger <vapier@gentoo.org>
+Date:	Tue Dec 18 03:23:25 2007 -0500
+
+    cleanup easylogo
+
+    - make the Makefile not suck
+    - include proper headers for prototypes
+    - fix obvious broken handling of strchr() when handling '.' in filenames
+
+    Signed-Off-By: Mike Frysinger <vapier@gentoo.org>
+
+commit 883e3925d99a8dd69c5b0201cba5b1887f88f95c
+Author: raptorbrino@aim.com <raptorbrino@aim.com>
+Date:	Thu Dec 13 21:23:28 2007 -0500
+
+    Fix build problems under Cygwin
+
+    This patch allows u-boot to build without error in a cygwin
+    environment.  Cygwin does not define __u64 in it's
+    include/asm/types.h file.  The -idirafter flag in the u-boot
+    build causes the inclusion of the cygwin types.h file as opposed
+    to u-bot/include/asm/types.h file which does define __u64.
+    Subsequently, sha1.c compile fails due to unknown symbol.
+
+    Signed-off-by: Brian Miller <raptorbrino@netscape.net>
+
+commit 43ef1c381f9195504a2488f5cb909227eb97d475
+Author: Hans-Christian Egtvedt <hcegtvedt@atmel.com>
+Date:	Fri Nov 30 17:29:59 2007 +0100
+
+    cmd_bmp: Add support for displaying gzip compressed bmps
+
+    The existing code can show information about a gzip compressed BMP
+    image, but can't actually display it.
+
+    Therefore, move the decompression code out of bmp_info() and use it in
+    bmp_display() as well in order to display a compressed BMP image.
+
+    Also, clean things up a bit and fix a memory leak while we're at it.
+
+    [hskinnemoen@atmel.com: a bit of refactoring]
+    Signed-off-by: Haavard Skinnemoen <hskinnemoen@atmel.com>
+
+commit d197ffd8172c6fdef38733424640a9a47295d6e9
+Author: Guennadi Liakhovetski <lg@denx.de>
+Date:	Thu Nov 29 21:15:56 2007 +0100
+
+    Fix and optimize MII operations on FEC (MPC8xx) controllers
+
+    This patch fixes several issues at least on a MPC885 based system with two
+    FEC interfaces used in MII mode.
+
+    1. PHY discovery should first read PHY_PHYIDR2 register and only then
+       PHY_PHYIDR1 like cpu/mpc8xx/fec.c::mii_discover_phy() does it,
+       otherwise the values read are wrong. Also notice, that PHY discovery
+       cannot work on MPC88x / MPC87x in setups with both FECs active at all
+       in its present form, because for both interfaces the registers from FEC
+       1 are used to communicate over MII.
+
+    2. Remove code duplication for resetting the FEC by isolating it into a
+       separate function.
+
+    3. Initialize MII on FEC 1 when communicating over FEC 2 in fec_init().
+
+    4. Optimize mii_init() to only reset the FEC 1 controller once.
+
+    5. Fix a typo in mii_init() using index i instead of j thus potentially
+       leading to unpredictable results.
+
+    Signed-off-by: Guennadi Liakhovetski <lg@denx.de>
+
+commit 6a5e1d75bf106fa157e9ce68bcaf4b13e8a1d214
+Author: Guennadi Liakhovetski <lg@denx.de>
+Date:	Tue Nov 20 13:14:20 2007 +0100
+
+    Fix endianness conversions in rtl8169 driver
+
+    It is unclear on what platforms this driver has been tested, since
+    noone up to now defines CONFIG_RTL8169 in the board configuration
+    header. Now it has been fixed for a big-endian mpc8241 based
+    linkstation platform. This patch presents the necessary endianness
+    conversion fixes.
+
+    Signed-off-by: Guennadi Liakhovetski <lg@denx.de>
+
+commit 07eb02687f008721974a2fb54cd7fdc28033ab3c
+Author: Wolfgang Denk <wd@denx.de>
+Date:	Wed Jan 9 13:43:38 2008 +0100
+
+    Coding Style clenaup; update CHANGELOG
+
+    Signed-off-by: Wolfgang Denk <wd@denx.de>
+
 commit c26acc1a43b31ddca5add42fd0360ff0eee90c80
 Author: Matthias Fuchs <matthias.fuchs@esd-electronics.com>
 Date:	Thu Dec 27 17:13:11 2007 +0100

tools/easylogo/easylogo.c

@@ -43,16 +43,8 @@ typedef struct {
 } yuyv_t;
 
 typedef struct {
-	void				*data,
+	void *data, *palette;
-					*palette ;
+	int width, height, pixels, bpp, pixel_size, size, palette_size, yuyv;
-	int				width,
-					height,
-					pixels,
-					bpp,
-					pixel_size,
-					size,
-					palette_size,
-					yuyv;
 } image_t;
 
 void StringUpperCase (char *str)
@@ -60,8 +52,7 @@ void StringUpperCase (char *str)
 	int count = strlen (str);
 	char c;
 
-    while(count--)
+	while (count--) {
-    {
 		c = *str;
 		if ((c >= 'a') && (c <= 'z'))
 			*str = 'A' + (c - 'a');
@@ -74,8 +65,7 @@ void StringLowerCase (char *str)
 	int count = strlen (str);
 	char c;
 
-    while(count--)
+	while (count--) {
-    {
 		c = *str;
 		if ((c >= 'A') && (c <= 'Z'))
 			*str = 'a' + (c - 'A');
@@ -102,10 +92,11 @@ void pixel_rgb_to_yuyv (rgb_t *rgb_pixel, yuyv_t *yuyv_pixel)
 void printlogo_rgb (rgb_t * data, int w, int h)
 {
 	int x, y;
-    for (y=0; y<h; y++)
+
-    {
+	for (y = 0; y < h; y++) {
 		for (x = 0; x < w; x++, data++)
-	    if ((data->r < 30)/*&&(data->g == 0)&&(data->b == 0)*/)
+			if ((data->r <
+			     30) /*&&(data->g == 0)&&(data->b == 0) */ )
 				printf (" ");
 			else
 				printf ("X");
@@ -116,8 +107,8 @@ void printlogo_rgb (rgb_t	*data, int w, int h)
 void printlogo_yuyv (unsigned short *data, int w, int h)
 {
 	int x, y;
-    for (y=0; y<h; y++)
+
-    {
+	for (y = 0; y < h; y++) {
 		for (x = 0; x < w; x++, data++)
 			if (*data == 0x1080)	/* Because of inverted on i386! */
 				printf (" ");
@@ -133,6 +124,7 @@ static inline unsigned short le16_to_cpu (unsigned short val)
 		unsigned char pval[2];
 		unsigned short val;
 	} swapped;
+
 	swapped.val = val;
 	return (swapped.pval[1] << 8) + swapped.pval[0];
 }
@@ -179,8 +171,7 @@ int image_load_tga (image_t *image, char *filename)
 	image->size = image->pixels * image->pixel_size;
 	image->data = malloc (image->size);
 
-    if (image->bpp != 24)
+	if (image->bpp != 24) {
-    {
 		printf ("Bpp not supported: %d!\n", image->bpp);
 		return -1;
 	}
@@ -190,8 +181,7 @@ int image_load_tga (image_t *image, char *filename)
 /* Swapping R and B values */
 
 	p = image->data;
-    for(i=0; i < image->pixels; i++, p++)
+	for (i = 0; i < image->pixels; i++, p++) {
-    {
 		app = p->r;
 		p->r = p->b;
 		p->b = app;
@@ -199,27 +189,25 @@ int image_load_tga (image_t *image, char *filename)
 
 /* Swapping image */
 
-    if(!(header.ImageDescriptorByte & 0x20))
+	if (!(header.ImageDescriptorByte & 0x20)) {
-    {
 		unsigned char *temp = malloc (image->size);
 		int linesize = image->pixel_size * image->width;
 		void *dest = image->data,
 			*source = temp + image->size - linesize;
 
 		printf ("S");
-	if (temp == NULL)
+		if (temp == NULL) {
-	{
 			printf ("Cannot alloc temp buffer!\n");
 			return -1;
 		}
 
 		memcpy (temp, image->data, image->size);
-	for(i = 0; i<image->height; i++, dest+=linesize, source-=linesize)
+		for (i = 0; i < image->height;
+		     i++, dest += linesize, source -= linesize)
 			memcpy (dest, source, linesize);
 
 		free (temp);
 	}
-
 #ifdef ENABLE_ASCII_BANNERS
 	printlogo_rgb (image->data, image->width, image->height);
 #endif
@@ -253,12 +241,12 @@ int image_rgb_to_yuyv (image_t *rgb_image, image_t *yuyv_image)
 	yuyv_image->height = rgb_image->height;
 	yuyv_image->pixels = yuyv_image->width * yuyv_image->height;
 	yuyv_image->size = yuyv_image->pixels * yuyv_image->pixel_size;
-	dest = (unsigned short *) (yuyv_image->data	= malloc(yuyv_image->size)) ;
+	dest = (unsigned short *) (yuyv_image->data =
+				   malloc (yuyv_image->size));
 	yuyv_image->palette = 0;
 	yuyv_image->palette_size = 0;
 
-	while((count++) < rgb_image->pixels)
+	while ((count++) < rgb_image->pixels) {
-	{
 		pixel_rgb_to_yuyv (rgb_ptr++, &yuyv);
 
 		if ((count & 1) == 0)	/* Was == 0 */
@@ -270,7 +258,8 @@ int image_rgb_to_yuyv (image_t *rgb_image, image_t *yuyv_image)
 	}
 
 #ifdef ENABLE_ASCII_BANNERS
-	printlogo_yuyv (yuyv_image->data, yuyv_image->width, yuyv_image->height);
+	printlogo_yuyv (yuyv_image->data, yuyv_image->width,
+			yuyv_image->height);
 #endif
 	return 0;
 }
@@ -281,13 +270,18 @@ int image_save_header (image_t *image, char *filename, char *varname)
 	char app[256], str[256] = "", def_name[64];
 	int count = image->size, col = 0;
 	unsigned char *dataptr = image->data;
+
 	if (file == NULL)
 		return -1;
 
 	/*  Author information */
-	fprintf(file, "/*\n * Generated by EasyLogo, (C) 2000 by Paolo Scaffardi\n *\n");
+	fprintf (file,
-	fprintf(file, " * To use this, include it and call: easylogo_plot(screen,&%s, width,x,y)\n *\n", varname);
+		 "/*\n * Generated by EasyLogo, (C) 2000 by Paolo Scaffardi\n *\n");
-	fprintf(file, " * Where:\t'screen'\tis the pointer to the frame buffer\n");
+	fprintf (file,
+		 " * To use this, include it and call: easylogo_plot(screen,&%s, width,x,y)\n *\n",
+		 varname);
+	fprintf (file,
+		 " * Where:\t'screen'\tis the pointer to the frame buffer\n");
 	fprintf (file, " *\t\t'width'\tis the screen width\n");
 	fprintf (file, " *\t\t'x'\t\tis the horizontal position\n");
 	fprintf (file, " *\t\t'y'\t\tis the vertical position\n */\n\n");
@@ -297,14 +291,20 @@ int image_save_header (image_t *image, char *filename, char *varname)
 	/*	Macros */
 	strcpy (def_name, varname);
 	StringUpperCase (def_name);
-	fprintf(file, "#define	DEF_%s_WIDTH\t\t%d\n", def_name, image->width);
+	fprintf (file, "#define	DEF_%s_WIDTH\t\t%d\n", def_name,
-	fprintf(file, "#define	DEF_%s_HEIGHT\t\t%d\n", def_name, image->height);
+		 image->width);
-	fprintf(file, "#define	DEF_%s_PIXELS\t\t%d\n", def_name, image->pixels);
+	fprintf (file, "#define	DEF_%s_HEIGHT\t\t%d\n", def_name,
+		 image->height);
+	fprintf (file, "#define	DEF_%s_PIXELS\t\t%d\n", def_name,
+		 image->pixels);
 	fprintf (file, "#define	DEF_%s_BPP\t\t%d\n", def_name, image->bpp);
-	fprintf(file, "#define	DEF_%s_PIXEL_SIZE\t%d\n", def_name, image->pixel_size);
+	fprintf (file, "#define	DEF_%s_PIXEL_SIZE\t%d\n", def_name,
-	fprintf(file, "#define	DEF_%s_SIZE\t\t%d\n\n", def_name, image->size);
+		 image->pixel_size);
+	fprintf (file, "#define	DEF_%s_SIZE\t\t%d\n\n", def_name,
+		 image->size);
 	/*  Declaration */
-	fprintf(file, "unsigned char DEF_%s_DATA[DEF_%s_SIZE] = {\n", def_name, def_name);
+	fprintf (file, "unsigned char DEF_%s_DATA[DEF_%s_SIZE] = {\n",
+		 def_name, def_name);
 
 	/*	Data */
 	while (count)
@@ -355,10 +355,8 @@ int image_save_header (image_t *image, char *filename, char *varname)
 
 int main (int argc, char *argv[])
 {
-    char
+	char inputfile[DEF_FILELEN],
-	inputfile[DEF_FILELEN],
+		outputfile[DEF_FILELEN], varname[DEF_FILELEN];
-	outputfile[DEF_FILELEN],
-	varname[DEF_FILELEN];
 
 	image_t rgb_logo, yuyv_logo;
 
@@ -370,13 +368,11 @@ int main (int argc, char *argv[])
 
 		if (argc > 2)
 			strcpy (varname, argv[2]);
-	else
+		else {
-	{
 			char *dot = strchr (inputfile, '.');
 			int pos = dot - inputfile;
 
-	    if (dot)
+			if (dot) {
-	    {
 				strncpy (varname, inputfile, pos);
 				varname[pos] = 0;
 			}
@@ -384,13 +380,11 @@ int main (int argc, char *argv[])
 
 		if (argc > 3)
 			strcpy (outputfile, argv[3]);
-	else
+		else {
-	{
 			char *dot = strchr (varname, '.');
 			int pos = dot - varname;
 
-	    if (dot)
+			if (dot) {
-	    {
 				char app[DEF_FILELEN];
 
 				strncpy (app, varname, pos);
@@ -418,8 +412,7 @@ int main (int argc, char *argv[])
 	/* Import TGA logo */
 
 	printf ("L");
-    if (image_load_tga (&rgb_logo, inputfile)<0)
+	if (image_load_tga (&rgb_logo, inputfile) < 0) {
-    {
 		printf ("input file not found!\n");
 		exit (1);
 	}