Skip to content

Conversation

@github-actions
Copy link

@github-actions github-actions bot commented Dec 3, 2025

Backport 66cda9d from #3584.

… over friendship

Commit fixes the issue when lpn device relays
broad- and group cast messages received over
friendship queue. This is specification violation,
see Table 3.13: Network layer Network PDU retransmission
requirements. Also, it does not have sense since
sending messages have already happened and lpn
just consumes extra power to retransmit useless data.
In general, lpn device will be more power efficient after
this fix.

Upstream PR #: 100285

Signed-off-by: Aleksandr Khromykh <[email protected]>
(cherry picked from commit b502dec131912d546de6e0e570bcb1aaf7c001e9)
(cherry picked from commit 66cda9d)
@alxelax
Copy link
Contributor

alxelax commented Dec 3, 2025

Hi @rlubos, while downstream PR was open the upstream PR has been merged. At the current moment, CI Commit tag step mandates to change [fromlist] to [fromtree] but this is backport PR. I'm not sure if this is correct to change anything manually in backport PRs. Could you (or anyone) clarify what to do with this?

@rlubos
Copy link
Contributor

rlubos commented Dec 3, 2025

Hi @rlubos, while downstream PR was open the upstream PR has been merged. At the current moment, CI Commit tag step mandates to change [fromlist] to [fromtree] but this is backport PR. I'm not sure if this is correct to change anything manually in backport PRs. Could you (or anyone) clarify what to do with this?

Just ignore it, I'll override the CI when it's time to merge.

@carlescufi carlescufi merged commit b15f3db into ncs-v3.2-branch Dec 3, 2025
1 check failed
@NordicBuilder
Copy link
Contributor

none

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

@alxelax alxelax deleted the backport-3584-to-ncs-v3.2-branch branch December 4, 2025 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants