Skip to content

Conversation

@gnumpi
Copy link
Collaborator

@gnumpi gnumpi commented Dec 1, 2024

  • optimized timing for sending PD messages
  • disabled cable detach / attach detection
  • delay automation state trigger by 1s

@gnumpi gnumpi self-assigned this Dec 1, 2024
@gnumpi gnumpi linked an issue Dec 1, 2024 that may be closed by this pull request
@ms1design
Copy link

I tested it on my rev4, here's the "pop" noises during boot matrix table:

YES = pops still happen.

When I directly connect to the MBP USB-C PD I get boot loop.

Power Adapter PD Connected Hardware Reset Software Reset USB-C Power Draw sensor Notes
Mac USB-C PD (direct) ??? ??? ??? Unknown Boot loop
Mac USB-C PD (through Adapter) YES YES NO Unknown Pops all the time till full boot
Anker Nano3 30W PD YES YES NO 3A @ 9V Pops a bit less that Mac USB-C
Apple USB-C PD 20W YES YES NO 2A @ 9V Pops just 2 times during boot
Unknown 5V 2A 10W YES YES NO Unknown A lot of pops during boot

@that1guy
Copy link
Collaborator

that1guy commented Dec 3, 2024

Overall, this is working very well for me @gnumpi! NOTE: In all my testing when I swapped to a new USB-C connection the devices is completely powered off. In other words, I was not "hot-swapping" my USB-C connections why keeping the Core board plugged in. Here are my findings:

Power Adapter PD Connected Hardware Reset Software Reset USB-C Power Draw sensor Notes
Mac USB-C PD (direct) YES NO NO 3A @ 5V Works Well! No Boot Loops!
Mac USB-C PD (through Adapter) N/A N/A N/A N/A Brad doesn't have USB-C Adapter
Anker Nano3 30W PD YES NO NO 3A @ 9V Seems to be reliable
Apple USB-C PD 20W N/A N/A N/A N/A Brad doesn't have this charger
Apple USB-C PD 67W YES NO NO 3A @ 9V Nice. Seems stable
Unknown 9V 2.23A 20W YES NO NO 2A @ 9V Nice. Go China! See screenshot below.

image

@that1guy
Copy link
Collaborator

that1guy commented Dec 3, 2024

@ms1design FYI - I don't think any of the code in this PR will address the popping on boot. That is a separate issue here: #127

@that1guy that1guy self-requested a review December 3, 2024 02:04
@that1guy that1guy merged commit 1ff892a into develop Dec 3, 2024
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.

BUG: Sat1 Rev4 Boot Loop HAT USB-C

4 participants