[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