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, 10 Mar 2020 16:51:36 +0100 From: Robert Foss <robert.foss@...aro.org> To: Fabio Estevam <festevam@...il.com> Cc: ben.kao@...el.com, Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring <robh+dt@...nel.org>, Mark Rutland <mark.rutland@....com>, Matthias Brugger <matthias.bgg@...il.com>, "David S. Miller" <davem@...emloft.net>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Jonathan.Cameron@...wei.com, Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, linux-media <linux-media@...r.kernel.org>, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>, linux-kernel <linux-kernel@...r.kernel.org>, "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" <linux-arm-kernel@...ts.infradead.org>, linux-mediatek@...ts.infradead.org, Dongchun Zhu <dongchun.zhu@...iatek.com>, Tomasz Figa <tfiga@...omium.org> Subject: Re: [v1 1/3] media: dt-bindings: ov8856: Document YAML bindings Hey Fabio, Thanks for having a look at this series so quickly. On Tue, 10 Mar 2020 at 14:57, Fabio Estevam <festevam@...il.com> wrote: > > Hi Robert, > > On Tue, Mar 10, 2020 at 10:46 AM Robert Foss <robert.foss@...aro.org> wrote: > > > + ov8856: camera-sensor@10 { > > + compatible = "ovti,ov8856"; > > + reg = <0x10>; > > + reset-gpios = <&pio 111 GPIO_ACTIVE_HIGH>; > > Could you double check this is correct? Other OmniVision sensors have > reset-gpios as active low. I have tested this, unfortunately I don't have access to a ov8856 datasheet that includes this level of detail. But I have tested this. > > I suspect that the driver has also an inverted logic, so that's why it works. That could explain it still working. Let me have a look into the driver and see what it does. > > I don't have access to the datasheet though, so I am just guessing. Me neither unfortunately, if anyone does have a link for it, I would very much appreciate it.
Powered by blists - more mailing lists