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
Attachment:
pgpPvZR4hZ3k4.pgp
Description: PGP signature