Skip to content

Added documentation for the rpi_pfio platform.#2597

Merged
balloob merged 3 commits into
home-assistant:nextfrom
basschipper:pfio
May 10, 2017
Merged

Added documentation for the rpi_pfio platform.#2597
balloob merged 3 commits into
home-assistant:nextfrom
basschipper:pfio

Conversation

@basschipper
Copy link
Copy Markdown
Contributor

Description:
Support for the PiFace Digital I/O module (Raspberry Pi add on board)
This is basically a port of the rpi_gpio component.

Pull request in home-assistant (if applicable): home-assistant/core#7494

Copy link
Copy Markdown
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks really good and only needed change is to update the release number before release.

If a better link is available I would recommend linking to the maker of the product instead of Element 14.

footer: true
logo: raspberry-pi.png
ha_category: Binary Sensor
ha_release: 0.44.3
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.45 will be the next release.

ha_release: 0.44.3
---

The `rpi_pfio` binary sensor platform allows you to read sensor values of the [PiFace Digital I/O](https://www.element14.com/community/docs/DOC-69001) .
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could the link be pointed to the manufacture instead of Element 14?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Element14 seems to be the manufacturer of the PiFace.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh. Well then my bad :)

@Landrash Landrash added the new-integration This PR adds documentation for a new Home Assistant integration label May 8, 2017
Copy link
Copy Markdown
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go and can be merged when parent PR is merged.

@balloob balloob merged commit f90d929 into home-assistant:next May 10, 2017
@roland1970
Copy link
Copy Markdown

Hello Bas ,Is it possible to add more then 1 piface?
I mean to add the address-rule so that we can use more pifaces.
Or can you tell me what I must change make that possible .

@klaasnicolaas
Copy link
Copy Markdown
Member

klaasnicolaas commented Apr 17, 2020

Please don't comment on old pull requests. You can create an issue here instead: https://github.com/home-assistant/core/issues/new/choose

@home-assistant home-assistant locked as off-topic and limited conversation to collaborators Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

new-integration This PR adds documentation for a new Home Assistant integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants