[ECOS] RedBoot FIS question

[ECOS] RedBoot FIS question

[Joerg Rapka]

If the option CYGOPT_REDBOOT_FIS is not set, then still space of RAM are
occupied for the FIS (fis_work_block, fisdir_size).
In this case, I think so this is not necessary. The RAM workspace of RedBoot
(workspace_end) is also reduced by the size for the FIS.

See code extract from function "do_flash_init" in file
"/packages/redboot/current/src/flash.c":
   ...
    fis_work_block = (unsigned char
*)(workspace_end-FLASH_MIN_WORKSPACE-block_size);
    workspace_end = fis_work_block;
    fisdir_size = block_size;
   ...

A solution could be:
   ...
    #ifdef CYGOPT_REDBOOT_FIS
        fis_work_block = (unsigned char
*)(workspace_end-FLASH_MIN_WORKSPACE-block_size);
        workspace_end = fis_work_block;
        fisdir_size = block_size;
    #else
        workspace_end = (unsigned char
*)(workspace_end-FLASH_MIN_WORKSPACE);
        fis_work_block = NULL;
        fisdir_size = 0;
    #endif // CYGOPT_REDBOOT_FIS
   ...

NOTE: The both FIS-variables "fis_work_block" and "fisdir_size" are not
necessary at all, if option "CYGOPT_REDBOOT_FIS" is not  set.


Best regards,
Joerg Rapka
Duagon GmbH

Re: [ECOS] RedBoot FIS question

[Jonathan Larmour]

Joerg Rapka wrote:
> 
> If the option CYGOPT_REDBOOT_FIS is not set, then still space of RAM are
> occupied for the FIS (fis_work_block, fisdir_size).
> In this case, I think so this is not necessary. The RAM workspace of RedBoot
> (workspace_end) is also reduced by the size for the FIS.

I've done something similar in our sources for this. You'll see it in the
next update.

Jifl
-- 
Red Hat, Rustat House, Clifton Road, Cambridge, UK. Tel: +44 (1223) 271062
Maybe this world is another planet's Hell -Aldous Huxley || Opinions==mine