W5500 Problems by open and sending UDP

Question

Hi everyone, at the moment I want to develop a solution with the W5500 chip. Im using the wiznet driver API. The loopback server UDP/ TCP works good. Now I implement a new abstraction layer to use the wiznet API in further projects. At the moment I cant post my code, so I try to explain what happens with pseudo code. void main(void) { /* Based on wiznet udp loopback server, Port will open correctly. Alter opening I want to send a UDP broarcast frame in the network (inside the openUdpPort function) */ openUdpPort(3000); sendto(something1); //After opening and sending upd broadcast send someting unicast sendto(something2);//After opening and sending upd broadcast send someting unicast while(1) { ownLoopbackUdp(); //Works stabile and fine (also based on wiznet loopbackserver) } } The whole testing based on check the network with the wireshark. Well… In the “openUdpPort” function, I close and open the socket 7 and configure him to port 3000. After the config (which seems to be ok) I want to send the broadcast. But this dosnt work stable. In debugging mode, I saw, that I´ve got the timeout error by the sendto() function. If this happen “send something1 & 2” dosnt work ( nothing happen ). But the loopbackserver has, whether the broadcast and the two unicast working or not, no failure and work every time very good. When I change the destination ip of the first frame in the openUdpPort function, from 192.168.1.255 (255.255.255.255) to a unicast address e.g. 192.168.1.162, every sendto() function works the most of the time very stable. Have somebody an idea, why I get the timeout failure? And why the sendto function sometimes dont want to work after I open the port? PS.: It seems to be that the chip is configured correctly with IP and so on, because the loopback works everytime very good. AND: If I debug the open function step by step, the broadcast and every other message where send to the network!? ._. I hope somebody can help me! Thanks in advantage, Steve

jim · Answer

Actually, I don’t understand your code well. Why don’t you upload your real code? I will make sure that after your uploading. I don’t know if that helps. Thanks.

steve · Answer

Ok, here is my code: First my Open UDP function: int8_t UDP_OpenPort(uint8_t sn, uint16_t port) { int8_t tmp = 0; int8_t retVal = UDP_OPEN_ERROR; /* Wird nur ausgeführt wenn der Socket nicht bereits geöffnet ist */ if (getSn_SR(sn) != SOCK_UDP) { /* Öffne Socket (Port config)*/ tmp = socket(sn, Sn_MR_UDP, port, 0x00); if (tmp != sn) { /* Signalisiere durch das 0 setzten, dass das öffnen des Ports nicht erfolgreich war. Kann zur Laufzeit als Bedingung genutzt werden */ W5500_RuntimeInformation.wizUdp0 = 0; W5500_RuntimeInformation.wizUdp0Port = 0; } else { /* Übernehme Config in Runtimeinformation, da 'socket' function erfolgreich */ W5500_RuntimeInformation.wizUdp0 = sn; W5500_RuntimeInformation.wizUdp0Port = port; retVal = UDP_OPEN_COMPLETE; } } return retVal; } Next my function where sending UDP frames out: int8_t UDP_Send(ipMessage_t *message) { int8_t tmp = 0; int8_t retVal = UDP_TRANSMISSION_NOT_COMPLETE; if(message != NULL) { switch (getSn_SR(W5500_RuntimeInformation.wizUdp0)) { case SOCK_UDP: tmp = sendto(W5500_RuntimeInformation.wizUdp0, message->payloadBuff, message->numberOfBytes, message->destinationIp, message->destinationPort); if (tmp == message->numberOfBytes) { retVal = UDP_TRASNMISSION_COMPLETE; } else if (tmp < 0) { retVal = UDP_TRANSMISSION_ERROR; } break; case SOCK_CLOSED: /* Wenn SEND-Request ausgeführt wird, der Port jedoch nicht geöffnet ist, wird automatisch ein OpenUDS ausgeführt (Anhand der Runtimeinformation [ethComInterface.c/h])*/ UDP_OpenPort(W5500_RuntimeInformation.wizUdp0, W5500_RuntimeInformation.wizUdp0Port); break; default: /* Error Handler */ retVal = UDP_TRANSMISSION_ERROR; break; } } return retVal; } Here is my init function of the chip: uint8_t ECI_InitComInterface(uint8_t *mac, uint8_t *ip, uint8_t *subnetMask, uint8_t *gateway, uint8_t *dns, bool bootFromFlash) { int8_t retValSetInfo = -1; int8_t retValGetInfo = -1; int8_t tmp; uint8_t memsize[2][8] = { { 2, 2, 2, 2, 2, 2, 2, 2 }, { 2, 2, 2, 2, 2, 2, 2, 2 } }; uint8_t…