On Wed, Mar 28, 2012 at 04:33:46PM +0200, Thierry Reding wrote: > + static struct pwm_lookup board_pwm_lookup[] = { > + PWM_LOOKUP("tegra-pwm", 0, "pwm-backlight"), > + }; The clock and regulator APIs namespace the consumers by struct device - might this not be sensible here? pwm_get() does already take the device as an argument. It'd feel safer, and for example there's plenty of phones out there with two backlit displays... > + * pwm_get() - look up and request a PWM device > + * @dev: device for PWM consumer > + * @provider: name of provider PWM chip > + * @index: per-chip index of PWM to request The documentation is out of sync with the signature.
Attachment:
signature.asc
Description: Digital signature