Fix: Explicitly stop DHCP client before setting static IP

[fcorri]

When configuring a static IP address using void EthernetClass::begin(IPAddress ip, IPAddress dns, IPAddress gateway, IPAddress subnet) the setup fails with the error: [E][NetworkInterface.cpp:527] config(): ETH IP could not be configured! Error: 0x5007: ESP_ERR_ESP_NETIF_DHCP_NOT_STOPPED.

The beginETH() function initializes the Ethernet interface, which (by default) has an active DHCP client. The ESP-IDF network stack (esp-netif) explicitly requires the DHCP client to be stopped before a static IP configuration can be applied. The original code did not perform this step, leading directly to the ESP_ERR_ESP_NETIF_DHCP_NOT_STOPPED error.

As per the official ESP-IDF documentation on esp_netif static IP configuration, this PR fixes the bug by adding a direct call to esp_netif_dhcpc_stop() immediately after beginETH() succeeds and before config() is called.

This ensures the DHCP client is correctly stopped, satisfying the API preconditions and allowing the static IP to be set reliably.

  if (beginETH(mac))
  {
    if (esp_netif_dhcpc_stop(netif()) != ESP_OK)
    {
      log_e("Failed to stop dhcp client");
      return;
    }
    if (config(localIP, gatewayIP, netmask, dnsIP))
    {
      hwStatus = EthernetHardwareFound;
    }
  }

Fixes #7

[JAndrassy]

NetworkInterface.config stops the DHCP client as first

https://github.com/espressif/arduino-esp32/blob/master/libraries/Network/src/NetworkInterface.cpp#L419

[fcorri]

Maybe my fix is not necessary, but there still exist some timing problems. If you put a delay(1) between EthernetClass::beginETH and NetworkInterface::config the static configuration works, otherwise it does not. Have you tested your current code?

[JAndrassy]

I tested it when I published it. It was with an older version of the esp32 platform.
It is a problem of the Network library, that it requires begin before config. and begin doesn't block. s it possible that even if the DHCP client is already stopped that a DHCP response is processed and overwrites the static settings?