Skip to content

periph/gpio: Add open drain as a push-pull option#4750

Closed
jnohlgard wants to merge 2 commits intoRIOT-OS:masterfrom
jnohlgard:pr/gpio-open-drain
Closed

periph/gpio: Add open drain as a push-pull option#4750
jnohlgard wants to merge 2 commits intoRIOT-OS:masterfrom
jnohlgard:pr/gpio-open-drain

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

Added GPIO_OPEN_DRAIN for configuring outputs as open drain. This relates to the discussion in #4428 and I also needed to configure an open drain output.

Also included is a change for the kinetis family to add open drain support.

@jnohlgard jnohlgard added Platform: ARM Platform: This PR/issue effects ARM-based platforms Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. Area: drivers Area: Device drivers labels Feb 5, 2016
@jnohlgard jnohlgard added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 5, 2016
@haukepetersen
Copy link
Copy Markdown
Contributor

see also #4472. By now, I would opt for making the push-pull configuration available by defining a direction option GPIO_DIR_OUT_OD instead of a pull resistor option.

@jnohlgard
Copy link
Copy Markdown
Member Author

Closing in favour of #4862

@jnohlgard jnohlgard closed this Mar 2, 2016
@jnohlgard jnohlgard deleted the pr/gpio-open-drain branch April 6, 2017 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Platform: ARM Platform: This PR/issue effects ARM-based platforms Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants