Put functions in the right places to minimize superfluous includes.

Makefile

@@ -253,7 +253,7 @@ MODULES		+= $(foreach sdir,$(LIBRARIES_DIR),$(wildcard $(sdir)/*))
 EXTRA_INCDIR 	= include .
 
 # libraries used in this project, mainly provided by the SDK
-LIBS = c gcc hal phy pp net80211 wpa main lwip_536 crypto
+LIBS = c gcc hal phy pp net80211 wpa main lwip crypto
 
 # compiler flags using during compilation of source files
 CFLAGS	+= -Os -ggdb -std=c99 -Werror -Wpointer-arith -Wundef -Wall -Wl,-EL -fno-inline-functions \

cmd/handlers.c

@@ -18,6 +18,7 @@
 #include <socket.h>
 #endif
 #include <ip_addr.h>
+#include "esp-link/cgi.h"
 
 #include "config.h"
 
@@ -379,6 +380,12 @@ void ICACHE_FLASH_ATTR cmdWifiQuerySSID(CmdPacket *cmd) {
   cmdResponseEnd();
 }
 
+// Start scanning, API interface
+void ICACHE_FLASH_ATTR cmdWifiStartScan(CmdPacket *cmd) {
+  // call a function that belongs in esp-link/cgiwifi.c due to variable access
+  wifiStartScan();
+}
+
 // Command handler for MQTT information
 void ICACHE_FLASH_ATTR cmdMqttGetClientId(CmdPacket *cmd) {
   CmdRequest req;

esp-link/cgiwifi.c

@@ -15,7 +15,6 @@ Cgi/template routines for the /wifi url.
 
 
 #include <esp8266.h>
-#include "cmd.h"
 #include "cgiwifi.h"
 #include "cgi.h"
 #include "status.h"
@@ -228,27 +227,6 @@ static void ICACHE_FLASH_ATTR scanStartCb(void *arg) {
   wifi_station_scan(NULL, wifiScanDoneCb);
 }
 
-// Start scanning, web interface
-static int ICACHE_FLASH_ATTR cgiWiFiStartScan(HttpdConnData *connData) {
-  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
-  jsonHeader(connData, 200);
-
-  // Don't duplicate code, reuse the function below
-  cmdWifiStartScan(0);
-
-  return HTTPD_CGI_DONE;
-}
-
-// Start scanning, API interface
-void ICACHE_FLASH_ATTR cmdWifiStartScan(CmdPacket *cmd) {
-  if (!cgiWifiAps.scanInProgress) {
-    cgiWifiAps.scanInProgress = 1;
-    os_timer_disarm(&scanTimer);
-    os_timer_setfn(&scanTimer, scanStartCb, NULL);
-    os_timer_arm(&scanTimer, 200, 0);
-  }
-}
-
 static int ICACHE_FLASH_ATTR cgiWiFiGetScan(HttpdConnData *connData) {
   if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
   char buff[1460];
@@ -295,6 +273,27 @@ static int ICACHE_FLASH_ATTR cgiWiFiGetScan(HttpdConnData *connData) {
   return HTTPD_CGI_MORE;
 }
 
+// Start scanning, without parameters
+void ICACHE_FLASH_ATTR wifiStartScan() {
+  if (!cgiWifiAps.scanInProgress) {
+    cgiWifiAps.scanInProgress = 1;
+    os_timer_disarm(&scanTimer);
+    os_timer_setfn(&scanTimer, scanStartCb, NULL);
+    os_timer_arm(&scanTimer, 200, 0);
+  }
+}
+
+// Start scanning, web interface
+int ICACHE_FLASH_ATTR cgiWiFiStartScan(HttpdConnData *connData) {
+  if (connData->conn==NULL) return HTTPD_CGI_DONE; // Connection aborted. Clean up.
+  jsonHeader(connData, 200);
+
+  // Don't duplicate code, reuse the function above.
+  wifiStartScan();
+
+  return HTTPD_CGI_DONE;
+}
+
 int ICACHE_FLASH_ATTR cgiWiFiScan(HttpdConnData *connData) {
     if (connData->requestType == HTTPD_METHOD_GET) {
         return cgiWiFiGetScan(connData);

esp-link/cgiwifi.h

@@ -28,7 +28,6 @@ int wifiGetApCount();
 void wifiGetApName(int, char *);
 int wifiSignalStrength(int);
 void connectToNetwork(char *, char *);
-void cmdWifiQuerySSID(CmdPacket *cmd);
-void cmdWifiStartScan(CmdPacket *cmd);
+void wifiStartScan();
 
 #endif

serial/serbridge.c

@@ -9,7 +9,6 @@
 #include "config.h"
 #include "console.h"
 #include "slip.h"
-#include "cmd.h"
 #ifdef SYSLOG
 #include "syslog.h"
 #else

socket/socket.c

@@ -7,7 +7,6 @@
 #include "c_types.h"
 #include "ip_addr.h"
 #include "socket.h"
-#include "cmd.h"
 
 #define SOCK_DBG