Hi Hans,
On 17.06.22 13:07, Hans de Goede wrote:
Thank you for providing this info. Can you please give
the attached patch series a try, this includes Stefan's 1/2 patch
and replaces Stefan's 2/2 patch.
This will hopefully fix the double key-presses for you, while
also keeping everything working for Stefan without requiring
a module option or DMI quirks.
Stefan can you also give this series a try please?
Works for me, almost out of the box.
I need to enable "report_key_events=1" in the video module, then the
panasonic-acpi module starts reporting brightness up/down keys.
Volume and mute keys work without manual changes.
(I tested against 5.18.2 because that's what was already prepared. That
old machine takes quite some time, even to just compile the platform/x86
subdirectory ;-) but I don't think this is relevant. If you think it is,
I can also test against latest 5.19-rc code)
Looking at this has also brought up an unrelated backlight question:
Kenneth, since you have acpi-video reporting keypresses you will
likely also have an acpi_video (or perhaps a native intel) backlight
under /sys/class/backlight and I noticed that panasonic-laptop
uncondirionally registers its backlight so you may very well end
up with 2 backlight controls under /sys/class/backlight, which
we generally try to avoid (so that userspace does not have to
guess which one to use).
Can you do:
ls /sys/class/backlight
toughbook:~ # ls -l /sys/class/backlight/
total 0
lrwxrwxrwx 1 root root 0 Jun 17 14:45 intel_backlight ->
../../devices/pci0000:00/0000:00:02.0/drm/card0/card0-LVDS-1/intel_backlight
lrwxrwxrwx 1 root root 0 Jun 17 14:49 panasonic ->
../../devices/virtual/backlight/panasonic
and let me know the output?
Also if there are 2 backlights there then please do:
cat /sys/class/backlight/<name>/max_brightness
to find out the range (0-value)
toughbook:/sys/class/backlight # grep . */max_brightness
intel_backlight/max_brightness:19531
panasonic/max_brightness:255
and then try if they both work by doing:
echo $number > /sys/class/backlight/<name>/brightness
with different $number values in the range and see
if this actually changes the brightness.
intel_backlight: does not work
panasonic: does work
While we are at it, Stefan can you do the same please?
See above.
But hey, this is an i855GM graphics chip, I'm happy if it is still
working *at all* (for example I need to avoid the xf86-intel driver and
use the modesetting driver instead to get a usable sytstem)
And I'm totally happy if all I have to do in the future is a
option video report_key_events=1
modprobe.conf file ;-)
Best regards,
Stefan
--
Stefan Seyfried
"For a successful technology, reality must take precedence over
public relations, for nature cannot be fooled." -- Richard Feynman
