This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: PrPMC1100 port
Mark Salter <msalter@redhat.com> writes:
> >>>>> Jonathan Larmour writes:
>
>
> >> + cdl_option CYGSEM_HAL_USE_ROM_MONITOR {
> >> + display "Work with a ROM monitor"
> >> + flavor booldata
> >> + legal_values { "Generic" "GDB_stubs" }
>
> > Is Generic really an option? Should just be bool surely. Again the ixdp425
> > is the same (at least!).
>
> Dunno. This appears in a lot of HALs (ARM/mips/v85x/etc).
The original meaning of CYGSEM_HAL_USE_ROM_MONITOR was that it named
the ROM monitor that was being used. For example the vrc4273 hal still
has:
legal_values { "Generic" "GDB_stubs" "PMON" }
The code was originally intended to contain two sorts of test: ifdefs
on CYGSEM_HAL_USE_ROM_MONITOR for things that apply to all ROM
monitors, and ifdefs on CYGSEM_HAL_USE_ROM_MONITOR_XXXX for stuff
specific to monitor XXXX.
Of course we now only really support one ROM monitor, RedBoot and we
have very little that distinguishes between monitors. But I think we
should keep the flexibility.
--
Nick Garnett eCos Kernel Architect
http://www.ecoscentric.com/ The eCos and RedBoot experts