This is the mail archive of the ecos-bugs@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 1001864] Data not relocated to RAM during ROMINT startup


Please do not reply to this email, use the link below.

http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001864

--- Comment #6 from Ilija Kocho [ÐÐÐÑÐ ÐÐÑÐ] <ilijak@siva.com.mk> ---
Hi Jifl

Thank you for your consideration.

(In reply to comment #5)
> Hi Ilija,
> 
> The specific proposal in comment #1 to use CYG_HAL_STARTUP_ROM doesn't feel
> quite right to me:
> 
> a) it would probably be better for this property to be accessible via CDL,
> not just source. A CDL interface presumably.
> b) It might make the code appear confusing in the case where you do need to
> distinguish "real" ROM startup from ROMINT/FLASH/whatever.

May be confusing indeed. I developed the presented method in order to avoid
changes at architecture level. Now, provided that it is not a taboo - why not?

then for protection of legacy code we can use something like

#if defined(CYG_HAL_STARTUP_ROM) || defined(CYGINT_HAL_...)

> 
> But this is easily dealt with just by using a different name for the
> abstract startup type. It could be CYGINT_HAL_STARTUP_ROM or
> CYGINT_HAL_ROM_STARTUP, but perhaps it might better express what we're after
> here if we called it CYGINT_HAL_STANDALONE_STARTUP.

/ROM/ does not imply /standalone/. FLASH startup from Bug #1001623 is being
started from RedBoot. And it shall implement both RAM and ROM interfaces.

> But i've no strong
> preferences.

Regarding remaining two options, I have no strong preference either, but
CYGINT_HAL_ROM_STARTUP seems slightly better distinguished from
CYG_HAL_STARTUP_ROM than the other one.

Ilija

-- 
You are receiving this mail because:
You are the assignee for the bug.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]