-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Applications: nrf_desktop: Add subscriber event #13451
Applications: nrf_desktop: Add subscriber event #13451
Conversation
Test specificationCI/Jenkins/NRF
CI/Jenkins/integration
Detailed information of selected test modules Note: This message is automatically posted and updated by the CI |
73b51b5
to
653f75a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add release note for application, please (changelog)
653f75a
to
8711087
Compare
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
8711087
to
eacf90a
Compare
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
@@ -823,7 +823,9 @@ nRF Desktop event propagation | |||
+-----------------------------------------------+-----------------------------------+ | | | | |||
| :ref:`nrf_desktop_motion` | ``motion_event`` | | | | | |||
+-----------------------------------------------+-----------------------------------+ | | | | |||
| :ref:`nrf_desktop_usb_state` | ``usb_hid_event`` | | | | | |||
| :ref:`nrf_desktop_hids` | ``hid_report_subscriber_event`` | | | | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please align with the updated arrays that were generated by me locally using a script (to keep consistency): https://github.com/MarekPieta/fw-nrfconnect-nrf/tree/kaga_new_arrays
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you updated the arrays ?
10ed9a5
to
0d33db3
Compare
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
uint8_t report_max; /**< Maximum number of reports with different ID, which can be | ||
* processed. | ||
*/ | ||
} params; /**< Subscriber parameters. Only needed when a subscriber is connecting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if it wouldn't be better to fill the parameter in both cases or zero them (to avoid submitting event with garbage data)
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
@@ -823,7 +823,9 @@ nRF Desktop event propagation | |||
+-----------------------------------------------+-----------------------------------+ | | | | |||
| :ref:`nrf_desktop_motion` | ``motion_event`` | | | | | |||
+-----------------------------------------------+-----------------------------------+ | | | | |||
| :ref:`nrf_desktop_usb_state` | ``usb_hid_event`` | | | | | |||
| :ref:`nrf_desktop_hids` | ``hid_report_subscriber_event`` | | | | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you updated the arrays ?
This adds HIDs subscriber event which allows different transports with configurable prority and pipeline size subscibe to the HIDs reports. Jira: NCSDK-25088 Signed-off-by: Kamil Gawor <[email protected]>
0d33db3
to
7c91ad6
Compare
Update information about events generated or processed by the given nrf_desktop modules. This updates also the application documentation. Jira: NCSDK-24644 Signed-off-by: Kamil Gawor <[email protected]>
Update changelog with new HID subscriber event in nRF Desktop application. Jira: NCSDK-24644 Signed-off-by: Kamil Gawor <[email protected]>
7c91ad6
to
7eb98df
Compare
This PR adds new HID subscriber events which allows to connect different transports for HIDs report data.