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

Fermée Adaptée aux débutants
  • Quelque chose ne va pas ?
#11,099 4 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@mikeysklar y travaille déjà.

Depuis le 12/10/2026.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
68/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
c
Domaine
embedded-iot

Piste de recherche

Commencez par les définitions des cartes adafruit_feather_rp2040_adalogger et adafruit_feather_rp2350_adalogger, puis comparez leurs noms SDIO avec les définitions de Metro RP2040/RP2350 et de Fruit Jam. Supprimez SD_CMD et SD_DAT0 à SD_DAT3 des deux cartes Adalogger, conservez SD_CLK et vérifiez que les alias SDIO_* restent présents.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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

Langage dominant
C
Étoiles
4.6k
Forks
1.4k
Merge moyen
1 j 17 h
PR mergées (30 j)
163

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de adafruit/circuitpython

Toutes les issues de adafruit/circuitpython

Issues similaires

Plus d'issues C

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.