A couple of trivial editorial errors, otherwise,

Reviewed-by: Shawn Guo <[email protected]>

On Wed, Mar 28, 2012 at 04:33:43PM +0200, Thierry Reding wrote:
> From: Sascha Hauer <[email protected]>
> 
> This patch adds framework support for PWM (pulse width modulation) devices.
> 
> The is a barebone PWM API already in the kernel under include/linux/pwm.h,

s/The/There

> but it does not allow for multiple drivers as each of them implements the
> pwm_*() functions.
> 
...

> +/**
> + * pwm_request() - request a PWM device
> + * @pwm: global PWM device index

s/pwm/pwm_id

> + * @label: PWM device label
> + */
> +struct pwm_device *pwm_request(int pwm_id, const char *label)
...

> +/**
> + * struct pwm_chip - abstract a PWM
> + * @label: for diagnostics
> + * @owner: helps prevent removal of modules exporting active PWMs

The doc needs an update.

Regards,
Shawn

> + * @ops: The callbacks for this PWM
> + */
> +struct pwm_chip {
> +     int                     pwm_id;
> +     const char              *label;
> +     struct pwm_ops          *ops;
> +};
> +
> +int pwmchip_add(struct pwm_chip *chip);
> +int pwmchip_remove(struct pwm_chip *chip);
> +#endif
> +
>  #endif /* __LINUX_PWM_H */
> -- 
> 1.7.9.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to