I’m implementing the interface defined in Demo/Common/include/serial.h. I also see hints in things like FreeRTOS-Plus and other forum posts that a refactoring of FreeRTOS may be underway. If that’s so, please consider:
* Adding documentation to interface declarations so ports know what the function is supposed to do. For example, various ports appear to have chosen to have the return values of xSerialGetChar and xSerialPutChar be either pdFAIL or pdPASS, although the return type would support just about anything, and my initial assumption is it would be like putchar(3).
* Consistently using either xPort or pxPort as the parameter name for the xComPortHandle. I think the pxPort uses are inconsistent with the coding standard, since the value is an instance of the type even though what the type hides is the fact that it’s a pointer to something which might be a structure. But I suppose that’s a grey area.