[PATCH v4 01/10] PWM: add pwm framework support

Thierry Reding thierry.reding at avionic-design.de
Wed Mar 14 16:57:34 EDT 2012


* Lars-Peter Clausen wrote:
> On 03/14/2012 04:56 PM, Thierry Reding wrote:
> > [...]
> > +struct pwm_chip {
> > +	int			pwm_id;
> > +	const char		*label;
> > +	struct pwm_ops		*ops;
> 
> Nothing major, but if you are going for another round it would be good to see
> ops made const.

I'll go for another round because I just noticed that I still have two open
TODO items that I forgot to fix from last round's review.

I'll take a note to make it const.

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120314/5fdebc1d/attachment.sig>


More information about the linux-arm-kernel mailing list