Hey Joe,
It would be possible to completely remove the EthernetDHCP and EthernetDNS library requirement, I think, and with the changes I made, I’ve all but removed them.
Some variables need to be defined that are used, that are expected to be initialised by the inclusion of the above libraries.
After my above post about the “dirty” changes I made, I made some more changes, reprogrammed the device, and realised I had it checking the DHCP lease constantly.
I subsequently made some more changes, using millis() to only check the lease every hour, which could probably be increased more.
I made some large changes to the DHCPcheck function, replacing all the switch/case statements with the return codes from the new version.
I didn’t really want to post my code, since it’s pretty ugly, but in case there’s someone else with the same issue, I want to give them something to work with, so here goes..
// start Ethernet
//EthernetDHCP.begin(mac, true); // start ethernet DHCP in non-blocking polling mode
if (Ethernet.begin(mac) == 1) { // start ethernet DHCP in non-blocking polling mode
Serial.println("DHCP discovery success");
Serial.print("ip: ");
Serial.println(Ethernet.localIP());
ipState = DhcpStateLeased;
}
else {
while (Ethernet.begin(mac) == 0){
Serial.println("DHCP discovery failed");
delay(5000); // wait 5 seconds
}
}
void loop() // main loop of the program
{
moistureCheck(); // check to see if moisture levels require Twittering out
wateringCheck(); // check to see if a watering event has occurred to report it
buttonCheck(); // check to see if the debugging button is pressed
analogWrite(COMMLED,0); // douse comm light if it was on
if (millis() % 3600000 == 0){
// an hour has passed
delay(2000);
dhcpCheck(); // check and update DHCP connection
}
if (millis() % 60000 == 0 && ipState != DhcpStateLeased && ipState != DhcpStateRenewing) {
blinkLED(COMMLED,1,30); // quick blnk of COMM led if there's no ip address
}
}
// check and attempt to create a DHCP leased IP address
void dhcpCheck() {
//DhcpState prevState = ipState; // record the current state
//ipState = EthernetDHCP.poll(); // poll for an updated state
Serial.println("DHCP update..");
int State = Ethernet.maintain(); // poll for an updated state
//if (prevState != ipState) { // if this is a new state then report it
//switch (ipState) {
switch (State) {
//case DhcpStateDiscovering:
//Serial.println("DHCP disc");
//break;
//case DhcpStateRequesting:
//Serial.println("DHCP req");
//break;
//case DhcpStateRenewing:
//Serial.println("DHCP renew");
//break;
//case DhcpStateLeased:
case '0':
{
// 0: nothing happened
Serial.println("DHCP: nothing happened");
break;
}
case '1':
{
// 1: renew failed
Serial.println("DHCP: renew failed");
break;
}
case '2':
{
// 2: renew success
Serial.println("DHCP: renewed OK!");
// We have a DHCP lease, so print the info
Serial.print("ip: ");
Serial.println(Ethernet.localIP());
ipState = DhcpStateLeased;
//const byte* ipAddr = EthernetDHCP.ipAddress();
//const byte* gatewayAddr = EthernetDHCP.gatewayIpAddress();
//const byte* dnsAddr = EthernetDHCP.dnsIpAddress();
//Serial.print("ip: ");
//Serial.println(ip_to_str(ipAddr));
//Serial.print("gw: ");
//Serial.println(ip_to_str(gatewayAddr));
//Serial.print("dns: ");
//Serial.println(ip_to_str(dnsAddr));
break;
}
case '3':
{
// 3: rebind fail
Serial.println("DHCP: rebind failed");
break;
}
case '4':
{
// 4: rebind success
Serial.println("DHCP: rebind success");
// We have a DHCP lease, so print the info
Serial.print("ip: ");
Serial.println(Ethernet.localIP());
ipState = DhcpStateLeased;
break;
}
//}
}
}
