search

NabuCasa / silabs-firmware-builder

Silicon Labs firmware builder
42 stars 54 forks source link

Treat build-time warnings as errors #72

Closed puddly closed 3 weeks ago

puddly commented 1 month ago

Enables -Wall -Wextra -Werror and fixes a few warnings and bugs with compilation.

There is a persistent warning with the PTI interface that can only be fixed if PTI is re-enabled verbosely in every config. I think this is a bug, as this component cannot be excluded:

// <<< sl:start pin_tool >>>
// <pti signal=DOUT,(DFRAME),(DCLK)> SL_RAIL_UTIL_PTI
// $[PTI_SL_RAIL_UTIL_PTI]
#ifndef SL_RAIL_UTIL_PTI_PERIPHERAL             
#warning "RAIL PTI peripheral not configured"
#define SL_RAIL_UTIL_PTI_PERIPHERAL              PTI
#endif

Until this is fixed in the Gecko SDK, this warning is explicitly disabled.

puddly commented 3 weeks ago

Looks like there may be some SLC-CLI flakiness. Happened only a single time so far so I'll deal with it if it happens again.