From 4c1c62a453236f8636140564893a2ef66da102b0 Mon Sep 17 00:00:00 2001
From: Kedar Sovani <kedars@gmail.com>
Date: Thu, 17 Aug 2017 09:41:34 +0530
Subject: [PATCH] wifi: refactor code to match ethernet's init function
 structure

---
 components/esp32/event_default_handlers.c    |  3 +--
 components/esp32/include/esp_event.h         |  6 +++++
 components/esp32/include/esp_wifi_internal.h |  6 +++++
 components/esp32/wifi_init.c                 | 23 ++++++++++++++++++++
 4 files changed, 36 insertions(+), 2 deletions(-)
 create mode 100644 components/esp32/wifi_init.c

diff --git a/components/esp32/event_default_handlers.c b/components/esp32/event_default_handlers.c
index 2f671f69f6..75ef164651 100644
--- a/components/esp32/event_default_handlers.c
+++ b/components/esp32/event_default_handlers.c
@@ -374,7 +374,7 @@ esp_err_t esp_event_process_default(system_event_t *event)
     return ESP_OK;
 }
 
-esp_err_t esp_wifi_init(wifi_init_config_t *config)
+void esp_event_set_default_wifi_handlers()
 {
      default_event_handlers[SYSTEM_EVENT_STA_START]        = system_event_sta_start_handle_default;
      default_event_handlers[SYSTEM_EVENT_STA_STOP]         = system_event_sta_stop_handle_default;
@@ -385,7 +385,6 @@ esp_err_t esp_wifi_init(wifi_init_config_t *config)
      default_event_handlers[SYSTEM_EVENT_AP_STOP]          = system_event_ap_stop_handle_default;
 
      esp_register_shutdown_handler((shutdown_handler_t)esp_wifi_stop);
-     return esp_wifi_init_internal(config);
 }
 
 void esp_event_set_default_eth_handlers()
diff --git a/components/esp32/include/esp_event.h b/components/esp32/include/esp_event.h
index 3000e50047..5cb60dba63 100644
--- a/components/esp32/include/esp_event.h
+++ b/components/esp32/include/esp_event.h
@@ -165,6 +165,12 @@ esp_err_t esp_event_process_default(system_event_t *event);
   */
 void esp_event_set_default_eth_handlers();
 
+/**
+  * @brief  Install default event handlers for Wi-Fi interfaces (station and AP)
+  *
+  */
+void esp_event_set_default_wifi_handlers();
+
 #ifdef __cplusplus
 }
 #endif
diff --git a/components/esp32/include/esp_wifi_internal.h b/components/esp32/include/esp_wifi_internal.h
index e86d60f7fa..6627636df0 100644
--- a/components/esp32/include/esp_wifi_internal.h
+++ b/components/esp32/include/esp_wifi_internal.h
@@ -45,6 +45,12 @@ extern "C" {
  *     Alloc resource for WiFi driver, such as WiFi control structure, RX/TX buffer,
  *     WiFi NVS structure among others.
  *
+ * For the most part, you need not call this function directly. It gets called
+ * from esp_wifi_init().
+ *
+ * This function may be called, if you only need to initialize the Wi-Fi driver
+ * without having to use the network stack on top.
+ *
  * @param  config provide WiFi init configuration
  *
  * @return
diff --git a/components/esp32/wifi_init.c b/components/esp32/wifi_init.c
new file mode 100644
index 0000000000..dd155d8f1a
--- /dev/null
+++ b/components/esp32/wifi_init.c
@@ -0,0 +1,23 @@
+// Copyright 2015-2016 Espressif Systems (Shanghai) PTE LTD
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include <esp_event.h>
+#include <esp_wifi.h>
+#include "esp_wifi_internal.h"
+
+esp_err_t esp_wifi_init(wifi_init_config_t *config)
+{
+     esp_event_set_default_wifi_handlers();
+     return esp_wifi_init_internal(config);
+}