|
|
Re: [PATCH v4 01/10] PWM: add pwm framework support |
* 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
Attachment:
pgp0pw_FOv0rk.pgp
Description: PGP signature
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
[Linux ARM (vger)] [Linux ARM MSM] [Linux Omap] [Linux Arm] [Linux Tegra] [Fedora ARM] [eCos] [Linux Fastboot] [Gcc Help] [Git] [DCCP] [IETF Announce] [Security] [PDAs] [Linux] [Linux MIPS] [Yosemite Campsites] [Photos]
![]() |
![]() |