Two more incoming Feathers #1334

Merged
PaintYourDragon merged 5 commits from master into master 2023-03-29 21:18:45 +03:00
PaintYourDragon commented 2023-03-29 07:09:07 +03:00 (Migrated from github.com)

ThinkINK (e-paper display) and USB Host.

ThinkINK (e-paper display) and USB Host.
earlephilhower commented 2023-03-29 21:04:00 +03:00 (Migrated from github.com)

Neat! You guys are coming out with a ton of cool boards.

Is this PR ready to merge or are you expecting more changes/additions?

Neat! You guys are coming out with a ton of cool boards. Is this PR ready to merge or are you expecting more changes/additions?
PaintYourDragon commented 2023-03-29 21:16:57 +03:00 (Migrated from github.com)

I think that’s it for this round (add RFM, ThinkINK & USB host, fix SCORPIO & DVI pins). Might be more in the future but probably not for some weeks.

I think that’s it for this round (add RFM, ThinkINK & USB host, fix SCORPIO & DVI pins). Might be more in the future but probably not for some weeks.
hathach (Migrated from github.com) reviewed 2023-04-10 12:56:21 +03:00
@@ -0,0 +15,4 @@
// USB host connector
#define PIN_USB_DP (16u)
#define PIN_USB_DN (17u)
hathach (Migrated from github.com) commented 2023-04-10 12:56:20 +03:00

@PaintYourDragon late to the party, but there is a typo, this should be DP/DM (not DN though). Since we need to make an PR to correct it, do you think we should rename them to PIN_USB_HOST_DP/DM to make it clearer that this is for host. Since I saw some other board has the same naming but for usb device ?

@PaintYourDragon late to the party, but there is a typo, this should be DP/DM (not DN though). Since we need to make an PR to correct it, do you think we should rename them to PIN_USB_HOST_DP/DM to make it clearer that this is for host. Since I saw some other board has the same naming but for usb device ?
PaintYourDragon (Migrated from github.com) reviewed 2023-04-10 20:23:29 +03:00
@@ -0,0 +15,4 @@
// USB host connector
#define PIN_USB_DP (16u)
#define PIN_USB_DN (17u)
PaintYourDragon (Migrated from github.com) commented 2023-04-10 20:23:29 +03:00

Good idea, I’ll add HOST in the pin names, and DM instead of DN (I was probably following the lead of the DVI Feather, where it’s [N]egative. Boards are not quite shipping to customers yet so the changes shouldn’t affect anyone.

Good idea, I’ll add HOST in the pin names, and DM instead of DN (I was probably following the lead of the DVI Feather, where it’s [N]egative. Boards are not quite shipping to customers yet so the changes shouldn’t affect anyone.
Sign in to join this conversation.