Skip to content

Conversation

@jfurcean
Copy link
Contributor

@jfurcean jfurcean commented Oct 6, 2021

This adds detection for the MacroPad RP2040 u2if. It needs a new USB PID before it can be merged. The PR execuc/u2if#7 should be merged before this one as well. @caternuson

@caternuson
Copy link
Contributor

@ladyada Can we offer up a new PID for this? We came up with new ones for Feather, Itsy, etc. as I remember:

# Feather RP2040
if product == 0x00F1:
return boards.FEATHER_U2IF
# Itsy Bitsy RP2040
if product == 0x00FD:
return boards.ITSYBITSY_U2IF
# QT Py RP2040
if product == 0x00F7:
return boards.QTPY_U2IF
# QT2040 Trinkey
if product == 0x0109:
return boards.QT2040_TRINKEY_U2IF

@ladyada
Copy link
Member

ladyada commented Oct 6, 2021

@caternuson yeah do you have access/know how to assign a new PID?

@caternuson
Copy link
Contributor

@ladyada I do not.

@ladyada
Copy link
Member

ladyada commented Oct 6, 2021

@dhalbert can illuminate it for ya!

@jfurcean jfurcean force-pushed the add_macropad_rp2040 branch from 329348e to 30144d6 Compare October 6, 2021 21:14
@dhalbert
Copy link
Contributor

dhalbert commented Oct 6, 2021

@caternuson Use 0x239A/0x0107 for MacroPad u2if. We mistakenly assigned this to bootloader, but like other RP2040's it has its own due to the ROM bootloader.

@caternuson
Copy link
Contributor

@jfurcean One quick change and I think we can merge. Tested this locally with a MacroPad loaded with new U2IF firwmare. All works as expected.

@caternuson caternuson merged commit 4769224 into adafruit:main Oct 14, 2021
@jfurcean jfurcean deleted the add_macropad_rp2040 branch October 14, 2021 15:17
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.

4 participants