From: dbrownell at B. <dbr...@ma...> - 2009-10-05 11:59:42
|
Author: dbrownell Date: 2009-10-05 10:18:17 +0200 (Mon, 05 Oct 2009) New Revision: 2799 Modified: trunk/src/target/arm926ejs.c Log: Minor cleanup to ARM926 debug entry: - don't needlessly export this function - handle "case 0" debug method-of-entry better (silent by default) The "case 0" is a valid debug entry mode so it doesn't deserve the warning int now gets. But it probably means that OpenOCD confused itself somehow; or that it confused the ARM9EJS target. Modified: trunk/src/target/arm926ejs.c =================================================================== --- trunk/src/target/arm926ejs.c 2009-10-05 08:14:57 UTC (rev 2798) +++ trunk/src/target/arm926ejs.c 2009-10-05 08:18:17 UTC (rev 2799) @@ -277,7 +277,7 @@ return ERROR_OK; } -int arm926ejs_examine_debug_reason(target_t *target) +static int arm926ejs_examine_debug_reason(target_t *target) { armv4_5_common_t *armv4_5 = target->arch_info; arm7_9_common_t *arm7_9 = armv4_5->arch_info; @@ -294,7 +294,11 @@ switch (debug_reason) { - /* case 0: no debug entry */ + case 0: + LOG_DEBUG("no *NEW* debug entry (?missed one?)"); + /* ... since last restart or debug reset ... */ + target->debug_reason = DBG_REASON_DBGRQ; + break; case 1: LOG_DEBUG("breakpoint from EICE unit 0"); target->debug_reason = DBG_REASON_BREAKPOINT; |