Attention is currently required from: Raul Rangel, Furquan Shaikh, Martin Roth, Eric Peers.
3 comments:
Commit Message:
Patch Set #1, Line 14: TP183
it was unused. Ought to comment as such. https://b.corp.google. […]
Marked it as NC in board ID 1.
Patch Set #1, Line 20: TEST=Build Guybrush mainboard.
probably need a boot board id #1 as well and confirm Speaker and reset are working. […]
Done
File src/mainboard/google/guybrush/variants/guybrush/gpio.c:
1. […]
There is some history involved. Initially EN_SPKR was tied to a 3.3 V GPIO whereas the speaker amplifier operates at 1.8 V level. To handle that, a rework was performed and the board version was incremented to identify the boards with that rework.
Later we realized that board version was not the right approach to identify the rework. Hence decided to reclaim the board version 2 for the upcoming hardware build.
So the speaker is enabled by default on boards with or without audio reworks. This is a safe assumption since the rework is done in all the boards (with board ID1).
To view, visit change 55148. To unsubscribe, or for help writing mail filters, visit settings.