Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility issue with ESP8266 2.1.0 version vs 2.0.0 wait for 2.2.0 #65

Closed
luc-github opened this issue Jan 10, 2016 · 17 comments
Closed

Comments

@luc-github
Copy link
Owner

Travis failed:

/tmp/build5286045909151746881.tmp/wifi.cpp: In member function 'bool WIFI_CONFIG::Setup()':
/tmp/build5286045909151746881.tmp/wifi.cpp:154:25: error: 'sleep_type' was not declared in this scope
   wifi_set_sleep_type ((sleep_type)bflag);
                         ^
/tmp/build5286045909151746881.tmp/wifi.cpp:175:26: error: 'phy_mode' was not declared in this scope
       wifi_set_phy_mode((phy_mode)bflag);
                          ^
/tmp/build5286045909151746881.tmp/wifi.cpp:205:22: error: 'phy_mode' was not declared in this scope
   wifi_set_phy_mode((phy_mode)bflag);
                      ^
Error compiling.
Failed to build esp8266/esp8266.ino

Some enum definition have been moved to another place ?

@luc-github luc-github added the bug label Jan 10, 2016
@luc-github
Copy link
Owner Author

seems type has been changed to
phy_mode_t
and sleep_type_t

@luc-github
Copy link
Owner Author

Need to wait git version is moved to stable version because doing modification now will make incompatible with current stable version or need to add some define which will not be necessary when git become stable

So fix is on hold until these 2 enum are in stable version : 2.1 or more

@luc-github
Copy link
Owner Author

2.1.0 is now out - it is time to try it http://arduino.esp8266.com/versions/2.1.0/package_esp8266com_index.json

@luc-github
Copy link
Owner Author

with 2.1.0 when safe mode is starting the module crash to exception 2
Need to investigate

@luc-github
Copy link
Owner Author

exception 2 also after reset EEPROM and starting as Access point

@luc-github luc-github changed the title Compatibility issue with ESP8266 Git version Compatibility issue with ESP8266 2.1.0 version vs 2.0.0 Feb 27, 2016
@luc-github
Copy link
Owner Author

disabling all feature allow to start AP - will check which feature(s) generate the crash

@luc-github
Copy link
Owner Author

crash at startup is linked to
TCP_IP_DATA_FEATURE, if only this one is disabled, module seems fine

@luc-github
Copy link
Owner Author

doing a simple sketch validate the possibility of having 2 ports is still working
so should be something in implementation code

#include <ESP8266WiFi.h>
#include <WiFiClient.h>
#include <ESP8266WebServer.h>
#include <ESP8266mDNS.h>

const char* ssid = "NETGEAR_2GEXT_OFFICE";
const char* password = "BUREAUCHAMBRE";

ESP8266WebServer server(80);
ESP8266WebServer server2(8888);

void handleRoot() {
  server.send(200, "text/plain", "hello from esp8266!");
}

void handleRoot2() {
  server2.send(200, "text/plain", "hello from esp8266!");
}

void handleNotFound(){
  String message = "File Not Found\n\n";
  message += "URI: ";
  message += server.uri();
  message += "\nMethod: ";
  message += (server.method() == HTTP_GET)?"GET":"POST";
  message += "\nArguments: ";
  message += server.args();
  message += "\n";
  for (uint8_t i=0; i<server.args(); i++){
    message += " " + server.argName(i) + ": " + server.arg(i) + "\n";
  }
  server.send(404, "text/plain", message);
}

void setup(void){
  Serial.begin(115200);
  WiFi.begin(ssid, password);
  Serial.println("");

  // Wait for connection
  while (WiFi.status() != WL_CONNECTED) {
    delay(500);
    Serial.print(".");
  }
  Serial.println("");
  Serial.print("Connected to ");
  Serial.println(ssid);
  Serial.print("IP address: ");
  Serial.println(WiFi.localIP());

  if (MDNS.begin("esp8266")) {
    Serial.println("MDNS responder started");
  }

  server.on("/", handleRoot);
  server2.on("/", handleRoot2);

  server.onNotFound(handleNotFound);

  server.begin();
  server2.begin();
  Serial.println("HTTP server started");
}

void loop(void){
  server.handleClient();
  server2.handleClient();
}

@luc-github
Copy link
Owner Author

Issue looks linked to presence of

data_server->setNoDelay(true);

Commenting it and module seems fine

Need long run test - additionally reaction time looks slower than with 2.0.0 but need more tests to confirm

@luc-github
Copy link
Owner Author

travis - read me are up to date
Still need to review all functions one by one to confirm it is Ok and no side effect, like on web update for example

@luc-github
Copy link
Owner Author

exception(2) issue is reported : esp8266/Arduino#1695

@luc-github
Copy link
Owner Author

seems module is definitly slow and now new issue appeared #85

@luc-github
Copy link
Owner Author

the setNoDelay issue is fixed in git so will be part of 2.2.0

@luc-github
Copy link
Owner Author

all fixes have been implemented - anyway 2.1.0 must not used due to critcal bugs, performances issues and so usability issue
2.2.0 should be the next stable release

@luc-github
Copy link
Owner Author

I have cleared thread to keep focus
After 3 days running non stop on monitoring printer page, not even 1 crash on nodeMCU, so now it is solid rock as 2.0.0 with current git version, tested all features and looks ok so far
will start long test run on ESP12 on my printer today to confirm status

@luc-github
Copy link
Owner Author

after 24h monitoring printer on ESP12E and even monitoring a 20min print - still no crash so I can confirm it fine until new test on final 2.2.0

@luc-github luc-github changed the title Compatibility issue with ESP8266 2.1.0 version vs 2.0.0 Compatibility issue with ESP8266 2.1.0 version vs 2.0.0 wait for 2.2.0 Apr 12, 2016
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant