Hello, round 3 for this series. I have now removed 'pll-bypass' property as suggested by Sakari, and restructured bindings description to list hsync and vsync properties as required. Changelog reported below. Thanks j v2->v3: - Drop 'pll-bypass' property - Make 'plck-hb-disable' a boolean optional property - List 'hsync' and 'vsync' polarities as required endpoint properties - Restructured 'ov7670_parse_dt()' function to reflect the above changes v1->v2: - Split bindings description and implementation - Addressed Sakari's comments on v1 - Check for return values of register writes in set_fmt() - TODO: decide if "pll-bypass" should be an OF property. Jacopo Mondi (2): media: dt-bindings: Add OF properties to ov7670 v4l2: i2c: ov7670: Implement OF mbus configuration .../devicetree/bindings/media/i2c/ov7670.txt | 18 +++- drivers/media/i2c/ov7670.c | 102 ++++++++++++++++++--- 2 files changed, 104 insertions(+), 16 deletions(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html