Skip to content

Commit 00c11d0

Browse files
committed
[toup] revert zephyr change for os_time_t to origin
Hostap define os_time_t as long but zephyr add own definition to long long and change hostap original code to fix warnings. Thus when DPP enabled, unfixed warning will pop out. We do not want to change so many original hostap code. So it is better to remain original os_time_t definition as long. Fix it by revert os_time_t own definition for zephyr and related printf format change also. Signed-off-by: Fengming Ye <[email protected]>
1 parent 4600ba3 commit 00c11d0

File tree

5 files changed

+6
-10
lines changed

5 files changed

+6
-10
lines changed

src/utils/os.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,7 @@
99
#ifndef OS_H
1010
#define OS_H
1111

12-
#if defined(__ZEPHYR__)
13-
typedef long long os_time_t;
14-
#else
1512
typedef long os_time_t;
16-
#endif
1713

1814
/**
1915
* os_sleep - Sleep (sec, usec)

wpa_supplicant/ctrl_iface.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8618,7 +8618,7 @@ static int wpas_ctrl_radio_work_show(struct wpa_supplicant *wpa_s,
86188618
int ret;
86198619

86208620
os_reltime_sub(&now, &work->time, &diff);
8621-
ret = os_snprintf(pos, end - pos, "%s@%s:%u:%u:%lld.%06lld\n",
8621+
ret = os_snprintf(pos, end - pos, "%s@%s:%u:%u:%ld.%06ld\n",
86228622
work->type, work->wpa_s->ifname, work->freq,
86238623
work->started, diff.sec, diff.usec);
86248624
if (os_snprintf_error(end - pos, ret))

wpa_supplicant/events.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5043,7 +5043,7 @@ void wpa_supplicant_event(void *ctx, enum wpa_event_type event,
50435043
os_get_reltime(&wpa_s->scan_start_time);
50445044
os_reltime_sub(&wpa_s->scan_start_time,
50455045
&wpa_s->scan_trigger_time, &diff);
5046-
wpa_dbg(wpa_s, MSG_DEBUG, "Own scan request started a scan in %lld.%06lld seconds",
5046+
wpa_dbg(wpa_s, MSG_DEBUG, "Own scan request started a scan in %ld.%06ld seconds",
50475047
diff.sec, diff.usec);
50485048
wpa_s->own_scan_requested = 0;
50495049
wpa_s->own_scan_running = 1;
@@ -5078,7 +5078,7 @@ void wpa_supplicant_event(void *ctx, enum wpa_event_type event,
50785078
os_reltime_sub(&now, &wpa_s->scan_start_time, &diff);
50795079
wpa_s->scan_start_time.sec = 0;
50805080
wpa_s->scan_start_time.usec = 0;
5081-
wpa_dbg(wpa_s, MSG_DEBUG, "Scan completed in %lld.%06lld seconds",
5081+
wpa_dbg(wpa_s, MSG_DEBUG, "Scan completed in %ld.%06ld seconds",
50825082
diff.sec, diff.usec);
50835083
}
50845084
if (wpa_supplicant_event_scan_results(wpa_s, data))

wpa_supplicant/wnm_sta.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -766,7 +766,7 @@ compare_scan_neighbor_results(struct wpa_supplicant *wpa_s, os_time_t age_secs,
766766
os_reltime_expired(&now, &target->last_update,
767767
age_secs)) {
768768
wpa_printf(MSG_DEBUG,
769-
"Candidate BSS is more than %lld seconds old",
769+
"Candidate BSS is more than %ld seconds old",
770770
age_secs);
771771
continue;
772772
}

wpa_supplicant/wpa_supplicant.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6251,7 +6251,7 @@ static void radio_start_next_work(void *eloop_ctx, void *timeout_ctx)
62516251
os_get_reltime(&now);
62526252
os_reltime_sub(&now, &work->time, &diff);
62536253
wpa_dbg(wpa_s, MSG_DEBUG,
6254-
"Starting radio work '%s'@%p after %lld.%06lld second wait",
6254+
"Starting radio work '%s'@%p after %ld.%06ld second wait",
62556255
work->type, work, diff.sec, diff.usec);
62566256
work->started = 1;
62576257
work->time = now;
@@ -6449,7 +6449,7 @@ void radio_work_done(struct wpa_radio_work *work)
64496449

64506450
os_get_reltime(&now);
64516451
os_reltime_sub(&now, &work->time, &diff);
6452-
wpa_dbg(wpa_s, MSG_DEBUG, "Radio work '%s'@%p %s in %lld.%06lld seconds",
6452+
wpa_dbg(wpa_s, MSG_DEBUG, "Radio work '%s'@%p %s in %ld.%06ld seconds",
64536453
work->type, work, started ? "done" : "canceled",
64546454
diff.sec, diff.usec);
64556455
radio_work_free(work);

0 commit comments

Comments
 (0)