Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hackheld PCB Revised [1] #3

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Hackheld PCB Revised [1] #3

wants to merge 8 commits into from

Conversation

ItsKarlito
Copy link

@ItsKarlito ItsKarlito commented Nov 1, 2021 •
edited
Loading

CHANGES:

  • Merged Stefan's changes
  • Updated netlist, pcb and gerbers to reflect Stefan's change (swapping OLED connector's 3V3 and GND)
  • Removed redundant traces
  • Removed <90deg trace angles
  • Connected GND pours
  • Alignment/centering of traces
  • Moved silk labels overlapping pads and silk overlapping neopxl footprint
  • Added missing neopxl footprint to footprint directory
BEFORE AFTER
image image
image image

@ItsKarlito
Copy link
Author

ItsKarlito commented Nov 5, 2021 •
edited
Loading

I updated the PCB to reflect the change @spacehuhn made (swapping OLED connector's 3V3 and GND, see issue #4).

@ItsKarlito ItsKarlito closed this Nov 19, 2021
@ItsKarlito ItsKarlito reopened this Nov 19, 2021
@LZ2DMV
Copy link

LZ2DMV commented Nov 19, 2021

It will probably be a good idea to move the left and right buttons on some other GPIO, as they can't be used currently (see issue #2).

@spacehuhn spacehuhn force-pushed the main branch 2 times, most recently from ac7a202 to f128325 Compare May 3, 2023 14:03
@jahid9137
Copy link

Hello please give me pcb diagram

@SoLongSidekick
Copy link

Love the changes, but just figured I'd mention the whole "90° traces are bad" thing is an old wive's tale.

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

@Sajidamali Sajidamali Sajidamali approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ItsKarlito @LZ2DMV @jahid9137 @SoLongSidekick @Sajidamali