On 7/15/20 11:50 PM, Luca Ceresoli wrote:
Hi Sowjanya,
On 15/07/20 06:24, Sowjanya Komatineni wrote:
This patch adds IMX274 optional external clock input and voltage
supplies to device tree bindings.
Signed-off-by: Sowjanya Komatineni <skomatineni@xxxxxxxxxx>
---
Documentation/devicetree/bindings/media/i2c/imx274.txt | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/media/i2c/imx274.txt b/Documentation/devicetree/bindings/media/i2c/imx274.txt
index 80f2e89..ee427f5 100644
--- a/Documentation/devicetree/bindings/media/i2c/imx274.txt
+++ b/Documentation/devicetree/bindings/media/i2c/imx274.txt
@@ -13,6 +13,11 @@ Required Properties:
Optional Properties:
- reset-gpios: Sensor reset GPIO
+- clocks: Reference to the xclk clock.
+- clock-names: Should be "xclk".
Not sure where the "xclk" name comes from, the datasheet I have calls
the pin "CKIN". Maybe using the same name as the datasheet is better?
Other than that looks good.
Thanks Luca. Using xclk as its external clock to IMX274.
Datasheet uses it as INCK. Will update in v2 to use "INCK" as referred
in its datasheet.
