Messages in this thread | | | Date | Tue, 10 Jul 2012 10:00:03 +0200 | From | Thierry Reding <> | Subject | Re: [PATCH] pwm: add lpc32xx pwm support |
| |
On Tue, Jul 10, 2012 at 09:56:08AM +0200, Roland Stigge wrote: > Hi, > > On 07/10/2012 08:48 AM, Thierry Reding wrote: > >> --- /dev/null +++ > >> b/Documentation/devicetree/bindings/pwm/lpc32xx-pwm.txt @@ -0,0 > >> +1,12 @@ +LPC32XX PWM controller + +Required properties: +- > >> compatible: should be "nxp,lpc3220-pwm" > > > > Does the compatible have to be lpc3220-pwm? Can't it be lpc32xx-pwm > > to match the driver and binding names? > > When creating the other NXP LPC compatible strings, we agreed on > taking the first LPC32xx chip, i.e., lpc3220, as prefix. (There are 4 > of them, -20, -30, -40, -50, most of them supporting things available > in lpc3220.) > > So lpc3220-pwm looks good here.
I see, that makes sense.
Thierry [unhandled content-type:application/pgp-signature] | |