Part Reviews
Reviewer Name | Manufacturer Part Number | Symbol Name | Footprint Name | Comments |
---|---|---|---|---|
Rayyan Mahmood | NCV8114ASN180T1G | NCV8114ASN180T1G | 1.8V_Output_LDO |
|
Rayyan Mahmood | ICM-20948 | IC_9DOF_IMU_W24 | ICM20948 - 9DOFIMU |
|
Rayyan Mahmood | TXB0104PWR | TXB0104PWR | TXB0104PWR 4-bit directional voltage-level translator |
|
Schematic Sheet Reviews
power.SchDoc (<Rayyan Mahmood>)
- Add a second LED for 1.8V out
- Change the colour of this title to match the colour of “GND”, “3V3”, etc.
- Add test points for 1.8V and 3.3V and a GND test point as well.
- Look at the 48-17V buck schematic and see which TPs were used for the power and GND and use the exact same ones here
- Annotate schematic → label components
Click annotate schematics quietly and that’ll do it
GPS_Module.SchDoc (<Rayyan Mahmood>)
- It is up to the designer’s discretion as to what should be reviewed for each schematic sheet
Generic Schematic Reviews
Net Naming (<Reviewer Name>)
- All major nets named
- No duplicate net names, or nets with multiple net names
- Differential pairs end with _N and _P
- Active low signals are indicated with either a “_n” at the end or with overbar notation (using a “\” character after each letter in the net name, i.e. E\N\)
Wiring and Connections (<Reviewer Name>)
- No 4 point connections
- All unconnected pins have an ERC directive
- Devices sharing an I2C or SPI bus must be simultaneously powered on/off (i.e. by powering them all off of the same power rail) - prevents unpowered devices from pulling the bus low
- Lines that require pull ups are pulled up in exactly one place (not pulled up multiple times, or missing pull-ups)
Component Labels (<Reviewer Name>)
- ICs show part number
- Resistors show resistance value
- Capacitors show capacitance and voltage rating
- For capacitors, voltage rating is 2x higher than expected voltage for low-voltage nets (<12V), 1.5x higher for high-voltage nets (>12V)
- Inductors show inductance and rated current
- Ferrites show impedance @ frequency value
- Diodes show part number, voltage rating, and current rating (if applicable). Exception is for LEDs, which only need part number
Design-for-Test (<Reviewer Name>)
- All voltage rails have test points and an LED indicator
- Enable/shutdown/reset/etc. lines for ICs have test points
- Communication interfaces have test points (i.e. I2C, SPI, UART)
- All sensor outputs have test points
- Any other signals that might want to be checked during testing should have a test point