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

Add support for MCH2022 badge #203

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cyber-murmel
Copy link
Contributor

@smunaut
Copy link

smunaut commented Jul 22, 2022

Is there anyway to specify some pins are input only, or open-drain outputs ?
Like LCD_MODE is an input and shouldn't be driven.
The IRQ and LCD reset, LCD chipselect are also open-drain and shouldn't be actively driven high.

@whitequark
Copy link
Member

Is there anyway to specify some pins are input only, or open-drain outputs ?

dir="i", dir="oe"

@smunaut
Copy link

smunaut commented Jul 22, 2022

Ok great thanks for the tip.
Then I'd probably include that to make sure things are driven the way they are meant to 😁

@cyber-murmel
Copy link
Contributor Author

cyber-murmel commented Jul 22, 2022

Should only these three addition pins be constraint in their direction and drive strength or all pins according to the schematic?

I can imagine that someone might want to use the 8 pins of the data bus going to the display as additional inputs pins for example, without having to use a different board definition.

@smunaut
Copy link

smunaut commented Jul 22, 2022

I don't see any issue in doing only those 3.
They're the only ones that are also connected to the ESP and so can create contention and so it's better if they're driven by default the correct way.
The others like the LCD data are only connected to the LCD and the LCD is hardwired in write mode so it'll never drive the lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants