I believe this patch addresses the issues you raised.
--
Øyvind Harboe
US toll free 1-866-980-3434 / International +47 51 63 25 00
http://www.zylin.com/zy1000.html
ARM7 ARM9 ARM11 XScale Cortex
JTAG debugger and flash programmer
From c8b8a34bb5e98660c4ce3683fb64af31b494d55c Mon Sep 17 00:00:00 2001
From: =?utf-8?q?=C3=98yvind=20Harboe?= <oyvind.har...@zylin.com>
Date: Tue, 15 Dec 2009 14:57:38 +0100
Subject: [PATCH] ecos: crisper implementation of timeval_ms()
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
A crisper/faster implementation under eCos that makes profiling a
tad easier.
Signed-off-by: Øyvind Harboe <oyvind.har...@zylin.com>
---
src/helper/Makefile.am | 4 +-
src/helper/time_support.c | 11 ---------
src/helper/time_support_common.c | 42 +++++++++++++++++++++++++++++++++++++
src/helper/time_support_ecos.c | 43 ++++++++++++++++++++++++++++++++++++++
4 files changed, 87 insertions(+), 13 deletions(-)
create mode 100644 src/helper/time_support_common.c
create mode 100644 src/helper/time_support_ecos.c
diff --git a/src/helper/Makefile.am b/src/helper/Makefile.am
index 244ada7..3ec4f31 100644
--- a/src/helper/Makefile.am
+++ b/src/helper/Makefile.am
@@ -7,9 +7,9 @@ METASOURCES = AUTO
noinst_LTLIBRARIES = libhelper.la
if ECOSBOARD
-CONFIGFILES =
+CONFIGFILES = time_support_ecos.c
else
-CONFIGFILES = options.c jim.c jim-eventloop.c
+CONFIGFILES = options.c jim.c jim-eventloop.c time_support_common.c
endif
diff --git a/src/helper/time_support.c b/src/helper/time_support.c
index 693528f..1934315 100644
--- a/src/helper/time_support.c
+++ b/src/helper/time_support.c
@@ -29,7 +29,6 @@
#include "time_support.h"
-
/* calculate difference between two struct timeval values */
int timeval_subtract(struct timeval *result, struct timeval *x, struct timeval *y)
{
@@ -73,16 +72,6 @@ int timeval_add_time(struct timeval *result, long sec, long usec)
return 0;
}
-int64_t timeval_ms()
-{
- struct timeval now;
- int retval = gettimeofday(&now, NULL);
- if (retval < 0)
- return retval;
- return (int64_t)now.tv_sec * 1000 + now.tv_usec / 1000;
-}
-
-
int duration_start(struct duration *duration)
{
return gettimeofday(&duration->start, NULL);
diff --git a/src/helper/time_support_common.c b/src/helper/time_support_common.c
new file mode 100644
index 0000000..1ad3676
--- /dev/null
+++ b/src/helper/time_support_common.c
@@ -0,0 +1,42 @@
+/***************************************************************************
+ * Copyright (C) 2006 by Dominic Rath *
+ * dominic.r...@gmx.de *
+ * *
+ * Copyright (C) 2007,2008 Øyvind Harboe *
+ * oyvind.har...@zylin.com *
+ * *
+ * Copyright (C) 2008 by Spencer Oliver *
+ * s...@spen-soft.co.uk *
+ * *
+ * This program is free software; you can redistribute it and/or modify *
+ * it under the terms of the GNU General Public License as published by *
+ * the Free Software Foundation; either version 2 of the License, or *
+ * (at your option) any later version. *
+ * *
+ * This program is distributed in the hope that it will be useful, *
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of *
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
+ * GNU General Public License for more details. *
+ * *
+ * You should have received a copy of the GNU General Public License *
+ * along with this program; if not, write to the *
+ * Free Software Foundation, Inc., *
+ * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *
+ ***************************************************************************/
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include "time_support.h"
+
+/* simple and low overhead fetching of ms counter. Use only
+ * the difference between ms counters returned from this fn.
+ */
+int64_t timeval_ms()
+{
+ struct timeval now;
+ int retval = gettimeofday(&now, NULL);
+ if (retval < 0)
+ return retval;
+ return (int64_t)now.tv_sec * 1000 + now.tv_usec / 1000;
+}
diff --git a/src/helper/time_support_ecos.c b/src/helper/time_support_ecos.c
new file mode 100644
index 0000000..b0911fc
--- /dev/null
+++ b/src/helper/time_support_ecos.c
@@ -0,0 +1,43 @@
+/***************************************************************************
+ * Copyright (C) 2006 by Dominic Rath *
+ * dominic.r...@gmx.de *
+ * *
+ * Copyright (C) 2007,2008 Øyvind Harboe *
+ * oyvind.har...@zylin.com *
+ * *
+ * Copyright (C) 2008 by Spencer Oliver *
+ * s...@spen-soft.co.uk *
+ * *
+ * This program is free software; you can redistribute it and/or modify *
+ * it under the terms of the GNU General Public License as published by *
+ * the Free Software Foundation; either version 2 of the License, or *
+ * (at your option) any later version. *
+ * *
+ * This program is distributed in the hope that it will be useful, *
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of *
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
+ * GNU General Public License for more details. *
+ * *
+ * You should have received a copy of the GNU General Public License *
+ * along with this program; if not, write to the *
+ * Free Software Foundation, Inc., *
+ * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *
+ ***************************************************************************/
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include "time_support.h"
+
+#include <cyg/kernel/kapi.h>
+
+int64_t timeval_ms()
+{
+ /* Faster/less noisy implementation of getting ms when
+ * profiling
+ */
+ static const int ms_per_tick =
+ (CYGNUM_HAL_RTC_NUMERATOR / CYGNUM_HAL_RTC_DENOMINATOR) / 1000000;
+ cyg_tick_count_t cur_time = cyg_current_time();
+ return ((int)cur_time) * ms_per_tick;
+}
--
1.6.3.3
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development
