]> git.baikalelectronics.ru Git - kernel.git/commit
platform/x86: pcengines-apuv2: revert wiring up simswitch GPIO as LED
authorFlorian Eckert <fe@dev.tdt.de>
Mon, 13 Jul 2020 08:50:10 +0000 (10:50 +0200)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 15 Jul 2020 09:49:12 +0000 (12:49 +0300)
commit6e6c593958d27447dd97f40fdd7d64571837befa
tree481ec4cb2414bad531eb6ab6108265dedb9ef9a3
parent54572a630fc4c14a3f77a433a945c19376cde8e3
platform/x86: pcengines-apuv2: revert wiring up simswitch GPIO as LED

This reverts commit 55219bef399fab00182e6f01da61178577a29d7a.

Explanation why this does not work:
This change connects the simswap to the LED subsystem of the kernel.
From my point of view, it's nonsense. If we do it this way, then this
can be switched relatively easily via the LED subsystem (trigger:
none/default-on) and that is dangerous! If this is used, it would be
unfavorable, since there is also another trigger (trigger:
heartbeat/netdev).

Therefore, this simswap GPIO should remain in the GPIO
subsystem and be switched via it and not be connected to the LED
subsystem. To avoid the problems mentioned above. The LED subsystem is
not made for this and it is not a good compromise, but rather dangerous.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/platform/x86/pcengines-apuv2.c