Skip to content

Conversation

BlitzCityDIY
Copy link
Contributor

Adding board definition for Adafruit Feather RP2350. This includes a delay for XOSC.

@earlephilhower
Copy link
Owner

Looks like there are merge issues, can you take a look at it, @BlitzCityDIY ? FYI, RP2350B patch needed a new set of libs built with different CMake options. Your new libs w/HSTX turned on might need to be rebuilt from head options...

@BlitzCityDIY
Copy link
Contributor Author

okay, thank you. @tannewt - can you take a look?

Copy link
Owner

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. One thing, though, is there any possible configuration on the HSTX port? Should there be #defines for the pinout in the variant.h file, or does it always live at the same IO location w/o GPIO muxing?

@tannewt
Copy link
Contributor

tannewt commented Sep 16, 2024

The pins don't vary due to the pin mux. The HSTX can vary what comes out each HSTX pin on its own. You could swap color channels for example.

@earlephilhower
Copy link
Owner

Got it, thx. Just noted the same from the RP2350 datasheet.

There's one old file, I think, causing CI failure now. The JSON file should be regenerated and not include pico-debug. See the style CI diff (note that pico-debug is no longer maintained or in OpenOCD):

diff --git a/tools/json/adafruit_feather_rp2350_hstx.json b/tools/json/adafruit_feather_rp2350_hstx.json
index 05a783a..4ed8480 100644
--- a/tools/json/adafruit_feather_rp2350_hstx.json
+++ b/tools/json/adafruit_feather_rp2350_hstx.json
@@ -47,8 +47,7 @@
             "jlink",
             "raspberrypi-swd",
             "picotool",
-            "picoprobe",
-            "pico-debug"
+            "picoprobe"
         ]
     },
     "url": "https://www.raspberrypi.org/products/raspberry-pi-pico/",
Error: Process completed with exit code 1.

@earlephilhower earlephilhower self-requested a review September 17, 2024 14:12
@earlephilhower earlephilhower merged commit 163b209 into earlephilhower:master Sep 17, 2024
20 checks passed
@tannewt
Copy link
Contributor

tannewt commented Sep 17, 2024

Thank you for the fast reviews!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants