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.

Sign up for GitHub

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

[gpio] Update README.md #9899

Merged
merged 3 commits into from
Feb 17, 2021
Merged

[gpio] Update README.md #9899

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8e4d3af
Update README.md
SebBoulet Jan 21, 2021
533892c
Update bundles/org.openhab.binding.gpio/README.md
SebBoulet Feb 8, 2021
bc36c06
Update bundles/org.openhab.binding.gpio/README.md
SebBoulet Feb 17, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 14 additions & 1 deletion bundles/org.openhab.binding.gpio/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,25 @@ sudo raspi-config

-> Interfacing Options --> Remote GPIO --> YES --> OK --> Finish

Note: if you are setting this up on a Raspberry Pi without `raspi-config` you can create the service config file manually:

```
sudo mkdir -p /etc/systemd/system/pigpiod.service.d/
sudo nano /etc/systemd/system/pigpiod.service.d/public.conf
```
[Service]
ExecStart=
ExecStart=/usr/bin/pigpiod
Comment on lines +32 to +33
Copy link
Member

Choose a reason for hiding this comment

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

Why are there two ExecStarts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because this is what the raspi-config script adds to the pigpiod config file and I copied verbatim (in case it matters).
https://github.com/RPi-Distro/raspi-config/blob/74885481d8f311ab3875b6601153132552b41d67/raspi-config#L1616

Copy link
Member

Choose a reason for hiding this comment

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

It seems an error to me. Can you test if it still works if you remove the empty ExecStart=?

Copy link
Member

Choose a reason for hiding this comment

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

It seems to be to override the default values. See https://bugzilla.redhat.com/show_bug.cgi?id=756787. So I guess it is to be sure no default settings are present. That would mean even if @SebBoulet would test this on his system and it works, it might not work for other people who have other configurations. So I would say this is legitimate use case.

Copy link
Member

Choose a reason for hiding this comment

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

I didn't find that info when doing a quick search. Thanks!

```
sudo systemctl daemon-reload
```
Now that Remote GPIO is enabled, get the daemon going:
```
sudo systemctl enable pigpiod
sudo systemctl start pigpiod
```

Set `host` to the address of the pi and the `port` to the port of pigpio (default: 8888).
In openHAB, set `host` to the address of the pi and the `port` to the port of pigpio (default: 8888).

## Channels

Expand Down