General:

• Very well done overall
• At any point, you can refer to the majority of (older) EPS boards, the ADF7021, CC1120 and their schematics/layouts to get an idea on how to improve your designs.
  • Side note: there may be some discrepancies between the boards, so if you’re confused about something you can always ask

Schematic

• This goes for basically all parts on any schematic:
  • We want the Manufacturer Part Number showing, not the Comment, so hide all comments and instead make the manufacturer part number visible (There is one exception to this rule: passive components, for which neither the comment nor the MPN should be visible**)**

• Don’t think we need an entirely new sheet for 3 header rows, I suggest making the other schematic sheet (DC_Motor_Schematic) bigger and put these in a corner somewhere
  • Also since we’d only have one schematic, lets rename it to something more general
• J7 is pretty self explanatory but its hard to tell what P1 and J8 are for, at least for someone who’s looking at it for the first time. Add a one or two word text line to describe what each header is for (eg. VN-100 Connector)
• The note is unnecessary since we already have the ‘No ERC’ mark on the pin

• Need a decoupling cap to put near power pins

• Anywhere you are using power on your board, your power rails and ground should not be generic net names, they should actually be using the symbol for the rail and ground, as shown below:

• Important tip: when your using any power rails/ground, don’t go and create a new rail from the top bar, copy an existing rail and paste it wherever you need it. Otherwise, Altium thinks its a brand new net and it messes stuff up
• Not a huge deal, but you can add a ‘+’ in front of your voltage values to ensure that its a positive potential difference

• Ports are not needed, they are meant to show connections between schematic sheets, but we really only have one main schematic sheet, so net names will suffice: