... > +cleanup: > + switch (state) { > + case SBI_SSE_STATE_ENABLED: > + ret = sbi_sse_disable(event_id); > + if (ret.error) { > + sbiret_report_error(&ret, SBI_SUCCESS, "disable event 0x%x", event_id); > + break; > + } > + case SBI_SSE_STATE_REGISTERED: > + sbi_sse_unregister(event_id); > + if (ret.error) > + sbiret_report_error(&ret, SBI_SUCCESS, "unregister event 0x%x", event_id); > + default: I had to add break here to make clang happy (and same for the other two defaults without statements). Thanks, drew