diff options
author | Matt Ranostay <matt.ranostay@konsulko.com> | 2017-12-18 18:48:10 -0800 |
---|---|---|
committer | Walt Miner <walt@linux.com> | 2017-12-19 07:34:02 +0000 |
commit | e12b97e7622a9cb1c7b76b777e38758ddcfcc7c2 (patch) | |
tree | f935bf8aff3f040cc09f0da98f872070e420fd10 | |
parent | b502af811a782bdda3a6c56567fdc2c7f7b44609 (diff) |
binding: wifi: fix logic check and reduce error messageseel_5.1.0eel_5.0.3eel_5.0.2eel_5.0.1eel_5.0.0eel_4.99.5eel/5.1.0eel/5.0.3eel/5.0.2eel/5.0.1eel/5.0.0eel/4.99.55.1.05.0.35.0.25.0.15.0.04.99.5eel
Don't attempt to keep checking wifi state if there isn't wifi
card detected by Connman on startup.
Also check the error state of setHMIStatus() to deduce if
another attempt is required.
Bug-AGL: SPEC-1201
Change-Id: Ica9dc3db62ad1b41a86b365edae0bf4e3447db16
Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
(cherry picked from commit 4c41e635d3006ee847079dd28d8333a58ac8e71f)
-rw-r--r-- | binding-wifi/wifi-api.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/binding-wifi/wifi-api.c b/binding-wifi/wifi-api.c index 45341e3..2117d6d 100644 --- a/binding-wifi/wifi-api.c +++ b/binding-wifi/wifi-api.c @@ -678,13 +678,13 @@ gboolean initial_homescreen(gpointer ptr) if (ret < 0) { g_object_unref(status); - return TRUE; + return FALSE; } error = setHMIStatus(status->connected == 1 ? BAR_FULL : BAR_NO); g_error_free(error); - return ret != NULL; + return error != NULL; } static void *thread_func() |