Drop SD_ aliases for SDIO signals (SD_CMD, SD_DAT0..3) on RP2040 Adalogger board

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#11,099 4 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@mikeysklar arbeitet bereits daran.

Seit 12.10.2026.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
68/100
Issue-Typ
Refactoring
Klarheit
Klar beschrieben
Aktivitätsstatus
Ruhig
Tech-Stack
c
Bereich
embedded-iot

Rechercherichtung

Beginne mit den Board-Definitionen für adafruit_feather_rp2040_adalogger und adafruit_feather_rp2350_adalogger und vergleiche dann ihre SDIO-Namen mit den Definitionen für Metro RP2040/RP2350 und Fruit Jam. Entferne SD_CMD und SD_DAT0 bis SD_DAT3 von den beiden Adalogger-Boards, behalte SD_CLK bei und überprüfe, dass die SDIO_*-Aliase erhalten bleiben.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

board breaks api

Per discussion in #11090, the Adalogger boards carry SD_-prefixed aliases for SDIO-only signals, and we should drop them at the next major release so SD_ consistently means SPI and SDIO_ means SDIO.

Naming rule

Bus Convention Names
SPI abbreviated, SPI terms SD_CLK, SD_MOSI, SD_MISO, SD_CS
SDIO spelled-out, SDIO terms SDIO_CLOCK, SDIO_COMMAND, SDIO_DATA0..3

The names to remove are the hybrids — the SPI SD_ prefix on SDIO-only concepts, which have no SPI meaning:

  • SD_CMD
  • SD_DAT0, SD_DAT1, SD_DAT2, SD_DAT3

SD_CLK stays (CLK is a real SPI term; there is no SD_CLOCK anywhere in the tree). Affected boards: adafruit_feather_rp2040_adalogger, adafruit_feather_rp2350_adalogger. The SDIO_* names already exist on Metro RP2040/RP2350 and Fruit Jam.

Learning Guide usage check (Adafruit_Learning_System_Guides)

Name CircuitPython guide references Safe to drop?
SD_DAT0..3 0 ✅ yes
SD_CMD 0 ✅ yes
SD_CLK 2 ⚠️ keep (SPI clock)

The two SD_CLK uses are Feather_RP2040_Adalogger/CircuitPython_SD_Card_{Read,Write}/code.py, both busio.SPI(board.SD_CLK, board.SD_MOSI, board.SD_MISO) — SPI, unaffected. The PIN_SD_DAT3_CS hits under Factory_Tests/ and runcpm-* are Arduino macros (pins_arduino.h), a separate namespace, so not affected.

cc @dhalbert @foamyguy

Vorherrschende Sprache
C
Sterne
4.6k
Forks
1.4k
Ø Merge
1 T. 13 Std.
Gemergte PRs (30 T.)
171

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus adafruit/circuitpython

Alle Issues in adafruit/circuitpython

Ähnliche Issues

Weitere Issues zu C

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.