* [ISSUE] ddcutil: udev rule
@ 2021-03-01 10:46 dkwo
2021-03-11 19:55 ` ericonr
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: dkwo @ 2021-03-01 10:46 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1390 bytes --]
New issue by dkwo on void-packages repository
https://github.com/void-linux/void-packages/issues/29152
Description:
With `kernel 5.11` and `ddcutil 1.0.1` it's now possible to control external display connected via docking station.
When running `ddcutil`, at the moment it is necessary to use `sudo/doas`, as no rules are in place.
In fact, `ddctutil environment` suggests:
```
Configuration suggestions:
Issue:
Group i2c does not exist.
Suggestion:
Create group i2c. To create group i2c, use command:
sudo groupadd --system i2c
Assign /dev/i2c-N devices to group i2c by adding a rule to /etc/udev/rules.d
Add the current user to group i2c:
sudo usermod -G i2c -a <username>
After this, you will have to logout and login again.
The changes to the user's group list are not read until a new login.
```
Is it fine to ship a new `udev` rule with `ddcutil`, something like the suggested
```
$ cat /usr/share/ddcutil/data/45-ddcutil-i2c.rules
# On some distributions, package i2c-tools provides a udev rule.
# For example, on Ubuntu, see 40-i2c-tools.rules.
# Assigns the i2c devices to group i2c, and gives that group RW access:
KERNEL=="i2c-[0-9]*", GROUP="i2c", MODE="0660"
# Gives everyone RW access to the /dev/i2c devices:
# KERNEL=="i2c-[0-9]*", MODE="0666"
```
Or what is the correct approach?
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: ddcutil: udev rule
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
@ 2021-03-11 19:55 ` ericonr
2021-03-12 10:21 ` dkwo
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-03-11 19:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 240 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/29152#issuecomment-797007651
Comment:
Shipping an udev rule sounds like a good idea, though I'm not sure what group we should be using.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: ddcutil: udev rule
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
2021-03-11 19:55 ` ericonr
@ 2021-03-12 10:21 ` dkwo
2021-03-12 15:20 ` ericonr
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: dkwo @ 2021-03-12 10:21 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 165 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/issues/29152#issuecomment-797391935
Comment:
Is `i2c` not appropriate?
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: ddcutil: udev rule
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
2021-03-11 19:55 ` ericonr
2021-03-12 10:21 ` dkwo
@ 2021-03-12 15:20 ` ericonr
2021-03-15 9:20 ` dkwo
2021-08-27 17:39 ` [ISSUE] [CLOSED] " ericonr
4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-03-12 15:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 229 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/29152#issuecomment-797557385
Comment:
Theoretically we want to use `_` prefixed group names for those created by packages...
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: ddcutil: udev rule
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
` (2 preceding siblings ...)
2021-03-12 15:20 ` ericonr
@ 2021-03-15 9:20 ` dkwo
2021-08-27 17:39 ` [ISSUE] [CLOSED] " ericonr
4 siblings, 0 replies; 6+ messages in thread
From: dkwo @ 2021-03-15 9:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 583 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/issues/29152#issuecomment-799257567
Comment:
Actually, I think this should better be done by the package `i2c-tools`, which is a dependency, as done in Debian and others, see e.g.
```
i2c-tools (3.0.2-4) unstable; urgency=low
* Add ${misc:Depends} to all binaries.
* Create an i2c group on install, and add a udev file to set the group
and the mode of /dev/i2c-* nodes.
-- Aurelien Jarno <aurel32@debian.org> Thu, 14 Jan 2010 06:39:06 +0100
```
I can do that later.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [ISSUE] [CLOSED] ddcutil: udev rule
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
` (3 preceding siblings ...)
2021-03-15 9:20 ` dkwo
@ 2021-08-27 17:39 ` ericonr
4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2021-08-27 17:39 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1393 bytes --]
Closed issue by dkwo on void-packages repository
https://github.com/void-linux/void-packages/issues/29152
Description:
With `kernel 5.11` and `ddcutil 1.0.1` it's now possible to control external display connected via docking station.
When running `ddcutil`, at the moment it is necessary to use `sudo/doas`, as no rules are in place.
In fact, `ddctutil environment` suggests:
```
Configuration suggestions:
Issue:
Group i2c does not exist.
Suggestion:
Create group i2c. To create group i2c, use command:
sudo groupadd --system i2c
Assign /dev/i2c-N devices to group i2c by adding a rule to /etc/udev/rules.d
Add the current user to group i2c:
sudo usermod -G i2c -a <username>
After this, you will have to logout and login again.
The changes to the user's group list are not read until a new login.
```
Is it fine to ship a new `udev` rule with `ddcutil`, something like the suggested
```
$ cat /usr/share/ddcutil/data/45-ddcutil-i2c.rules
# On some distributions, package i2c-tools provides a udev rule.
# For example, on Ubuntu, see 40-i2c-tools.rules.
# Assigns the i2c devices to group i2c, and gives that group RW access:
KERNEL=="i2c-[0-9]*", GROUP="i2c", MODE="0660"
# Gives everyone RW access to the /dev/i2c devices:
# KERNEL=="i2c-[0-9]*", MODE="0666"
```
Or what is the correct approach?
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-08-27 17:39 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-01 10:46 [ISSUE] ddcutil: udev rule dkwo
2021-03-11 19:55 ` ericonr
2021-03-12 10:21 ` dkwo
2021-03-12 15:20 ` ericonr
2021-03-15 9:20 ` dkwo
2021-08-27 17:39 ` [ISSUE] [CLOSED] " ericonr
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).