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. Thanks also for the review. Will have a second look when Alexandre posts an update. Thank you two! Roland -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
