Hi Jacek, On 08/04/15 13:58, Jacek Anaszewski wrote: >>> --- a/Documentation/devicetree/bindings/leds/common.txt >>> >> +++ b/Documentation/devicetree/bindings/leds/common.txt >>> >> @@ -29,13 +29,15 @@ Optional properties for child nodes: >>> >> "ide-disk" - LED indicates disk activity >>> >> "timer" - LED flashes at a fixed, configurable rate >>> >> >>> >> -- max-microamp : maximum intensity in microamperes of the LED >>> >> - (torch LED for flash devices) >>> >> -- flash-max-microamp : maximum intensity in microamperes of the >>> >> - flash LED; it is mandatory if the LED should >>> >> - support the flash mode >>> >> -- flash-timeout-us : timeout in microseconds after which the flash >>> >> - LED is turned off >>> >> +- max-microamp : Maximum intensity in microamperes of the LED >>> >> + (torch LED for flash devices). If omitted this will >>> >> default >>> >> + to the maximum current allowed by the device. >>> >> +- flash-max-microamp : Maximum intensity in microamperes of the flash >>> >> LED. >>> >> + If omitted this will default to the maximum >>> >> + current allowed by the device. >>> >> +- flash-timeout-us : Timeout in microseconds after which the flash >>> >> + LED is turned off. If omitted this will default to >>> >> the >>> >> + maximum timeout allowed by the device. >> > >> > Sorry about late comments on that, but since we can still change these >> > properties and it seems we're going to do that, I'd like throw in my >> > few preferences on the colour of this bike... >> > >> > IMO "max-microamp" is a poor property name, how about: >> > >> > s/max-microamp/led-max-current-ua, >> > s/flash-max-microamp/flash-max-current-ua, >> > >> > so we have more consistent set of properties like: >> > >> > led-max-current-ua >> > flash-max-current-ua >> > flash-timeout-us > > The "-microamp' suffix is consistent with regulator bindings. > Please refer to [1].
OK, in a perfect world we would have clean and consistent notation of units. If it's acked let's leave it, I didn't know it was, sorry about that. When I read yesterday Documentation/devicetree/bindings/leds/common.txt the set of new properties looked rather sloppy, especially "max-microamp" looked incomplete to me, as if the subject was missing. Anyway, I'll just get used to it, let's complete this whole Flash/LED integration story. -- Thanks, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
