There's at least 1 simple bug that I found: ring_buffer_get() returns an error code, but it's being ignored in uart_getchar().