[PATCH v6 05/17] pwm: Add device tree support

[Shawn Guo]

On Tue, Apr 10, 2012 at 05:06:28PM +0200, Thierry Reding wrote:
> This patch adds helpers to support device tree bindings for the generic
> PWM API. Device tree binding documentation for PWM controllers is also
> provided.
> 
> Signed-off-by: Thierry Reding <thierry.reding at avionic-design.de>
> Acked-by: Arnd Bergmann <arnd at arndb.de>
> ---
> Changes in v6:
> - don't override error code in of_pwm_simple_xlate()
> - no longer export of_pwm_request(), only pwm_get() should be used

Ok, I read this here ...

> +/**
> + * of_pwm_request() - request a PWM via the PWM framework
> + * @np: device node to get the PWM from
> + * @con_id: consumer name
> + *
> + * Returns the PWM device parsed from the phandle and index specified in the
> + * "pwms" property of a device tree node or a negative error-code on failure.
> + * Values parsed from the device tree are stored in the returned PWM device
> + * object.
> + *
> + * If con_id is NULL, the first PWM device listed in the "pwms" property will
> + * be requested. Otherwise the "pwm-names" property is used to do a reverse
> + * lookup of the PWM index. This also means that the "pwm-names" property
> + * becomes mandatory for devices that look up the PWM device via the con_id
> + * parameter.
> + */
> +static struct pwm_device *of_pwm_request(struct device_node *np,
> +					 const char *con_id)
> +{
...
> +}
> +EXPORT_SYMBOL(of_pwm_request);

... so this line should be removed.  Otherwise,

Reviewed-by: Shawn Guo <shawn.guo at linaro.org>