zephyrproject-rtos/zephyr

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

Offen

#40.378 geöffnet am 16.11.2021

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C (9.144 Forks)batch import
EnhancementGood first issuearea: Bluetootharea: Bluetooth Controller

Repository-Metriken

Stars
 (15.249 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die verlinkte PR #31009 an den spezifischen Kommentarzeilen, um zu identifizieren, wo hartcodierte Array Größen für access addr und crc init auftreten. Definieren Sie dann Makros für diese Größen (z.B. BT CTLR ACCESS ADDR SIZE und BT CTLR CRC INIT SIZE) und ersetzen Sie die hartcodierten Werte im gesamten Bluetooth Controller Code. Die Änderungen sollten auf das Controller Modul beschränkt sein. Stellen Sie sicher, dass keine weiteren Vorkommen in der Codebasis existieren.
Tech Stack
c
Domain
Keine
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80