led: trig: netdev: Fix requesting offload device
authorAndrew Lunn <andrew@lunn.ch>
Tue, 8 Aug 2023 21:04:33 +0000 (23:04 +0200)
committerJakub Kicinski <kuba@kernel.org>
Fri, 11 Aug 2023 00:35:00 +0000 (17:35 -0700)
commit7df1f14c04cbb1950e79c19793420f87227c3e80
tree2a3714a3ff26f55e7bde0c6d08ac5380d684d9ed
parent215c44fa69d7a873e058b2e1c451c12025d94bee
led: trig: netdev: Fix requesting offload device

When the netdev trigger is activates, it tries to determine what
device the LED blinks for, and what the current blink mode is.

The documentation for hw_control_get() says:

 * Return 0 on success, a negative error number on failing parsing the
 * initial mode. Error from this function is NOT FATAL as the device
 * may be in a not supported initial state by the attached LED trigger.
 */

For the Marvell PHY and the Armada 370-rd board, the initial LED blink
mode is not supported by the trigger, so it returns an error. This
resulted in not getting the device the LED is blinking for. As a
result, the device is unknown and offloaded is never performed.

Change to condition to always get the device if offloading is
supported, and reduce the scope of testing for an error from
hw_control_get() to skip setting trigger internal state if there is an
error.

Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Tested-by: Daniel Golle <daniel@makrotopia.org>
Link: https://lore.kernel.org/r/20230808210436.838995-2-andrew@lunn.ch
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/leds/trigger/ledtrig-netdev.c