zephyrproject-rtos/zephyr

Bluetooth: Controller: Use defines for access_addr and crc_init array sizes

Open

#40.378 aperta il 16 nov 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)C (9144 fork)batch import
EnhancementGood first issuearea: Bluetootharea: Bluetooth Controller

Metriche repository

Star
 (15.249 star)
Metriche merge PR
 (Merge medio 5g 20h) (1000 PR mergiate in 30 g)

Descrizione

Is your enhancement proposal related to a problem? Please describe. access_addr always seems to be of size 4; that should be #defined. crc_init always seem to be of size 3; that should be #defined.

https://github.com/zephyrproject-rtos/zephyr/pull/31009/files#r736229677 https://github.com/zephyrproject-rtos/zephyr/pull/31009/files#r736229386

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.

Guida contributor

Direzione di ricerca
Esamina le righe di commento specifiche nel PR collegato #31009 per identificare dove compaiono le dimensioni degli array hardcoded per access addr e crc init. Quindi, definisci macro per quelle dimensioni (ad es. BT CTLR ACCESS ADDR SIZE e BT CTLR CRC INIT SIZE) e sostituisci i valori hardcoded in tutto il codice del controller Bluetooth. Le modifiche dovrebbero essere isolate al modulo del controller. Verifica che non esistano altre occorrenze nella codebase.
Tech stack
c
Dominio
Nessuno
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80