[Feature Request] In-band handling of cellular_comm_interface errors

[tpecar-ltek]

The cellular_comm_interface, which handles low level transfers between the modem and the cellular library, has provisions to report errors on receive / transmit (ie. caused by buffer overruns, etc.)

However, the pktio layer currently disregards this status information

FreeRTOS-Cellular-Interface/source/cellular_pktio.c

Lines 590 to 592 in 936d4f7

	( void ) pContext->pCommIntf->recv( pContext->hPktioCommIntf, ( uint8_t * ) pRead,
	bufferEmptyLength,
	CELLULAR_COMM_IF_RECV_TIMEOUT_MS, &bytesRead );

FreeRTOS-Cellular-Interface/source/cellular_pktio.c

Lines 1290 to 1292 in 936d4f7

	( void ) pContext->pCommIntf->send( pContext->hPktioCommIntf,
	( const uint8_t * ) &pContext->pktioSendBuf, newCmdLen,
	CELLULAR_COMM_IF_SEND_TIMEOUT_MS, &sentLen );

It would be beneficial if the errors on the low-level layer could be propagated back to the user API.

[paulbartell]

Hello @tpecar-ltek. I've applied the feature request tag to this request and forwarded it to the cellular library team.

We will need to map from the CellularCommInterfaceError_t type to the CellularError_t type. Feel free to do so and open a pull request if you would like. Otherwise we will prioritize your request and add it to a future release.