probe_oen.pio doesn't set SWCLK as a sideset pin

Abierto
#196 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Necesita aclaración
Estado de actividad
Estancado
Stack tecnológico
c
Área
embedded-iot

Línea de trabajo

Compare the sideset configuration in src/probe.pio around line 104 with src/probe_oen.pio around line 83. Read the surrounding PIO setup to determine whether probe_oen.pio should configure SWCLK as a sideset pin or whether its comment is incorrect. Done means the intended behavior is established and the code and comment agree.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Whilst looking at #194 I noticed that probe.pio says:

    // Set SWCLK as a sideset pin
    sm_config_set_sideset_pins(sm_config, PROBE_PIN_SWCLK);

(i.e. the comment matches the code), whereas probe_oen.pio says:

    // Set SWDIOEN and SWCLK as sideset pins
    sm_config_set_sideset_pins(sm_config, PROBE_PIN_SWDIOEN);

i.e. despite what the comment says, it doesn't actually set SWCLK as a sideset pin.

Not being familiar with how this code works, I don't know if probe_oen.pio is accidentally missing a sm_config_set_sideset_pins(sm_config, PROBE_PIN_SWCLK); line, or whether the comment simply needs updating to // Set SWDIOEN as a sideset pin ? 🤷‍♂️

Lenguaje dominante
C
Estrellas
1.2k
Forks
363
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de raspberrypi/debugprobe

Todos los issues de raspberrypi/debugprobe

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.