uchar env_get_char_spec (int index)
 {
        uchar c;
-       read_dataflash (CFG_ENV_ADDR+index+offsetof(env_t,data),1,&c);
+       read_dataflash(CFG_ENV_ADDR + index + offsetof(env_t,data),
+       1, (char *)&c);
        return (c);
 }
 
 void env_relocate_spec (void)
 {
-       read_dataflash (CFG_ENV_ADDR,CFG_ENV_SIZE,(uchar *)env_ptr);
+       read_dataflash(CFG_ENV_ADDR, CFG_ENV_SIZE, (char *)env_ptr);
 }
 
 int saveenv(void)
 {
-/* env must be copied to do not alter env structure in memory*/
-unsigned char temp[CFG_ENV_SIZE];
-int i;
+       /* env must be copied to do not alter env structure in memory*/
+       unsigned char temp[CFG_ENV_SIZE];
        memcpy(temp, env_ptr, CFG_ENV_SIZE);
-       return write_dataflash (CFG_ENV_ADDR, (unsigned long)temp, CFG_ENV_SIZE);
+       return write_dataflash(CFG_ENV_ADDR, (unsigned long)temp, CFG_ENV_SIZE);
 }
 
 /************************************************************************
                AT91F_DataflashInit();  /* prepare for DATAFLASH read/write */
 
                /* read old CRC */
-               read_dataflash (CFG_ENV_ADDR+offsetof(env_t,crc),sizeof(ulong),&crc);
+               read_dataflash(CFG_ENV_ADDR + offsetof(env_t, crc),
+                       sizeof(ulong), (char *)&crc);
                new = 0;
                len = ENV_SIZE;
                off = offsetof(env_t,data);
                while (len > 0) {
                        int n = (len > sizeof(buf)) ? sizeof(buf) : len;
-                       read_dataflash (CFG_ENV_ADDR+off,n , buf);
+                       read_dataflash(CFG_ENV_ADDR + off, n, (char *)buf);
                        new = crc32 (new, buf, n);
                        len -= n;
                        off += n;