Skip to content

Conversation

@wopu-ot
Copy link
Contributor

@wopu-ot wopu-ot commented Nov 11, 2019

Host checks whether an update is needed out of loop to look for
peer. This enables skipping looking for the peer if no update is
needed in the first place. No functional change intended.

Signed-off-by: Wolfgang Puffitsch [email protected]

@wopu-ot wopu-ot requested review from Vudentz and fnde-ot November 11, 2019 16:29
@wopu-ot wopu-ot added the Enhancement Changes/Updates/Additions to existing features label Nov 11, 2019
Check peer address only if active.

Signed-off-by: Wolfgang Puffitsch <[email protected]>
@wopu-ot wopu-ot force-pushed the ccc_update_optimize branch from 0fe4573 to 50e1cee Compare November 13, 2019 17:05
Copy link
Contributor

@Vudentz Vudentz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@fnde-ot fnde-ot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM too!

@jhedberg jhedberg changed the title Bluetooth: host: Hoist checks for updating CCC out of loop Bluetooth: host: Host checks for updating CCC out of loop Nov 14, 2019
@wopu-ot wopu-ot changed the title Bluetooth: host: Host checks for updating CCC out of loop Bluetooth: host: Refactor checks in update_ccc for efficiency Nov 14, 2019
@aescolar aescolar requested a review from joerchan November 14, 2019 08:03
@aescolar aescolar added this to the v2.2.0 milestone Nov 14, 2019
@jhedberg jhedberg closed this Dec 12, 2019
@jhedberg jhedberg reopened this Dec 12, 2019
@jhedberg jhedberg merged commit b0207eb into zephyrproject-rtos:master Dec 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Bluetooth Enhancement Changes/Updates/Additions to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants