HI Laurent, On Wed, May 8, 2013 at 7:31 AM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > Hi Prabhakar, > > Thank you for the patch. > > On Friday 03 May 2013 12:21:59 Prabhakar Lad wrote: >> From: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> >> >> add OF support for the mt9p031 sensor driver. >> Alongside this patch sorts the header inclusion alphabetically. >> >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> >> Cc: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> >> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> >> Cc: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> >> Cc: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> >> Cc: Sakari Ailus <sakari.ailus@xxxxxx> >> Cc: Grant Likely <grant.likely@xxxxxxxxxxxx> >> Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> >> Cc: Rob Herring <rob.herring@xxxxxxxxxxx> >> Cc: Rob Landley <rob@xxxxxxxxxxx> >> Cc: Arnd Bergmann <arnd@xxxxxxxx> >> Cc: devicetree-discuss@xxxxxxxxxxxxxxxx >> Cc: davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx >> Cc: linux-doc@xxxxxxxxxxxxxxx >> Cc: linux-kernel@xxxxxxxxxxxxxxx >> --- >> Changes for v2: >> 1: Used '-' instead of '_' for device properties. >> 2: Specified gpio reset pin as phandle in device node. >> 3: Handle platform data properly even if kernel is compiled with >> devicetree support. >> 4: Used dev_* for messages in drivers instead of pr_*. >> 5: Changed compatible property to "aptina,mt9p031" and "aptina,mt9p031m". >> 6: Sorted the header inclusion alphabetically and fixed some minor code >> nits. >> >> Changes for v3: >> 1: Dropped check if gpio-reset is valid. >> 2: Fixed some code nits. >> 3: Included a reference to the V4L2 DT bindings documentation. >> >> .../devicetree/bindings/media/i2c/mt9p031.txt | 40 +++++++++++++ >> drivers/media/i2c/mt9p031.c | 60 ++++++++++++++++- >> 2 files changed, 97 insertions(+), 3 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/media/i2c/mt9p031.txt >> >> diff --git a/Documentation/devicetree/bindings/media/i2c/mt9p031.txt >> b/Documentation/devicetree/bindings/media/i2c/mt9p031.txt new file mode >> 100644 >> index 0000000..e740541 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/media/i2c/mt9p031.txt >> @@ -0,0 +1,40 @@ >> +* Aptina 1/2.5-Inch 5Mp CMOS Digital Image Sensor >> + >> +The Aptina MT9P031 is a 1/2.5-inch CMOS active pixel digital image sensor >> with +an active array size of 2592H x 1944V. It is programmable through a >> simple +two-wire serial interface. >> + >> +Required Properties : >> +- compatible : value should be either one among the following >> + (a) "aptina,mt9p031" for mt9p031 sensor >> + (b) "aptina,mt9p031m" for mt9p031m sensor >> + >> +- input-clock-frequency : Input clock frequency. >> + >> +- pixel-clock-frequency : Pixel clock frequency. >> + >> +Optional Properties : >> +-gpio-reset: Chip reset GPIO > > According to Documentation/devicetree/bindings/gpio/gpio.txt, this should be > "gpios" or "reset-gpios". > OK I'll make it as "reset-gpios" Regards, --Prabhakar Lad -- 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