Nick Garnett wrote:
Simon Kallweit <[email protected]> writes:
Nick Garnett wrote:
The "#if 0" is an oversight on my part. When debugging via JTAG I was
having difficulties breaking in to an idle system. Stopping it using
WFI solved this. The #else part is also a consequence of this, so that
code should really be:
Well that's actually the reason for me to propose a
CYGHWR_HAL_CORTEXM_IDLE_SLEEP configuration option. During debugging,
wfi is rather troublesome, so I thought it might be a good idea to
make it configurable. Or is this too much cruft?
I think it might be too much cruft.
I don't know how other people work but I tend to like
enabling/disabling debug stuff by editing the source rather than
messing with the configuration. The latter is often more complicated
and it takes a lot longer to recompile everything. CVS usually reminds
me to undo the change before checking in -- except in this case where
there was no original to diff against, and I forgot about this edit.
Well you're probably right. So let's just clean things up then :)
Simon
--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss
