W5500: incomplete MACRAW frames

Question

Hi, In the project I’m working on I wrote support for w5500 MACRAW reading. Frames are passed to the TCP/IP stack (lwIP). At low load, everything works OK, but if I start to send larger data packages (eg I want to update the uC firmware), there are some errors in the data. The problem looks like this: I check if there are data in the RX buffer with the ‘getSn_RX_RSR ()’ function. The function returns the value 1900. OK, so I can read first 2 bytes from buffer - there should be the length of MACRAW, but this is where problem appears: after calculating length of the MAC frame I get value 22,000 bytes. I started to dump all data received from SPI. First dump 1900 bytes. I noticed that at the beginning of buffer are some random bytes – this is why I get 22,000 MAC frame length. Next MACRAW frame starts at byte number 800 and has a length of 1492 bytes. So there is missing 392 bytes (800 + 1492 – 1900). But at the next reading (1000 B), I get the missing 392 bytes of previous frame + half of the next MACRAW frame, and so on. I solved this problem temporarily by analyzing the readed data in my software and sending complete MAC frames to the TCP/IP stack, but this solution is very slow compared to just reading data over SPI. To read data from w5500, I used this library https://github.com/d-a-v/W5500MacRaw with some small modifications. Here are 4 method used to get MACRAW frames. The connection over SPI to w5500 looks like this: Wiznet5500::begin bool Wiznet5500::begin(const uint8_t *mac_address) { if (vspi == NULL) { memcpy(_mac_address, mac_address, 6); gpio_pad_select_gpio(_cs); gpio_set_direction(_cs, GPIO_MODE_INPUT_OUTPUT); wizchip_cs_deselect(); vspi = new SPIClass(VSPI); vspi->begin(); vspi->setClockDivider(spiClk); vspi->setBitOrder(MSBFIRST); vspi->setDataMode(SPI_MODE0); wizchip_sw_reset(); // Use the full 16Kb of RAM for Socket 0 setSn_RXBUF_SIZE(2); setSn_TXBUF_SIZE(16); // Set our local MAC address setSHAR(_mac_address); // Open Socket 0 in MACRaw mode setSn_MR(Sn_MR_MACRAW | Sn_MR_MFEN); uint8_t cfg = getPHYCFGR(); cfg |= PHYCFGR_OPMD; cfg |= PHYCFGR_OPMDC_ALLA; setPHYCFGR(cfg); wizphy_reset(); } setSn_CR(Sn_CR_OPEN); if (getSn_SR() != SOCK_MACRAW) { // Failed to put socket 0 into MACRaw mode printf("[Wiznet5500] Failed to put socket 0 into MACRaw mode "); return false; } // Success printf("[Wiznet5500] Ready "); return true; } Checking if there is data to be read + checking the length of the MACRAW: Wiznet5500::readFrameSize uint16_t Wiznet5500::readFrameSize() { uint16_t tmpBufLen = getSn_RX_RSR(); //no data to read if (tmpBufLen == 0) return 0; uint8_t head[2]; uint16_t data_len = 0; //get 2 data length bytes wizchip_recv_data(head, 2); setSn_CR(Sn_CR_RECV); while (getSn_CR()) ; data_len = head[0]; data_len = (data_len << 8) + head[1]; if (data_len > 1) { data_len -= 2; } else { return 0; } return data_len; } MACRAW reading: Wiznet5500::readFrame + Wiznet5500::readFrameData uint16_t Wiznet5500::readFrame(uint8_t *buffer, uint16_t…

Eugeny · Answer

It would help if you share a SPI data dump and Wireshark log. bilu: // Use the full 16Kb of RAM for Socket 0 setSn_RXBUF_SIZE(2); setSn_TXBUF_SIZE(16); I suspect you wanted to allocate 16K for RX. bilu: //get 2 data length bytes wizchip_recv_data(head, 2); But if tmpBufLen is 1, you will get wrong value. bilu: wizchip_recv_data(head, 2); setSn_CR(Sn_CR_RECV); I did not use MACRAW mode, but I suspect that you should perform differently. Instead of taking frame size only and then performing RECV command, you must flush whole data size indicated by RSR into your working RAM buffer, perform RECV and then parse data in working RAM buffer afterwards.