On Tue, Aug 06, 2019 at 06:39:36PM +0530, Manivannan Sadhasivam wrote: ... > +Required Properties: > +- compatible: Should be "sony,imx290" > +- reg: I2C bus address of the device > +- clocks: Reference to the xclk clock. > +- clock-names: Should be "xclk". > +- clock-frequency: Frequency of the xclk clock. > +- vdddo-supply: Sensor digital IO regulator. > +- vdda-supply: Sensor analog regulator. > +- vddd-supply: Sensor digital core regulator. Could you also add the link-frequencies property, please? -- Sakari Ailus