On Mon, 24 Jun 2024, Heiner Kallweit wrote: > On 20.06.2024 11:09, Lee Jones wrote: > > On Thu, 13 Jun 2024, Lee Jones wrote: > > > >> If both set_brightness functions return -ENOTSUPP, then the LED doesn't > >> support setting a fixed brightness value, and the error message isn't > >> helpful. This can be the case e.g. for LEDs supporting a specific hw > >> trigger only. > >> > >> Pinched the subject line and commit message from Heiner: > >> Link: https://lore.kernel.org/all/44177e37-9512-4044-8991-bb23b184bf37@xxxxxxxxx/ > >> > >> Reworked the function to provide Heiner's required semantics whilst > >> simultaneously increasing readability and flow. > >> > >> Cc: Pavel Machek <pavel@xxxxxx> > >> Cc: linux-leds@xxxxxxxxxxxxxxx > >> Suggested-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > > > > Heiner, you good with this solution? > > > > A Tested-by or Reviewed-by would be good if you have the time. > > > >> Signed-off-by: Lee Jones <lee@xxxxxxxxxx> > > Reviewed-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> Added, thank you. -- Lee Jones [李琼斯]