Hi Andrew, On 25/08/15 15:18, Andrew Lunn wrote: > On Tue, Aug 25, 2015 at 02:34:00PM +0300, Tomi Valkeinen wrote: >> This patch adds basic support for a kernel driver to get a LED device. >> This will be used by the led-backlight driver. >> >> Only OF version is implemented for now, and the behavior is similar to >> PWM's of_pwm_get() and pwm_put(). > > Hi Tomi > > Is this the correct way to do it? I would of expected an xlate > function. I just sent v2, but I don't use xlate there. If I understand the purpose of xlate (in pwm, for example) correctly, xlate is a function in the pwm chip to allow custom bindings for the pwm outputs from that pwm chip. The problem with LEDs is that there's no "LED chip". Each LED is modelled as individual device, without a well defined parent. Thus there's no place to add such an xlate function. Do you have any thoughts on what the xlate for LEDs should look like? Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
