E2E @ c6e30ffe - push to main #184

Closed
shrikanthup wants to merge 2 commits from refs/pull/184/head into main
shrikanthup commented 2026-05-15 16:19:43 +00:00 (Migrated from staging.allspice.dev)

Source: push to main
Ref: c6e30ffe3ac1966ed8063b8b6251d4bcc3fba134
Mode: e2e
Cache: enabled (s3://allspice-storage-allspice-cloudstaging-staging/datasheet_cache)
Cleanup: auto-closed after 7 days

Resolved config
{
  "dump_analysis_dir": "debug_dump",
  "libraries": [
    {
      "ipn_column": "IPN",
      "mpn_column": "MPN",
      "path": ".allspice/library.csv",
      "type": "csv"
    },
    {
      "digikey_client_id_env": "DIGIKEY_CLIENT_ID",
      "digikey_client_secret_env": "DIGIKEY_CLIENT_SECRET",
      "type": "digikey"
    }
  ]
}

Triggered at 2026-05-15T16:19:40.811269Z. Branch 9af1556e will be deleted on cleanup.

**Source:** push to main **Ref:** [`c6e30ffe3ac1966ed8063b8b6251d4bcc3fba134`](https://github.com/AllSpiceIO/connections-checker/tree/c6e30ffe3ac1966ed8063b8b6251d4bcc3fba134) **Mode:** e2e **Cache:** enabled (s3://allspice-storage-allspice-cloudstaging-staging/datasheet_cache) **Cleanup:** auto-closed after 7 days <details> <summary>Resolved config</summary> ```json { "dump_analysis_dir": "debug_dump", "libraries": [ { "ipn_column": "IPN", "mpn_column": "MPN", "path": ".allspice/library.csv", "type": "csv" }, { "digikey_client_id_env": "DIGIKEY_CLIENT_ID", "digikey_client_secret_env": "DIGIKEY_CLIENT_SECRET", "type": "digikey" } ] } ``` </details> Triggered at 2026-05-15T16:19:40.811269Z. Branch `9af1556e` will be deleted on cleanup.
AllSpiceAlice commented 2026-05-15 16:25:34 +00:00 (Migrated from staging.allspice.dev)

DRCY has reviewed this Design Review, and there should be a review posted below.

DRCY has reviewed this Design Review, and there should be a review posted below.
AllSpiceAlice (Migrated from staging.allspice.dev) reviewed 2026-05-15 17:06:49 +00:00
AllSpiceAlice (Migrated from staging.allspice.dev) left a comment

DRCY Connections Checker Review

DRCY reviewed the connections in the 25 page(s) that changed in this DR. From these pages, DRCY selected 10 component(s) to review, and found 7 potential issue(s) in 4 component(s). DRCY has posted comments on the schematic for each potential issue. For more details on the components reviewed and their connections, click on the dropdown below.

Component Details

DRCY selected and reviewed all connections from the following components of the schematic:

CPU1 - INTEL_ATOM_E3825_SOC

DRCY flagged 2 potential issues in this component.

📄 DRCY referred to this Datasheet for this component. 📤 Replace a datasheet

Pin Designator Pin Name Net Correct? Analysis
BC1 VGA_DDCCLK CRT_CLK
VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.
  • Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol (from datasheet 140-0004628)
  • I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation (reasoning)
  • Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low (reasoning)
  • If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low (reasoning)
  • At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination (reasoning)
  • Typical termination resistors for unused I2C lines are 10K or higher (reasoning)
  • The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled (from schematic)
  • VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented (from schematic)
  • The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled (reasoning)
BC2 VGA_DDCDATA CRT_DAT
VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.
  • Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol (from datasheet 140-0004628)
  • I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation (reasoning)
  • Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low (reasoning)
  • If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low (reasoning)
  • At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination (reasoning)
  • Typical termination resistors for unused I2C lines are 10K or higher (reasoning)
  • The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled (from schematic)
  • VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented (from schematic)
  • The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled (reasoning)
AK12 DDI0_RCOMP_P DDI_RCOMP_N
DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.
  • Pin AK13 is connected to net DDI_RCOMP_P (from schematic)
  • Pin AK12 is connected to net DDI_RCOMP_N (from schematic)
  • Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet (from datasheet 140-0004628)
  • Pin AK12 is designated as DDI0_RCOMP_P in the datasheet (from datasheet 140-0004628)
  • Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets (from schematic)
  • The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin (reasoning)
  • The net connections are reversed compared to the datasheet pin designations (reasoning)
  • Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net (reasoning)
AK13 ~DDI0_RCOMP DDI_RCOMP_P
DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.
  • Pin AK13 is connected to net DDI_RCOMP_P (from schematic)
  • Pin AK12 is connected to net DDI_RCOMP_N (from schematic)
  • Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet (from datasheet 140-0004628)
  • Pin AK12 is designated as DDI0_RCOMP_P in the datasheet (from datasheet 140-0004628)
  • Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets (from schematic)
  • The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin (reasoning)
  • The net connections are reversed compared to the datasheet pin designations (reasoning)
  • Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net (reasoning)
A29 RESERVED_A29 GPIO_NC13 RESERVED_A29 pin connected to GPIO_NC13 net with 10K pull-down resistor R102. This appears to be an intentional configuration for Bay Trail-I variant.
B30 GPIO_S0_NC12 $3N566 GPIO_S0_NC12 pin connected to test point TP15 (DNI). This provides a test point for the GPIO signal.
C26 DDI0_DDCDATA HDMI_DDCDAT DDI0_DDCDATA pin is correctly connected to HDMI_DDCDAT net for I2C-based Display Data Channel communication with HDMI display.
C28 DDI0_DDCCLK HDMI_DDCCLK DDI0_DDCCLK pin is correctly connected to HDMI_DDCCLK net for I2C-based Display Data Channel communication with HDMI display.
D27 DDI0_HPD HDMI_HPD_B DDI0_HPD pin correctly connected to HDMI_HPD_B net, which is the output of inverter U39 providing inverted hot plug detect signal.
G30 DDI1_DDCCLK GND DDI1_DDCCLK pin tied to GND, intentionally disabling DDI1 interface as indicated by 'Bay Trail-M Remove eDP Port' note.
K30 DDI1_HPD GND DDI1_HPD pin tied to GND, intentionally disabling DDI1 interface as indicated by 'Bay Trail-M Remove eDP Port' note.
P14 RESERVED_P14 MCSI_RCOMP RESERVED_P14 pin connected to MCSI_RCOMP net with 150-ohm resistor R213 to ground. This appears to be an intentional use for MCSI compensation as indicated by 'Bay Trail-I MCSI Port' note.
P30 DDI1_DDCDATA DDI1_DDCDAT DDI1_DDCDATA pin connected to DDI1_DDCDAT net with 2.2K pull-down resistor R257. This is part of the DDI1 disable configuration.
AH2 RESERVED_VSS3 $3N554 RESERVED_VSS3 pin connected through 0-ohm resistor R43 to GND, which is the correct configuration per datasheet recommendations.
AH3 RESERVED_VSS2 $3N552 RESERVED_VSS2 pin connected through 0-ohm resistor R46 to GND, which is the correct configuration per datasheet recommendations.
AM2 RESERVED_VSS1 $3N589 RESERVED_VSS1 pin connected through 0-ohm resistor R41 to GND, which is the correct configuration per datasheet recommendations.
AM3 RESERVED_VSS0 $3N579 RESERVED_VSS0 pin connected through 0-ohm resistor R42 to GND, which is the correct configuration per datasheet recommendations.
AP2 DDI0_TXN_3 HDMI_CLK_DN DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AP3 DDI0_TXP_3 HDMI_CLK_DP DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AR1 DDI0_TXN_2 HDMI_TX0_DN DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AR3 DDI0_TXP_2 HDMI_TX0_DP DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AT2 DDI0_TXP_1 HDMI_TX1_DP DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AT3 DDI0_TXN_1 HDMI_TX1_DN DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AV2 DDI0_TXN_0 HDMI_TX2_DN DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AV3 DDI0_TXP_0 HDMI_TX2_DP DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals.
AW1 VGA_IREF $3N548 VGA_IREF pin correctly connected to 357-ohm reference resistor R40 to ground, matching datasheet specifications for VGA DAC current reference.
AY3 VGA_IRTN GND VGA_IRTN pin correctly connected to GND, serving as the current return path for the VGA DAC.
U39 - SN74LVC1G14DCKR

DRCY found no issues in this component 🎉

📄 DRCY referred to this Datasheet for this component. 📤 Replace a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 NC NC (No Connect) pin is correctly left unconnected.
2 A HDMI_HPD Input pin A is correctly connected to HDMI_HPD signal for Schmitt-trigger inversion.
3 GND GND Ground pin is correctly connected to GND.
4 Y HDMI_HPD_B Output pin Y is correctly connected to HDMI_HPD_B, providing inverted HPD signal to CPU.
5 VCC +V1P8S VCC pin is correctly connected to +V1P8S power rail with appropriate decoupling.
R39 - 150 ohm 1% 1/16W 0402

DRCY flagged 2 potential issues in this component.

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 CRT_CLK
Pin 1 is connected to CRT_CLK (VGA_DDCCLK from CPU1). This resistor is incorrectly configured as a pull-down on an I2C clock line.
  • Pin 1 is connected to net CRT_CLK (from schematic)
  • Net CRT_CLK originates from CPU1 pin BC1 (VGA_DDCCLK) (from schematic)
  • VGA_DDCCLK is an I2C clock signal for DDC communication operating at 100 kHz (from datasheet 140-0004628)
  • I2C clock lines require pull-up resistors, not pull-down resistors (reasoning)
  • This resistor should be reconfigured as a pull-up resistor to a positive supply voltage (reasoning)
2 2 GND
Pin 2 is connected to ground, completing the incorrect pull-down configuration.
  • Pin 2 is connected to the GND net (from schematic)
  • This creates a pull-down resistor configuration which is incorrect for I2C signals (reasoning)
  • For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground (reasoning)
R40 - 357 ohm 1% 1/4W 0402

DRCY found no issues in this component 🎉

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 $3N548 Pin 1 is connected to VGA_IREF from CPU1, correctly setting the reference current for the VGA DAC.
2 2 GND Pin 2 is correctly connected to ground, completing the reference current path.
R38 - 150 ohm 1% 1/16W 0402

DRCY flagged 2 potential issues in this component.

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 CRT_DAT
Pin 1 is connected to CRT_DAT (VGA_DDCDATA from CPU1). This resistor is incorrectly configured as a pull-down on an I2C data line.
  • Pin 1 is connected to net CRT_DAT (from schematic)
  • Net CRT_DAT originates from CPU1 pin BC2 (VGA_DDCDATA) (from schematic)
  • VGA_DDCDATA is an I2C data signal for DDC communication (from datasheet 140-0004628)
  • I2C data lines require pull-up resistors, not pull-down resistors (reasoning)
  • This resistor should be reconfigured as a pull-up resistor to a positive supply voltage (reasoning)
2 2 GND
Pin 2 is connected to ground, completing the incorrect pull-down configuration.
  • Pin 2 is connected to the GND net (from schematic)
  • This creates a pull-down resistor configuration which is incorrect for I2C signals (reasoning)
  • For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground (reasoning)
R217 - 402 ohm 1% 1/4W 0402

DRCY found no issues in this component 🎉

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 DDI_RCOMP_N DDI compensation resistor correctly connected between DDI_RCOMP pins, though the net names are swapped due to CPU1 pin swap.
2 2 DDI_RCOMP_P DDI compensation resistor correctly connected between DDI_RCOMP pins, though the net names are swapped due to CPU1 pin swap.
R213 - 150 ohm 1% 1/16W 0402

DRCY found no issues in this component 🎉

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 MCSI_RCOMP MCSI compensation resistor correctly connected between MCSI_RCOMP pin and ground.
2 2 GND MCSI compensation resistor correctly connected between MCSI_RCOMP pin and ground.
R257 - 2.2K ohm 1% 1/10W 0402

DRCY flagged 1 potential issues in this component.

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 DDI1_DDCDAT
R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.
  • R257 pin 1 is connected to net DDI1_DDCDAT (from schematic)
  • R257 pin 2 is connected to GND (from schematic)
  • R257 is a 2.2K ohm resistor (from schematic)
  • Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) (from schematic)
  • DDI1_DDCDATA is an I2C data line for the Display Data Channel (from datasheet 140-0004628)
  • The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND (from schematic)
  • Text note 'Bay Trail-M Remove eDP Port' confirms the DDI1/eDP interface is intentionally removed (from schematic)
  • For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating (reasoning)
  • The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor (reasoning)
  • The pull-down resistor serves no functional purpose for a disabled interface and should be removed (reasoning)
  • Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration (reasoning)
2 2 GND
R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.
  • R257 pin 1 is connected to net DDI1_DDCDAT (from schematic)
  • R257 pin 2 is connected to GND (from schematic)
  • R257 is a 2.2K ohm resistor (from schematic)
  • Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) (from schematic)
  • DDI1_DDCDATA is an I2C data line for the Display Data Channel (from datasheet 140-0004628)
  • The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND (from schematic)
  • Text note 'Bay Trail-M Remove eDP Port' confirms the DDI1/eDP interface is intentionally removed (from schematic)
  • For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating (reasoning)
  • The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor (reasoning)
  • The pull-down resistor serves no functional purpose for a disabled interface and should be removed (reasoning)
  • Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration (reasoning)
R102

DRCY found no issues in this component 🎉

⚠️ DRCY couldn't retrieve a Datasheet for this component. 📤 Upload a datasheet

Pin Designator Pin Name Net Correct? Analysis
1 1 GND 10K pull-down resistor correctly connected between GPIO_NC13 and GND.
2 2 GPIO_NC13 10K pull-down resistor correctly connected between GPIO_NC13 and GND.
TP15

DRCY found no issues in this component 🎉

ℹ️ DRCY didn't use a Datasheet for this component.

Pin Designator Pin Name Net Correct? Analysis
1 1 $3N566 Test point for GPIO_S0_NC12 signal, marked as DNI (Do Not Install).
📤 Upload Missing Datasheets

DRCY was unable to find datasheets for the following components. You can upload datasheets to your repository to use them in future reviews.

⚠️ Analysis Failures

DRCY was unable to analyze 22 component(s). These issues are usually temporary, and may be resolved with another review request. If problems persist, please contact AllSpice support.

  • ERR4 (Page Processing Failed): Unable to complete analysis for this page
  • ERR5 (Page Processing Failed): Unable to complete analysis for this page
  • ERR6 (Page Processing Failed): Unable to complete analysis for this page
  • ERR7 (Page Processing Failed): Unable to complete analysis for this page
  • ERR8 (Page Processing Failed): Unable to complete analysis for this page
  • ERR9 (Page Processing Failed): Unable to complete analysis for this page
  • ERR10 (Page Processing Failed): Unable to complete analysis for this page
  • ERR11 (Page Processing Failed): Unable to complete analysis for this page
  • ERR12 (Page Processing Failed): Unable to complete analysis for this page
  • ERR13 (Page Processing Failed): Unable to complete analysis for this page
  • ERR14 (Page Processing Failed): Unable to complete analysis for this page
  • ERR15 (Page Processing Failed): Unable to complete analysis for this page
  • ERR16 (Page Processing Failed): Unable to complete analysis for this page
  • ERR17 (Page Processing Failed): Unable to complete analysis for this page
  • ERR18 (Page Processing Failed): Unable to complete analysis for this page
  • ERR19 (Page Processing Failed): Unable to complete analysis for this page
  • ERR20 (Page Processing Failed): Unable to complete analysis for this page
  • ERR21 (Page Processing Failed): Unable to complete analysis for this page
  • ERR22 (Page Processing Failed): Unable to complete analysis for this page
  • ERR23 (Page Processing Failed): Unable to complete analysis for this page
  • ERR24 (Page Processing Failed): Unable to complete analysis for this page
  • ERR25 (Page Processing Failed): Unable to complete analysis for this page

Note: DRCY uses AI. Please verify the outputs.

# DRCY Connections Checker Review DRCY reviewed the connections in the 25 page(s) that changed in this DR. From these pages, DRCY selected 10 component(s) to review, and found 7 potential issue(s) in 4 component(s). DRCY has posted comments on the schematic for each potential issue. For more details on the components reviewed and their connections, click on the dropdown below. <details> <summary><b>Component Details</b></summary> DRCY selected and reviewed all connections from the following components of the schematic: <details> <summary><b>CPU1</b> - INTEL_ATOM_E3825_SOC ❌</summary> DRCY flagged 2 potential issues in this component. 📄 [DRCY referred to this Datasheet for this component.](https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf) [📤 Replace a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/140-0004628) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | BC1 | VGA_DDCCLK | CRT_CLK | ❌ | <details><summary>VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="55.34,40.37,62.84,47.87" aspect-ratio="1.29" } <ul><li>Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) <em>(from schematic)</em></li><li>Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) <em>(from schematic)</em></li><li>Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation <em>(reasoning)</em></li><li>Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low <em>(reasoning)</em></li><li>If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low <em>(reasoning)</em></li><li>At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination <em>(reasoning)</em></li><li>Typical termination resistors for unused I2C lines are 10K or higher <em>(reasoning)</em></li><li>The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled <em>(from schematic)</em></li><li>VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented <em>(from schematic)</em></li><li>The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled <em>(reasoning)</em></li></ul></details> | | BC2 | VGA_DDCDATA | CRT_DAT | ❌ | <details><summary>VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="55.34,40.96,62.84,48.46" aspect-ratio="1.29" } <ul><li>Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) <em>(from schematic)</em></li><li>Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) <em>(from schematic)</em></li><li>Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation <em>(reasoning)</em></li><li>Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low <em>(reasoning)</em></li><li>If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low <em>(reasoning)</em></li><li>At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination <em>(reasoning)</em></li><li>Typical termination resistors for unused I2C lines are 10K or higher <em>(reasoning)</em></li><li>The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled <em>(from schematic)</em></li><li>VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented <em>(from schematic)</em></li><li>The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled <em>(reasoning)</em></li></ul></details> | | AK12 | DDI0_RCOMP_P | DDI_RCOMP_N | ❌ | <details><summary>DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="38.07,31.55,45.57,39.05" aspect-ratio="1.29" } <ul><li>Pin AK13 is connected to net DDI_RCOMP_P <em>(from schematic)</em></li><li>Pin AK12 is connected to net DDI_RCOMP_N <em>(from schematic)</em></li><li>Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>Pin AK12 is designated as DDI0_RCOMP_P in the datasheet <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets <em>(from schematic)</em></li><li>The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin <em>(reasoning)</em></li><li>The net connections are reversed compared to the datasheet pin designations <em>(reasoning)</em></li><li>Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net <em>(reasoning)</em></li></ul></details> | | AK13 | ~DDI0_RCOMP | DDI_RCOMP_P | ❌ | <details><summary>DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="38.07,30.96,45.57,38.46" aspect-ratio="1.29" } <ul><li>Pin AK13 is connected to net DDI_RCOMP_P <em>(from schematic)</em></li><li>Pin AK12 is connected to net DDI_RCOMP_N <em>(from schematic)</em></li><li>Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>Pin AK12 is designated as DDI0_RCOMP_P in the datasheet <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets <em>(from schematic)</em></li><li>The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin <em>(reasoning)</em></li><li>The net connections are reversed compared to the datasheet pin designations <em>(reasoning)</em></li><li>Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net <em>(reasoning)</em></li></ul></details> | | A29 | RESERVED_A29 | GPIO_NC13 | ✅ | RESERVED_A29 pin connected to GPIO_NC13 net with 10K pull-down resistor R102. This appears to be an intentional configuration for Bay Trail-I variant. | | B30 | GPIO_S0_NC12 | $3N566 | ✅ | GPIO_S0_NC12 pin connected to test point TP15 (DNI). This provides a test point for the GPIO signal. | | C26 | DDI0_DDCDATA | HDMI_DDCDAT | ✅ | DDI0_DDCDATA pin is correctly connected to HDMI_DDCDAT net for I2C-based Display Data Channel communication with HDMI display. | | C28 | DDI0_DDCCLK | HDMI_DDCCLK | ✅ | DDI0_DDCCLK pin is correctly connected to HDMI_DDCCLK net for I2C-based Display Data Channel communication with HDMI display. | | D27 | DDI0_HPD | HDMI_HPD_B | ✅ | DDI0_HPD pin correctly connected to HDMI_HPD_B net, which is the output of inverter U39 providing inverted hot plug detect signal. | | G30 | DDI1_DDCCLK | GND | ✅ | DDI1_DDCCLK pin tied to GND, intentionally disabling DDI1 interface as indicated by 'Bay Trail-M Remove eDP Port' note. | | K30 | DDI1_HPD | GND | ✅ | DDI1_HPD pin tied to GND, intentionally disabling DDI1 interface as indicated by 'Bay Trail-M Remove eDP Port' note. | | P14 | RESERVED_P14 | MCSI_RCOMP | ✅ | RESERVED_P14 pin connected to MCSI_RCOMP net with 150-ohm resistor R213 to ground. This appears to be an intentional use for MCSI compensation as indicated by 'Bay Trail-I MCSI Port' note. | | P30 | DDI1_DDCDATA | DDI1_DDCDAT | ✅ | DDI1_DDCDATA pin connected to DDI1_DDCDAT net with 2.2K pull-down resistor R257. This is part of the DDI1 disable configuration. | | AH2 | RESERVED_VSS3 | $3N554 | ✅ | RESERVED_VSS3 pin connected through 0-ohm resistor R43 to GND, which is the correct configuration per datasheet recommendations. | | AH3 | RESERVED_VSS2 | $3N552 | ✅ | RESERVED_VSS2 pin connected through 0-ohm resistor R46 to GND, which is the correct configuration per datasheet recommendations. | | AM2 | RESERVED_VSS1 | $3N589 | ✅ | RESERVED_VSS1 pin connected through 0-ohm resistor R41 to GND, which is the correct configuration per datasheet recommendations. | | AM3 | RESERVED_VSS0 | $3N579 | ✅ | RESERVED_VSS0 pin connected through 0-ohm resistor R42 to GND, which is the correct configuration per datasheet recommendations. | | AP2 | DDI0_TXN_3 | HDMI_CLK_DN | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AP3 | DDI0_TXP_3 | HDMI_CLK_DP | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AR1 | DDI0_TXN_2 | HDMI_TX0_DN | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AR3 | DDI0_TXP_2 | HDMI_TX0_DP | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AT2 | DDI0_TXP_1 | HDMI_TX1_DP | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AT3 | DDI0_TXN_1 | HDMI_TX1_DN | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AV2 | DDI0_TXN_0 | HDMI_TX2_DN | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AV3 | DDI0_TXP_0 | HDMI_TX2_DP | ✅ | DDI0 differential transmit pairs are correctly connected to HDMI signals with proper polarity. These high-speed differential pairs carry HDMI video data (three data lanes) and clock signals. | | AW1 | VGA_IREF | $3N548 | ✅ | VGA_IREF pin correctly connected to 357-ohm reference resistor R40 to ground, matching datasheet specifications for VGA DAC current reference. | | AY3 | VGA_IRTN | GND | ✅ | VGA_IRTN pin correctly connected to GND, serving as the current return path for the VGA DAC. | </details> <details> <summary><b>U39</b> - SN74LVC1G14DCKR ✅</summary> DRCY found no issues in this component 🎉 📄 [DRCY referred to this Datasheet for this component.](https://www.ti.com/lit/ds/symlink/sn74lvc1g14.pdf?ts=1753365169590&ref_url=https%25253A%25252F%25252Fwww.ti.com%25252Fproduct%25252FSN74LVC1G14) [📤 Replace a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/SN74LVC1G14DCKR) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | NC | | ✅ | NC (No Connect) pin is correctly left unconnected. | | 2 | A | HDMI_HPD | ✅ | Input pin A is correctly connected to HDMI_HPD signal for Schmitt-trigger inversion. | | 3 | GND | GND | ✅ | Ground pin is correctly connected to GND. | | 4 | Y | HDMI_HPD_B | ✅ | Output pin Y is correctly connected to HDMI_HPD_B, providing inverted HPD signal to CPU. | | 5 | VCC | +V1P8S | ✅ | VCC pin is correctly connected to +V1P8S power rail with appropriate decoupling. | </details> <details> <summary><b>R39</b> - 150 ohm 1% 1/16W 0402 ❌</summary> DRCY flagged 2 potential issues in this component. ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | CRT_CLK | ❌ | <details><summary>Pin 1 is connected to CRT_CLK (VGA_DDCCLK from CPU1). This resistor is incorrectly configured as a pull-down on an I2C clock line.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="68.52,39.19,76.02,46.69" aspect-ratio="1.29" } <ul><li>Pin 1 is connected to net CRT_CLK <em>(from schematic)</em></li><li>Net CRT_CLK originates from CPU1 pin BC1 (VGA_DDCCLK) <em>(from schematic)</em></li><li>VGA_DDCCLK is an I2C clock signal for DDC communication operating at 100 kHz <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>I2C clock lines require pull-up resistors, not pull-down resistors <em>(reasoning)</em></li><li>This resistor should be reconfigured as a pull-up resistor to a positive supply voltage <em>(reasoning)</em></li></ul></details> | | 2 | 2 | GND | ❌ | <details><summary>Pin 2 is connected to ground, completing the incorrect pull-down configuration.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="69.43,39.19,76.93,46.69" aspect-ratio="1.29" } <ul><li>Pin 2 is connected to the GND net <em>(from schematic)</em></li><li>This creates a pull-down resistor configuration which is incorrect for I2C signals <em>(reasoning)</em></li><li>For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground <em>(reasoning)</em></li></ul></details> | </details> <details> <summary><b>R40</b> - 357 ohm 1% 1/4W 0402 ✅</summary> DRCY found no issues in this component 🎉 ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0004695) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | $3N548 | ✅ | Pin 1 is connected to VGA_IREF from CPU1, correctly setting the reference current for the VGA DAC. | | 2 | 2 | GND | ✅ | Pin 2 is correctly connected to ground, completing the reference current path. | </details> <details> <summary><b>R38</b> - 150 ohm 1% 1/16W 0402 ❌</summary> DRCY flagged 2 potential issues in this component. ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | CRT_DAT | ❌ | <details><summary>Pin 1 is connected to CRT_DAT (VGA_DDCDATA from CPU1). This resistor is incorrectly configured as a pull-down on an I2C data line.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="68.52,40.96,76.02,48.46" aspect-ratio="1.29" } <ul><li>Pin 1 is connected to net CRT_DAT <em>(from schematic)</em></li><li>Net CRT_DAT originates from CPU1 pin BC2 (VGA_DDCDATA) <em>(from schematic)</em></li><li>VGA_DDCDATA is an I2C data signal for DDC communication <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>I2C data lines require pull-up resistors, not pull-down resistors <em>(reasoning)</em></li><li>This resistor should be reconfigured as a pull-up resistor to a positive supply voltage <em>(reasoning)</em></li></ul></details> | | 2 | 2 | GND | ❌ | <details><summary>Pin 2 is connected to ground, completing the incorrect pull-down configuration.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="69.43,40.96,76.93,48.46" aspect-ratio="1.29" } <ul><li>Pin 2 is connected to the GND net <em>(from schematic)</em></li><li>This creates a pull-down resistor configuration which is incorrect for I2C signals <em>(reasoning)</em></li><li>For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground <em>(reasoning)</em></li></ul></details> | </details> <details> <summary><b>R217</b> - 402 ohm 1% 1/4W 0402 ✅</summary> DRCY found no issues in this component 🎉 ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0004476) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | DDI_RCOMP_N | ✅ | DDI compensation resistor correctly connected between DDI_RCOMP pins, though the net names are swapped due to CPU1 pin swap. | | 2 | 2 | DDI_RCOMP_P | ✅ | DDI compensation resistor correctly connected between DDI_RCOMP pins, though the net names are swapped due to CPU1 pin swap. | </details> <details> <summary><b>R213</b> - 150 ohm 1% 1/16W 0402 ✅</summary> DRCY found no issues in this component 🎉 ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | MCSI_RCOMP | ✅ | MCSI compensation resistor correctly connected between MCSI_RCOMP pin and ground. | | 2 | 2 | GND | ✅ | MCSI compensation resistor correctly connected between MCSI_RCOMP pin and ground. | </details> <details> <summary><b>R257</b> - 2.2K ohm 1% 1/10W 0402 ❌</summary> DRCY flagged 1 potential issues in this component. ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0001984) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | DDI1_DDCDAT | ❌ | <details><summary>R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="64.43,26.26,71.93,33.76" aspect-ratio="1.29" } <ul><li>R257 pin 1 is connected to net DDI1_DDCDAT <em>(from schematic)</em></li><li>R257 pin 2 is connected to GND <em>(from schematic)</em></li><li>R257 is a 2.2K ohm resistor <em>(from schematic)</em></li><li>Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) <em>(from schematic)</em></li><li>DDI1_DDCDATA is an I2C data line for the Display Data Channel <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND <em>(from schematic)</em></li><li>Text note &#x27;Bay Trail-M Remove eDP Port&#x27; confirms the DDI1/eDP interface is intentionally removed <em>(from schematic)</em></li><li>For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating <em>(reasoning)</em></li><li>The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor <em>(reasoning)</em></li><li>The pull-down resistor serves no functional purpose for a disabled interface and should be removed <em>(reasoning)</em></li><li>Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration <em>(reasoning)</em></li></ul></details> | | 2 | 2 | GND | ❌ | <details><summary>R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.</summary>!thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="65.34,26.26,72.84,33.76" aspect-ratio="1.29" } <ul><li>R257 pin 1 is connected to net DDI1_DDCDAT <em>(from schematic)</em></li><li>R257 pin 2 is connected to GND <em>(from schematic)</em></li><li>R257 is a 2.2K ohm resistor <em>(from schematic)</em></li><li>Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) <em>(from schematic)</em></li><li>DDI1_DDCDATA is an I2C data line for the Display Data Channel <em>(from datasheet <a href="https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf">140-0004628</a>)</em></li><li>The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND <em>(from schematic)</em></li><li>Text note &#x27;Bay Trail-M Remove eDP Port&#x27; confirms the DDI1/eDP interface is intentionally removed <em>(from schematic)</em></li><li>For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating <em>(reasoning)</em></li><li>The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor <em>(reasoning)</em></li><li>The pull-down resistor serves no functional purpose for a disabled interface and should be removed <em>(reasoning)</em></li><li>Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration <em>(reasoning)</em></li></ul></details> | </details> <details> <summary><b>R102</b> ✅</summary> DRCY found no issues in this component 🎉 ⚠️ DRCY couldn't retrieve a Datasheet for this component. [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0001875) | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | GND | ✅ | 10K pull-down resistor correctly connected between GPIO_NC13 and GND. | | 2 | 2 | GPIO_NC13 | ✅ | 10K pull-down resistor correctly connected between GPIO_NC13 and GND. | </details> <details> <summary><b>TP15</b> ✅</summary> DRCY found no issues in this component 🎉 ℹ️ DRCY didn't use a Datasheet for this component. | Pin Designator | Pin Name | Net | Correct? | Analysis | |---------------:|----------|-----|:--------:|----------| | 1 | 1 | $3N566 | ✅ | Test point for GPIO_S0_NC12 signal, marked as DNI (Do Not Install). | </details> </details> <details> <summary>📤 Upload Missing Datasheets</summary> DRCY was unable to find datasheets for the following components. You can upload datasheets to your repository to use them in future reviews. - **R102** (110-0001875): [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0001875) - **R257** (110-0001984): [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0001984) - **R213, R38, R39** (110-0002631): [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631) - **R217** (110-0004476): [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0004476) - **R40** (110-0004695): [📤 Upload a datasheet](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0004695) </details> <details> <summary>⚠️ Analysis Failures</summary> DRCY was unable to analyze 22 component(s). These issues are usually temporary, and may be resolved with another review request. If problems persist, please contact AllSpice support. - **ERR4** (Page Processing Failed): Unable to complete analysis for this page - **ERR5** (Page Processing Failed): Unable to complete analysis for this page - **ERR6** (Page Processing Failed): Unable to complete analysis for this page - **ERR7** (Page Processing Failed): Unable to complete analysis for this page - **ERR8** (Page Processing Failed): Unable to complete analysis for this page - **ERR9** (Page Processing Failed): Unable to complete analysis for this page - **ERR10** (Page Processing Failed): Unable to complete analysis for this page - **ERR11** (Page Processing Failed): Unable to complete analysis for this page - **ERR12** (Page Processing Failed): Unable to complete analysis for this page - **ERR13** (Page Processing Failed): Unable to complete analysis for this page - **ERR14** (Page Processing Failed): Unable to complete analysis for this page - **ERR15** (Page Processing Failed): Unable to complete analysis for this page - **ERR16** (Page Processing Failed): Unable to complete analysis for this page - **ERR17** (Page Processing Failed): Unable to complete analysis for this page - **ERR18** (Page Processing Failed): Unable to complete analysis for this page - **ERR19** (Page Processing Failed): Unable to complete analysis for this page - **ERR20** (Page Processing Failed): Unable to complete analysis for this page - **ERR21** (Page Processing Failed): Unable to complete analysis for this page - **ERR22** (Page Processing Failed): Unable to complete analysis for this page - **ERR23** (Page Processing Failed): Unable to complete analysis for this page - **ERR24** (Page Processing Failed): Unable to complete analysis for this page - **ERR25** (Page Processing Failed): Unable to complete analysis for this page </details> <sub><sup>Note: DRCY uses AI. Please verify the outputs.</sup></sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component CPU1, pins BC1, BC2: VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.

  • Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) (from schematic)
  • Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol (from datasheet 140-0004628)
  • I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation (reasoning)
  • Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low (reasoning)
  • If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low (reasoning)
  • At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination (reasoning)
  • Typical termination resistors for unused I2C lines are 10K or higher (reasoning)
  • The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled (from schematic)
  • VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented (from schematic)
  • The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled (reasoning)

Replace a datasheet: 📤 CPU1

[🚨 Error] **Component `CPU1`, pins `BC1, BC2`: VGA_DDCCLK and VGA_DDCDATA pins are connected to 150-ohm pull-down resistors (R39 and R38 respectively) to ground. DDC signals use I2C protocol which requires pull-up resistors for proper operation, not pull-down resistors. If VGA is intentionally disabled, the 150-ohm value is too low for termination and should be 10K or higher to avoid excessive current draw.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="55.34,40.37,62.84,48.46" aspect-ratio="1.29" } - Pin BC1 (VGA_DDCCLK) connects to net CRT_CLK, which connects to R39 pin 1 (150-ohm resistor to ground) *(from schematic)* - Pin BC2 (VGA_DDCDATA) connects to net CRT_DAT, which connects to R38 pin 1 (150-ohm resistor to ground) *(from schematic)* - Pins BC1 and BC2 are designated as VGA_DDCCLK and VGA_DDCDATA respectively, implementing Display Data Channel using I2C protocol *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - I2C protocol requires pull-up resistors (typically 2.2K-10K ohms) on both SDA and SCL lines for proper operation *(reasoning)* - Pull-down resistors prevent I2C bus from functioning correctly by holding the lines low *(reasoning)* - If VGA is intentionally disabled, pull-down resistors could be used to terminate unused lines, but 150 ohms is too low *(reasoning)* - At 3.3V, a 150-ohm pull-down would draw 22mA, which is excessive for termination *(reasoning)* - Typical termination resistors for unused I2C lines are 10K or higher *(reasoning)* - The schematic shows DDI1 interface is partially disabled (DDI1_DDCCLK and DDI1_HPD tied to ground), suggesting some display interfaces may be intentionally disabled *(from schematic)* - VGA_IREF (pin AW1) is connected to a 357-ohm resistor to ground, suggesting VGA circuitry is at least partially implemented *(from schematic)* - The 150-ohm pull-downs should be changed to pull-ups (2.2K-10K) if VGA is intended to be used, or to higher value pull-downs (10K+) if VGA is intentionally disabled *(reasoning)* <sub>Replace a datasheet: [📤 CPU1](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/140-0004628)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component CPU1, pins AK12, AK13: DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.

  • Pin AK13 is connected to net DDI_RCOMP_P (from schematic)
  • Pin AK12 is connected to net DDI_RCOMP_N (from schematic)
  • Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet (from datasheet 140-0004628)
  • Pin AK12 is designated as DDI0_RCOMP_P in the datasheet (from datasheet 140-0004628)
  • Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets (from schematic)
  • The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin (reasoning)
  • The net connections are reversed compared to the datasheet pin designations (reasoning)
  • Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net (reasoning)

Replace a datasheet: 📤 CPU1

[🚨 Error] **Component `CPU1`, pins `AK12, AK13`: DDI compensation pins are swapped. Pin AK13 (DDI_RCOMP_N per datasheet) connects to DDI_RCOMP_P net, while pin AK12 (DDI0_RCOMP_P per datasheet) connects to DDI_RCOMP_N net. These connections should be reversed.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="38.07,30.96,45.57,39.05" aspect-ratio="1.29" } - Pin AK13 is connected to net DDI_RCOMP_P *(from schematic)* - Pin AK12 is connected to net DDI_RCOMP_N *(from schematic)* - Pin AK13 is designated as D̅D̅I̅0̅_̅R̅C̅O̅M̅P̅ (DDI_RCOMP_N with overline notation indicating negation) in the datasheet *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - Pin AK12 is designated as DDI0_RCOMP_P in the datasheet *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - Compensation resistor R217 (402 ohm) is connected between DDI_RCOMP_N and DDI_RCOMP_P nets *(from schematic)* - The pin names in the datasheet clearly indicate AK13 should be the negative pin and AK12 should be the positive pin *(reasoning)* - The net connections are reversed compared to the datasheet pin designations *(reasoning)* - Pin AK13 should connect to DDI_RCOMP_N net and pin AK12 should connect to DDI_RCOMP_P net *(reasoning)* <sub>Replace a datasheet: [📤 CPU1](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/140-0004628)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component R39, pin 1: Pin 1 is connected to CRT_CLK (VGA_DDCCLK from CPU1). This resistor is incorrectly configured as a pull-down on an I2C clock line.

  • Pin 1 is connected to net CRT_CLK (from schematic)
  • Net CRT_CLK originates from CPU1 pin BC1 (VGA_DDCCLK) (from schematic)
  • VGA_DDCCLK is an I2C clock signal for DDC communication operating at 100 kHz (from datasheet 140-0004628)
  • I2C clock lines require pull-up resistors, not pull-down resistors (reasoning)
  • This resistor should be reconfigured as a pull-up resistor to a positive supply voltage (reasoning)

Upload a datasheet: 📤 R39

[🚨 Error] **Component `R39`, pin `1`: Pin 1 is connected to CRT_CLK (VGA_DDCCLK from CPU1). This resistor is incorrectly configured as a pull-down on an I2C clock line.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="68.52,39.19,76.02,46.69" aspect-ratio="1.29" } - Pin 1 is connected to net CRT_CLK *(from schematic)* - Net CRT_CLK originates from CPU1 pin BC1 (VGA_DDCCLK) *(from schematic)* - VGA_DDCCLK is an I2C clock signal for DDC communication operating at 100 kHz *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - I2C clock lines require pull-up resistors, not pull-down resistors *(reasoning)* - This resistor should be reconfigured as a pull-up resistor to a positive supply voltage *(reasoning)* <sub>Upload a datasheet: [📤 R39](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component R39, pin 2: Pin 2 is connected to ground, completing the incorrect pull-down configuration.

  • Pin 2 is connected to the GND net (from schematic)
  • This creates a pull-down resistor configuration which is incorrect for I2C signals (reasoning)
  • For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground (reasoning)

Upload a datasheet: 📤 R39

[🚨 Error] **Component `R39`, pin `2`: Pin 2 is connected to ground, completing the incorrect pull-down configuration.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="69.43,39.19,76.93,46.69" aspect-ratio="1.29" } - Pin 2 is connected to the GND net *(from schematic)* - This creates a pull-down resistor configuration which is incorrect for I2C signals *(reasoning)* - For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground *(reasoning)* <sub>Upload a datasheet: [📤 R39](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component R38, pin 1: Pin 1 is connected to CRT_DAT (VGA_DDCDATA from CPU1). This resistor is incorrectly configured as a pull-down on an I2C data line.

  • Pin 1 is connected to net CRT_DAT (from schematic)
  • Net CRT_DAT originates from CPU1 pin BC2 (VGA_DDCDATA) (from schematic)
  • VGA_DDCDATA is an I2C data signal for DDC communication (from datasheet 140-0004628)
  • I2C data lines require pull-up resistors, not pull-down resistors (reasoning)
  • This resistor should be reconfigured as a pull-up resistor to a positive supply voltage (reasoning)

Upload a datasheet: 📤 R38

[🚨 Error] **Component `R38`, pin `1`: Pin 1 is connected to CRT_DAT (VGA_DDCDATA from CPU1). This resistor is incorrectly configured as a pull-down on an I2C data line.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="68.52,40.96,76.02,48.46" aspect-ratio="1.29" } - Pin 1 is connected to net CRT_DAT *(from schematic)* - Net CRT_DAT originates from CPU1 pin BC2 (VGA_DDCDATA) *(from schematic)* - VGA_DDCDATA is an I2C data signal for DDC communication *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - I2C data lines require pull-up resistors, not pull-down resistors *(reasoning)* - This resistor should be reconfigured as a pull-up resistor to a positive supply voltage *(reasoning)* <sub>Upload a datasheet: [📤 R38](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component R38, pin 2: Pin 2 is connected to ground, completing the incorrect pull-down configuration.

  • Pin 2 is connected to the GND net (from schematic)
  • This creates a pull-down resistor configuration which is incorrect for I2C signals (reasoning)
  • For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground (reasoning)

Upload a datasheet: 📤 R38

[🚨 Error] **Component `R38`, pin `2`: Pin 2 is connected to ground, completing the incorrect pull-down configuration.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="69.43,40.96,76.93,48.46" aspect-ratio="1.29" } - Pin 2 is connected to the GND net *(from schematic)* - This creates a pull-down resistor configuration which is incorrect for I2C signals *(reasoning)* - For correct I2C operation, this pin should be connected to a positive supply voltage instead of ground *(reasoning)* <sub>Upload a datasheet: [📤 R38](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0002631)</sub>
AllSpiceAlice (Migrated from staging.allspice.dev) commented 2026-05-15 17:06:49 +00:00

[🚨 Error] Component R257, pins 1, 2: R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.

  • R257 pin 1 is connected to net DDI1_DDCDAT (from schematic)
  • R257 pin 2 is connected to GND (from schematic)
  • R257 is a 2.2K ohm resistor (from schematic)
  • Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) (from schematic)
  • DDI1_DDCDATA is an I2C data line for the Display Data Channel (from datasheet 140-0004628)
  • The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND (from schematic)
  • Text note 'Bay Trail-M Remove eDP Port' confirms the DDI1/eDP interface is intentionally removed (from schematic)
  • For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating (reasoning)
  • The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor (reasoning)
  • The pull-down resistor serves no functional purpose for a disabled interface and should be removed (reasoning)
  • Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration (reasoning)

Upload a datasheet: 📤 R257

[🚨 Error] **Component `R257`, pins `1, 2`: R257 provides an inappropriate pull-down resistor on the DDI1_DDCDATA I2C line. For a disabled interface, the data line should be tied directly to GND (like DDI1_DDCCLK) or left floating, not pulled down through a resistor.** !thumbnail[](Turbot.prj){ diff="AI-Evals/E2E-Turbot:05990542eebf7494ab9568ebcaeca5c0bea19065...32ae8bd60e3089c5025eb65fa32074f5d24b3f4d" pr="184" doc-id="c6d51d1759888fc83971" diff-visibility="full" variant="default" view-coords="64.43,26.26,72.84,33.76" aspect-ratio="1.29" } - R257 pin 1 is connected to net DDI1_DDCDAT *(from schematic)* - R257 pin 2 is connected to GND *(from schematic)* - R257 is a 2.2K ohm resistor *(from schematic)* - Net DDI1_DDCDAT connects to CPU1 pin P30 (DDI1_DDCDATA) *(from schematic)* - DDI1_DDCDATA is an I2C data line for the Display Data Channel *(from datasheet [140-0004628](<https://staging.allspice.dev/AI-Evals/E2E-Turbot/raw/commit/32ae8bd60e3089c5025eb65fa32074f5d24b3f4d/.allspice/datasheets/140-0004628/atom_e3800_family_datasheet-1522396.pdf>))* - The DDI1 interface is intentionally disabled, as evidenced by DDI1_DDCCLK (G30) and DDI1_HPD (K30) being tied directly to GND *(from schematic)* - Text note &#x27;Bay Trail-M Remove eDP Port&#x27; confirms the DDI1/eDP interface is intentionally removed *(from schematic)* - For a disabled I2C interface, standard practice is to tie the lines directly to GND or leave them floating *(reasoning)* - The configuration is inconsistent: DDI1_DDCCLK is tied directly to GND while DDI1_DDCDATA has a pull-down resistor *(reasoning)* - The pull-down resistor serves no functional purpose for a disabled interface and should be removed *(reasoning)* - Recommendation: Remove R257 (mark as DNI) and tie DDI1_DDCDATA directly to GND to match the DDI1_DDCCLK configuration *(reasoning)* <sub>Upload a datasheet: [📤 R257](https://staging.allspice.dev/AI-Evals/E2E-Turbot/_upload/9af1556e/.allspice/datasheets/110-0001984)</sub>

Design review closed

Sign in to join this conversation.
No description provided.