diff options
author | Santosh Shukla <santosh.shukla@linaro.org> | 2014-04-08 16:08:11 +0530 |
---|---|---|
committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2014-04-09 11:46:51 +0400 |
commit | 23e9056418d446496a8c481498d8aea0860c5f6c (patch) | |
tree | e5053e6772a6ecfd61f6430359226e7384b248c3 /test | |
parent | 962ec2bf83a12a77a4e7ba1eb3caab42506994f3 (diff) |
Timer: build fix for arm
source/odp_timer.c: In function ‘odp_timer_cancel_tmo’:wq!
source/odp_timer.c:136:3: error: format ‘%lu’ expects argument of type ‘long unsigned int’,
but argument 8 has type ‘uint64_t’ [-Werror=format=]
ODP_ERR("cancel failed for tim id %d tmo id :%d tick idx %lu\n", id, tmo, tick_idx);
odp_timer_ping.c: In function ‘ping_init’:
odp_timer_ping.c:257:31: error: cast increases required alignment of target type [-Werror=cast-align]
dst_addr.sin_addr.s_addr = *(long *)hname->h_addr;
- Move ODP_ERR outside spinlock
- Replace ODP_ERR with ODP_DBG, as search_and_delete_tmo() func may
fail for valid case like entry in list already deleted by timeout handler
which is a valid case but then odp_err messaging is wrong..(pointed
out by Ola). The invalid case still fall into same loop and may confuse
for now, thus need to identify possible erronoing so to decode error message.
I am leaving this open for now (possible errorno in timer implementation)
..will evolve in phases (TODO)
Signed-off-by: santosh shukla <santosh.shukla@linaro.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/api_test/odp_timer_ping.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/api_test/odp_timer_ping.c b/test/api_test/odp_timer_ping.c index 6c84a56f..8b297d8d 100644 --- a/test/api_test/odp_timer_ping.c +++ b/test/api_test/odp_timer_ping.c @@ -254,7 +254,7 @@ static int ping_init(int count, char *name[]) bzero(&dst_addr, sizeof(dst_addr)); dst_addr.sin_family = hname->h_addrtype; dst_addr.sin_port = 0; - dst_addr.sin_addr.s_addr = *(long *)hname->h_addr; + dst_addr.sin_addr.s_addr = *(long *)(void *)hname->h_addr; } printf("ping to addr %s\n", name[1]); |