This is the mail archive of the ecos-patches@sources.redhat.com 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]

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


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