here are a set of patches that start a cleanup for MISRA compliance required for automotive. In general while the MISRA rules are debatable, they helped point at code that can be optimized or made more readable.
Additional information on remaining opens can be seen here: https://sonarcloud.io/project/issues?directories=src%2Farch%2Fxtensa%2Csrc%2...
Pierre-Louis Bossart (10): sof: define each identifier in a dedicated statement pipeline: add missing default case pipeline: simplify prepare loop SRC: replace switch by if for readability tone: replace switch by if for readability volume: replace switch by if for readability ssp: replace switch by if for readability ssp: remove unnecessary direction parameter in ssp_stop wait: reduce number of nested breaks ipc: remove unnecessary tests
src/arch/xtensa/timer.c | 9 ++++-- src/audio/dai.c | 17 ++++++---- src/audio/eq_fir.c | 22 ++++++++++--- src/audio/eq_iir.c | 21 +++++++++--- src/audio/host.c | 17 +++++++--- src/audio/iir.c | 6 ++-- src/audio/mixer.c | 15 ++++++--- src/audio/pipeline.c | 26 +++++++-------- src/audio/pipeline_static.c | 5 ++- src/audio/src.c | 38 ++++++++++++++-------- src/audio/src_core.c | 42 ++++++++++++++++++------ src/audio/tone.c | 24 +++++++------- src/audio/volume.c | 69 +++++++++++++++++++++++----------------- src/drivers/dw-dma.c | 15 ++++++--- src/drivers/ssp.c | 24 ++++++++------ src/include/reef/wait.h | 4 +-- src/ipc/byt-ipc.c | 5 +-- src/ipc/dma-copy.c | 14 +++++--- src/ipc/intel-ipc.c | 9 ++++-- src/ipc/ipc.c | 3 +- src/ipc/pmc-ipc.c | 6 ++-- src/lib/alloc.c | 24 ++++++++++---- src/lib/lib.c | 17 +++++++--- src/lib/schedule.c | 13 +++++--- src/lib/work.c | 18 ++++++++--- src/math/trig.c | 8 +++-- src/platform/baytrail/clk.c | 3 +- src/platform/baytrail/platform.c | 18 ++++++++--- src/platform/baytrail/timer.c | 8 +++-- 29 files changed, 334 insertions(+), 166 deletions(-)