This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1001864] Data not relocated to RAM during ROMINT startup
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: unassigned at bugs dot ecos dot sourceware dot org
- Date: Tue, 07 Oct 2014 16:54:16 +0000
- Subject: [Bug 1001864] Data not relocated to RAM during ROMINT startup
- Authentication-results: sourceware.org; auth=none
- Auto-submitted: auto-generated
- References: <bug-1001864-777 at http dot bugs dot ecos dot sourceware dot org/>
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.