feat(uac_host): Add global suspend/resume

[peter-marcisovsky]

Description

This MR adds Suspend/Resume events for UAC class driver as a follow-up for the Global/Suspend resume

Changes

Added Suspend and Resume UAC Host interface events

// For backward compatibility with IDF versions which do not have suspend/resume api
#ifdef USB_HOST_LIB_EVENT_FLAGS_AUTO_SUSPEND
#define UAC_HOST_SUSPEND_RESUME_API_SUPPORTED
#endif

Related

• Blocked by refactor(uac_host): Suspend/Resume public API terminology #363
• Closes Suspend/Resume support for UAC Class driver
• Related feat(usb_host): Global suspend/resume #275

Testing

• Added target tests to test UAC Host reaction to suspend/resume events

---

Checklist

Before submitting a Pull Request, please ensure the following:

• 🚨 This PR does not introduce breaking changes.
• All CI checks (GH Actions) pass.
• Documentation is updated as needed.
• Tests are updated or added as necessary.
• Code is well-commented, especially in complex areas.
• Git history is clean — commits are squashed to the minimum necessary.

[peter-marcisovsky]

Hi @leeebo in this MR I am adding support for global suspend/resume of the root port.

In the current uac driver, there are public API functions like:

/**
 * @brief Resume a UAC stream with same stream configuration
 */
esp_err_t uac_host_device_resume(uac_host_device_handle_t uac_dev_handle);

and

/**
 * @brief Suspend a UAC stream
 *
 * @param[in] uac_dev_handle  UAC device handle
 */
esp_err_t uac_host_device_suspend(uac_host_device_handle_t uac_dev_handle);

And other static functions like uac_host_interface_suspend(), uac_host_interface_resume().

Which are suspending and resuming the UAC stream, not the device itself in terms of power consumption lowering.

My concern here is, that users might get confused with naming of those functions. And might mistakenly call uac_host_device_suspend() with intention to put a UAC device into a suspended state, to lower it's power consumption. But the function only suspends a UAC stream.

Do you thinks we should consider renaming the functions to something like uac_host_device_suspend_stream() and uac_host_device_resume_stream() ? Can you PTAL and advise how to handle this?

[leeebo]

Yes, I think rename to _suspend_stream() and _resume_stream() is a good choice

[peter-marcisovsky]

@leeebo renaming the public api would cause a breaking change for the UAC component.
Should should we consider bumping a Major version when releasing this feature ?

[leeebo]

@peter-marcisovsky Sorry for the late response. I prefer using Minor (v1.4.0) for this new feature, because in my understanding, Major usually refers to a significant update or refactoring, such as the introduction of support for UAC 2.0

[leeebo]

@peter-marcisovsky The implements LGTM! Thanks

[peter-marcisovsky]

Blocked by:

• refactor(uac_host): Suspend/Resume public API terminology #363