On Mon, Nov 17, 2025 at 11:48:11PM +0530, Ayaan Mirza Baig wrote:
The MODE_SWITCH_TIMEOUT_MS constant included a FIXME suggesting the timeout could be reduced once the SVC core supported parallel event processing. Greybus SVC logic is stable today and no such change is planned, and the existing timeout has been used reliably for years. Remove the obsolete FIXME and replace it with a descriptive comment.
No, just leave the FIXME in place.
-/*
- FIXME: Reduce this timeout once svc core handles parallel processing of
- events from the SVC, which are handled sequentially today.
- */
+/* Timeout for mode switching operations, based on current SVC behaviour */ #define MODE_SWITCH_TIMEOUT_MS 10000
Johan