Skip to content

Commit

Permalink
sbus: disable chaining for SetActive and SetInconsistent
Browse files Browse the repository at this point in the history
Even in the unlikely case when multiple of these methods where
called in parallel, we must treat them as separate calls and
handle them individually to avoid situations like:

- SetInconsistent
- SetActive
- SetInconsistent

Here, if chained, it translates to SetInconsistent -> SetActive, which
is wrong. Therefore we can not chain these signals.
  • Loading branch information
pbrezina committed Oct 5, 2023
1 parent d19c5ad commit f6dbac8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/sss_iface/sss_iface.xml
Original file line number Diff line number Diff line change
Expand Up @@ -170,10 +170,10 @@
<annotation name="codegen.Name" value="resp_domain" />
<annotation name="codegen.SyncCaller" value="false" />
<signal name="SetActive">
<arg name="name" type="s" direction="in" key="1" />
<arg name="name" type="s" direction="in" />
</signal>
<signal name="SetInconsistent">
<arg name="name" type="s" direction="in" key="1" />
<arg name="name" type="s" direction="in" />
</signal>
</interface>

Expand Down

0 comments on commit f6dbac8

Please sign in to comment.