pinout-xyz / Pinout.xyz

Source files for the Raspberry Pi Pinout documentation website.
http://pinout.xyz/
Creative Commons Attribution Share Alike 4.0 International
695 stars 197 forks source link

Fix pinout of button Y on unicorn hat mini #436

Closed please-insert-username-here closed 2 years ago

lurch commented 2 years ago

Hmmm? The GPIO numbers shown on https://pinout.xyz/pinout/unicorn_hat_mini (5, 6, 16, 20) seem to match up with what https://github.com/pimoroni/unicornhatmini-python/blob/master/library/unicornhatmini/__init__.py says? :man_shrugging:

Gadgetoid commented 2 years ago

And... yet... https://github.com/pimoroni/unicornhatmini-python/blob/master/examples/buttons.py

https://github.com/pimoroni/unicornhatmini-python/blob/6fbeed56433a7474de1005a5ef1f44b194051be8/examples/buttons.py#L20-L28

In many cases we moved away from the i2s pin so we wouldn't break support for audio boards, but I don't recall if this was one of the boards we did that with. Might have to dig up the schematic. Right now the buttons example and library disagreeing is... odd.

Gadgetoid commented 2 years ago

This does - in fact - match the schematic. The library is wrong :grimacing: - or perhaps outdated since some boards got a respin to change the Y button.

lurch commented 2 years ago

Pesky hardware not matching up with what the software is telling it to do... :rofl: