Skip to content

espressif-esp32-c3: Fix cross-type bus parameter resolution error#448

Merged
napowderly merged 4 commits intomainfrom
fix/esp32-c3-bus-parameter-mismatch
Apr 8, 2026
Merged

espressif-esp32-c3: Fix cross-type bus parameter resolution error#448
napowderly merged 4 commits intomainfrom
fix/esp32-c3-bus-parameter-mismatch

Conversation

@napowderly
Copy link
Copy Markdown
Contributor

@napowderly napowderly commented Apr 8, 2026

Summary

  • Connect GPIO (ElectricLogic) to ADC/USB (ElectricSignal) at the .line level instead of full interface ~
  • The full connection forms a bus across mismatched types, causing Key not mapping to parameter when resolving characteristic_impedance — a bus parameter on ElectricSignal that doesn't exist on ElectricLogic
  • Affects gpio[0-5] ~ adc and gpio[18-19] ~ usb_if.d.n/p connections

Test plan

  • Build esp32_minimal example — should no longer error with "Key not mapping to parameter"
  • Verify USB and ADC connections still produce correct netlists

🤖 Generated with Claude Code

napowderly and others added 4 commits April 7, 2026 17:30
Connect GPIO (ElectricLogic) to ADC/USB (ElectricSignal) at the line
level instead of the full interface level. The full `~` connection
forms a bus across mismatched types, causing "Key not mapping to
parameter" when resolving `characteristic_impedance` — a bus parameter
that exists on ElectricSignal but not ElectricLogic.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Regenerated PCB layouts to match the updated ADC/USB GPIO connections.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove unused imports (Capacitor, Resistor, USB2_0, has_designator_prefix,
  has_part_picked, is_atomic_part) from base model
- Replace deprecated has_single_electric_reference_shared with
  has_single_electric_reference in mini and wroom modules
- Update buttons dependency from 0.2.4 to 0.4.0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@napowderly napowderly merged commit cfb333c into main Apr 8, 2026
4 checks passed
@napowderly napowderly deleted the fix/esp32-c3-bus-parameter-mismatch branch April 8, 2026 01:04
napowderly added a commit that referenced this pull request Apr 8, 2026
…tion

Same fix as espressif-esp32-c3 (#448): connect GPIO (ElectricLogic) to
ADC/USB/touch (ElectricSignal) at the .line level to avoid forming a
cross-type bus that breaks characteristic_impedance resolution.

ESP32-S3: fix USB d.p/d.n, touch[1-14], adc[1-20] connections
STM32H723: fix USB d.p/d.n connections

- espressif-esp32-s3: 0.2.1 → 0.3.0
- st-stm32h723: 0.1.2 → 0.2.0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
napowderly added a commit that referenced this pull request Apr 8, 2026
…tion (#451)

* espressif-esp32-s3, st-stm32h723: Fix cross-type bus parameter resolution

Same fix as espressif-esp32-c3 (#448): connect GPIO (ElectricLogic) to
ADC/USB/touch (ElectricSignal) at the .line level to avoid forming a
cross-type bus that breaks characteristic_impedance resolution.

ESP32-S3: fix USB d.p/d.n, touch[1-14], adc[1-20] connections
STM32H723: fix USB d.p/d.n connections

- espressif-esp32-s3: 0.2.1 → 0.3.0
- st-stm32h723: 0.1.2 → 0.2.0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* rebuild locally

* st-stm32h723: Remove unused imports and commented-out lan8742a ref

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* st-stm32h723: Sync README usage block with usage.ato

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant