The Wayback Machine - https://web.archive.org/web/20220529031111/https://github.com/zephyrproject-rtos/zephyr/issues/40375
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.

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

Bluetooth: Controller: Move BT_EVT_MASK_LE_META_EVENT check to a common place #40375

Open
cvinayak opened this issue Nov 16, 2021 · 1 comment
Open

Bluetooth: Controller: Move BT_EVT_MASK_LE_META_EVENT check to a common place #40375

cvinayak opened this issue Nov 16, 2021 · 1 comment
Assignees
@cvinayak
Labels
area: Bluetooth Controller area: Bluetooth Enhancement Good first issue

Comments

@cvinayak
Copy link
Contributor

@cvinayak cvinayak commented Nov 16, 2021

Is your enhancement proposal related to a problem? Please describe.
(event_mask & BT_EVT_MASK_LE_META_EVENT) is check many places in the code. Would it be possible to move this checks to a single place in e.g. encode_control for all LE events?

Describe the solution you'd like
https://github.com/zephyrproject-rtos/zephyr/pull/31009/files#r736212741

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or graphics (drag-and-drop an image) about the feature request here.
@cvinayak cvinayak added the Enhancement label Nov 16, 2021
@cvinayak cvinayak changed the title Bluetooth: Controller: Move BT_EVT_MASK_LE_META_EVENT to a common place Bluetooth: Controller: Move BT_EVT_MASK_LE_META_EVENT check to a common place Nov 16, 2021
@cvinayak cvinayak mentioned this issue Nov 16, 2021
Merged
@cvinayak cvinayak self-assigned this Nov 16, 2021
@wolterhv
Copy link

@wolterhv wolterhv commented Jan 21, 2022

@cvinayak I see that the pull request, which you referenced as the solution to this issue in the issue description, is merged. Is this issue still open in spite of that?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Controller area: Bluetooth Enhancement Good first issue
2 participants
@wolterhv @cvinayak