Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible Pin Misconfiguration for PIN_10 (daisy pin 9) in Comments #10

Closed
harshnoiise opened this issue Feb 20, 2022 · 1 comment
Closed

Comments

@harshnoiise
Copy link

pub type SeedPin9 = hal::gpio::gpiob::PB4 <hal::gpio::Alternate<hal::gpio::AF0>>; // PIN_10, SPI1 MOSI

pub type SeedPin10 = hal::gpio::gpiob::PB5 <hal::gpio::Analog>; // PIN_11, SPI1 MISO

The code itself appears to be correct, but I think the associated comment for both lines is accidentally swapped?
I could be completely wrong bc I'm still very new to the embedded world and if so I'll close this asap!

But based off looking at the pinout diagram, it looks like SPI1 MISO is PIN_10/SeedPin9 and SPI1 MOSI is PIN_11/SeedPin10 yeah? If the comments are swapped, no big deal they're just comments, but can clear up some confusion and despair like a nub like me 🥲

Daisy Seed Pinout Reference

Spent the entire weekend trying to configure an SSD1306 display via SPI and unfortunately no luck, however it's forced me to study this codebase a lot and the pin out diagram relentlessly lmao.

@antoinevg
Copy link
Owner

My apologies for taking so long to get to this.

You are entirely correct, thank you!

I've updated the comments to reflect this: 1ffe4e0

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

No branches or pull requests

2 participants