Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ensure exclusive access to radios per editorset #695

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Ensure exclusive access to radios per editorset #695

wants to merge 1 commit into from
+51 −39

Commits on Jul 1, 2023

  1. Ensure exclusive access to radios per editorset 

    A ChirpLiveEditorSet represents a conversation with a single radio
endpoint, but may involve multiple independent event streams. We
really don't need multiple threads and queues for talking to a single
radio, however radios with multiple sub-devices will end up with
different Radio classes by nature. So, instead of collapsing editors
into a single thread, we can just make them share a lock, established
at the EditorSet level, which is what this patch does.

This also means the other drivers (that I know of) that do their
own locking (kenwood_live, kenwood_d7, because settings) and ic9x
(because multiple sub-devices) can simplify and rely on this being
handled in the UI.

Related to #10683
    @kk7ds
    kk7ds committed Jul 1, 2023
    Configuration menu
    Copy the full SHA
    d8b7f22 View commit details
    Browse the repository at this point in the history