USB7216 support and switch pad peel strength improvement #31
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
this should help prevent them from tearing off if a key is misaligned or has bent pins when inserted
The USB7216 uses an alternate location, pin 80 for VBUS monitoring. This adds a voltage divider, R18 and R28 which should only be installed if a USB7216 is used. Pads for pull down resistors are provided for DP1_CC1/2, but will not be populated.
It's in IPC-A-600, but adding to be clear there is a minimum copper requirement
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1st commit
Add polygon fills around switch connector pads to increase peel strength of connector pads
2nd commit
Add support to the PCB for USB7216 hub IC as an alternate. This consists of the R18, R28 divider to provide VBUS_MON_UP. R18 and R28 should be installed for USB7216, but should not be installed for USB7206. Two "DNP" (do not place) resistor footprints are added, R33, R34. These likely will never be used, but are included due to unclear notes in the data sheet about pull downs on these pins. I believe there are internal pull downs on these pins it is referencing rather than a requirement for external.
The PCB revision is increased to 1.1.
3rd commit
Add fab note with minimum outer layer finished copper requirement.
Versioning
Prior to merging this PR, PCB revision 1.0 should probably be tagged. After merging 1.1 should be tagged. Alternately, directories for each PCB revision could be created as there will always be old boards to support. Version directories in Git are common for hardware.
Logo
The System76 logo was removed when updating from the schematic and will be added back in a further commit.