zephyrproject-rtos/zephyr

Bluetooth: Controller: Use defined for phy flags for S2 and S8 Coded PHY options

Open

#37,458 opened on Aug 5, 2021

View on GitHub
 (1 comment) (1 reaction) (2 assignees)C (15,249 stars) (9,144 forks)batch import
EnhancementGood first issuearea: Bluetootharea: Bluetooth Controller

Description

Is your enhancement proposal related to a problem? Please describe. _ A clear and concise description of what the problem is._

Use defines in

  • radio_phy_set
  • radio_switch_complete_and_tx
  • and so on

Refer to: https://github.com/zephyrproject-rtos/zephyr/pull/37283/files#r680822843

Describe the solution you'd like A clear and concise description of what you want to happen.

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.

Contributor guide

Tech stack
c
Domain
backend
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
C programmingZephyr Bluetooth controller architecture
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Refer to PR #37283 for example usage of defines for phy flags. Search in Bluetooth controller source files, specifically functions radio phy set, radio switch complete and tx, and replace hardcoded values for S2 and S8 Coded PHY with appropriate defines. Ensure consistency across the codebase.