Skip to content

Wiring diagram: show the battery in the Cirkit schematic - #20

Closed
Ptitlouis6012 wants to merge 1 commit into
TigerTag-Project:mainfrom
Ptitlouis6012:docs/wiring-diagram-battery
Closed

Ptitlouis6012 wants to merge 1 commit into
TigerTag-Project:mainfrom
Ptitlouis6012:docs/wiring-diagram-battery

Conversation

@Ptitlouis6012

Copy link
Copy Markdown
Contributor

What this changes

The Cirkit Designer project linked from README.md and docs/WIRING.md now also shows the 3.7V battery wired to the board's BAT connector, so the exported PNG is refreshed to match. The PN532 wiring itself is unchanged — VCC to 3V3, GND to GND, TXD/RXD crossed on GPIO44/43 — and still matches firmware/include/config.h (documented reader wiring matches config.h guard passes). The linked Cirkit project URL is unchanged, it now just shows the current schematic.

What you tested it on

Docs only.

Checklist

  • One concern. A rename and a behaviour change in one diff cannot be reviewed.
  • English — comments, identifiers, log messages.
  • No credentials, keys or tokens added anywhere.
  • Any hardware constant carries a comment saying what breaks without it. (n/a, no hardware constant touched)
  • Docs updated if behaviour changed. (image only, no wiring behaviour changed)
  • CHANGELOG.md updated for anything user-visible. (not user-visible firmware behaviour; left to the release ritual)

Same PN532 wiring as before (VCC to 3V3, GND to GND, TXD/RXD crossed
on GPIO44/43); the Cirkit Designer project now also shows the 3.7V
battery wired to the board's BAT connector, so the exported image is
refreshed to match. The linked project is unchanged.
@Ptitlouis6012

Copy link
Copy Markdown
Contributor Author

Superseded by cf81a6a, which already refreshes this diagram on main (with its own Cirkit project and updated alt text). Closing in favour of that.

@Ptitlouis6012
Ptitlouis6012 deleted the docs/wiring-diagram-battery branch October 1, 2026 03:59
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