lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 25 Aug 2020 17:17:01 +0200 From: Kévin L'hôpital <kevin.lhopital@...tlin.com> To: Maxime Ripard <maxime@...no.tech> Cc: linux-media@...r.kernel.org, mchehab@...nel.org, robh+dt@...nel.org, mark.rutland@....com, wens@...e.org, yong.deng@...ewell.com, p.zabel@...gutronix.de, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, paul.kocialkowski@...tlin.com, thomas.petazzoni@...tlin.com Subject: Re: [PATCH 2/7] dt-bindings: media: i2c: Add documentation for ov8865 Hello, Le Mon, 24 Aug 2020 18:59:10 +0200, Maxime Ripard <maxime@...no.tech> a écrit : > Hi, > > On Fri, Aug 21, 2020 at 04:59:30PM +0200, Kévin L'hôpital wrote: > > Add a documentation for the sensor ov8865 from Omnivision. > > > > Signed-off-by: Kévin L'hôpital <kevin.lhopital@...tlin.com> > > In order to ease the submission of both drivers, you should probably > split this series into two, one with the MIPI-CSI driver, and one with > the ov8865 driver. > Yes, you are right. I will do this. > > --- > > .../devicetree/bindings/media/i2c/ov8865.txt | 51 > > +++++++++++++++++++ 1 file changed, 51 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/media/i2c/ov8865.txt > > > > diff --git a/Documentation/devicetree/bindings/media/i2c/ov8865.txt > > b/Documentation/devicetree/bindings/media/i2c/ov8865.txt new file > > mode 100644 index 000000000000..ac5a662288de > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/i2c/ov8865.txt > > @@ -0,0 +1,51 @@ > > +* Omnivision OV8865 MIPI CSI-2 > > + > > +Required Properties: > > +- compatible: should be "ovti,ov8865" > > +- clocks: reference to the xclk input clock. > > +- clock-names: should be "xclk". > > +- DOVDD-supply: Digital I/O voltage supply, 2.8 volts > > +- AVDD-supply: Analog voltage supply, 2.8 volts > > +- AFVDD-supply: Analog voltage supply, 2.8 volts > > +- DVDD-supply: Digital core voltage supply, 1.2 volts > > +- reset-gpios: reference to the GPIO connected to the reset pin. > > + This is an active low signal to the OV8865. > > +- powerdown-gpios: reference to the GPIO connected to the > > powerdown pin. > > + This is an active low signal to the OV8865. > > +- rotation: as defined in > > + > > Documentation/devicetree/bindings/media/video-interfaces.txt, > > + valid values are 0 (sensor mounted upright) and 180 > > (sensor > > + mounted upside down). > > +- remote-endpoint: a phandle to the bus receiver's endpoint node. > > +- clock-lanes: should be set to <0> (clock lane on hardware lane > > 0). +- data-lanes: should be set to <4> (four CSI-2 lanes > > supported). + > > +The device node must contain one 'port' child node for its digital > > output video +port, in accordance with the video interface bindings > > defined in > > +Documentation/devicetree/bindings/media/video-interfaces.txt. > > Free form DT documentation is deprecated nowadays, you should be > doing a YAML schema instead (like the ov8856 driver). > All right, I will do a YAML schema. > Maxime Thank you very much for the review. Kévin -- Kevin L'Hopital, Bootlin Embedded Linux and kernel engineering https://bootlin.com
Powered by blists - more mailing lists