AllSpice-Demos/AI-DRCY-Review
AllSpice-Demos/AI-DRCY-Review
2
Fork 0
Files Issues 4 Design Reviews 1 Actions Releases Activity

Fix D53 pinouts #5

New Issue
Open
opened 2026-01-13 17:12:14 +00:00 by allspice-carah · 1 comment
allspice-carah commented 2026-01-13 17:12:14 +00:00
Owner
Copy Link

Originally posted by @DRCY in AllSpice-Demos/AI-Design-Review-v3#1 (comment)

_Originally posted by @DRCY in https://hub.allspice.io/AllSpice-Demos/AI-Design-Review-v3/pulls/1#diff-41108d21273198ed8c5983ec6b585df21600e8f7-issuecomment-41143_
AllSpiceAlice was assigned by allspice-carah 2026-01-13 17:12:24 +00:00
allspice-carah commented 2026-01-13 17:13:02 +00:00
Author
Owner
Copy Link

Component D53, pin K: Cathode correctly connected to NetD51_A but diode is non-functional due to unconnected anode.


The cathode connects to NetD51_A as intended for negative clamping of the TACH_2 signal conditioning circuit. According to the datasheet (page 1), the cathode is the 'Negative terminal of the Schottky diode'. However, since the anode is unconnected, this diode cannot conduct and provides no clamping function. The connection to NetD51_A is correct in principle, but the diode is rendered non-functional by the missing anode connection. This leaves the TACH_2 circuit without negative voltage protection, unlike the other three tachometer circuits which have proper negative clamps.

**Component `D53`, pin `K`: Cathode correctly connected to NetD51_A but diode is non-functional due to unconnected anode.** !thumbnail[](Fans.SchDoc){ diff="AllSpice-Demos/AI-Design-Review-v3:26590581be02618ce3f47deedb643c2d47d4182d...b09435216bda90650e7fdf6b009674871e52ddc7" pr="1" diff-visibility="full" variant="default" view-coords="63.06,33.47,70.56,40.97" aspect-ratio="1.29" } The cathode connects to NetD51_A as intended for negative clamping of the TACH_2 signal conditioning circuit. According to the [datasheet](https://www.mccsemi.com/pdf/Products/BAT54WX(SOD-523).pdf#page=1) (page 1), the cathode is the 'Negative terminal of the Schottky diode'. However, since the anode is unconnected, this diode cannot conduct and provides no clamping function. The connection to NetD51_A is correct in principle, but the diode is rendered non-functional by the missing anode connection. This leaves the TACH_2 circuit without negative voltage protection, unlike the other three tachometer circuits which have proper negative clamps.
allspice-carah added the
bug
 label 2026-01-13 17:13:20 +00:00
allspice-carah added this to the Pilot Release milestone 2026-01-13 17:13:26 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something doesn't work
dfm
Design for manufacturing consideration
documentation
Improvements to supporting documentation
eol
End of life component
feature
New feature or request
firmware
Requires FW attention
layout
Requires PCB layout attention
mechanical
Requires ME attention
priority/1 - idea
Nice to have or idea to capture
priority/2 - low
Should be resolved eventually
priority/3 - medium
Should be resolved before product launch
priority/4 - high
Should be resolved before next release
priority/5 - critical
Must be fixed immediately - blocking item
bug
Something doesn't work
dfm
Design for manufacturing consideration
documentation
Improvements to supporting documentation
eol
End of life component
feature
New feature or request
firmware
Requires FW attention
layout
Requires PCB layout attention
mechanical
Requires ME attention
priority/1 - idea
Nice to have or idea to capture
priority/2 - low
Should be resolved eventually
priority/3 - medium
Should be resolved before product launch
priority/4 - high
Should be resolved before next release
priority/5 - critical
Must be fixed immediately - blocking item
No Label
bug
Milestone
No items
No Milestone Pilot Release
Assignees
Clear assignees
allspice-admin-daniel allspice-alex AllSpiceAlice allspice-carah allspice-cuong allspice-eli allspice-hermes allspice-ian allspice-kyle allspice-nick allspice-thomas brendan coleblaeser daniel-allspice emmy jacqueline john-allspice justina MikaChanical nick PavelInPurchasing RevaReviewa rowan sam sherry shrikanth-allspice shrikanthup Tayawiggins valentina
No Assignees AllSpiceAlice
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: AllSpice-Demos/AI-DRCY-Review#5
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?