can: mark led trigger as broken
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 27 Jun 2018 09:20:52 +0000 (11:20 +0200)
committerJacek Anaszewski <jacek.anaszewski@gmail.com>
Thu, 28 Jun 2018 19:34:45 +0000 (21:34 +0200)
The driver was suggested for deletion as it implements a subset of the
netdev trigger. It's in the way for further cleanups in the trigger
code but doesn't get an Ack by someone who can actually test and confirm
that the netdev trigger works for can devices. So marking as broken to
get forward with the cleanups.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>
drivers/net/can/Kconfig

index 2cb75988b3280400df7d1f4c0fb8040b60f6fb5f..7cdd0cead693dc5d78565f5fd988c6c25c7a2740 100644 (file)
@@ -73,6 +73,12 @@ config CAN_CALC_BITTIMING
 config CAN_LEDS
        bool "Enable LED triggers for Netlink based drivers"
        depends on LEDS_CLASS
+       # The netdev trigger (LEDS_TRIGGER_NETDEV) should be able to do
+       # everything that this driver is doing. This is marked as broken
+       # because it uses stuff that is intended to be changed or removed.
+       # Please consider switching to the netdev trigger and confirm it
+       # fulfills your needs instead of fixing this driver.
+       depends on BROKEN
        select LEDS_TRIGGERS
        ---help---
          This option adds two LED triggers for packet receive and transmit