Minor style improvements found by cppcheck

[dp111]

Minor style improvements found by cppcheck.

[HiFiPhile]

Thank you for the PR, but we are not interested for these variable name changing, for example buffer is used elsewhere.

[dp111]

Just so I know are you interested in the function prototypes matching the functions ? Would you like to cherry pick the break; to be inside the #if ?

[HiFiPhile]

Just so I know are you interested in the function prototypes matching the functions ?

Could you elaborate, is there any mismatched prototype ?

Would you like to cherry pick the break; to be inside the #if ?

Yes please.

[dp111]

In tusb_fifo.h we currently have :
bool tu_fifo_read(tu_fifo_t* f, void * p_buffer);
in tusb_fifo.c :
bool tu_fifo_read(tu_fifo_t* f, void * buffer)

This is super minor but the PR intended to make the second parameter match.

[HiFiPhile]

In tusb_fifo.h we currently have :
bool tu_fifo_read(tu_fifo_t* f, void * p_buffer);
in tusb_fifo.c :
bool tu_fifo_read(tu_fifo_t* f, void * buffer)

Didn't notice that, please change the prototype to use buffer since it's more used.

Speaking of which the project doesn't have a naming convention yet...