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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 9 Jan 2021 03:47:32 +0200 From: Laurent Pinchart <laurent.pinchart@...asonboard.com> To: Robert Foss <robert.foss@...aro.org> Cc: agross@...nel.org, bjorn.andersson@...aro.org, todor.too@...il.com, mchehab@...nel.org, robh+dt@...nel.org, catalin.marinas@....com, will@...nel.org, shawnguo@...nel.org, leoyang.li@....com, geert+renesas@...der.be, arnd@...db.de, Anson.Huang@....com, michael@...le.cc, agx@...xcpu.org, max.oss.09@...il.com, linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, AngeloGioacchino Del Regno <kholk11@...il.com>, Andrey Konovalov <andrey.konovalov@...aro.org>, Tomasz Figa <tfiga@...omium.org>, Azam Sadiq Pasha Kapatrala Syed <akapatra@...cinc.com>, Sarvesh Sridutt <Sarvesh.Sridutt@...rtwirelesscompute.com> Subject: Re: [PATCH v1 16/17] arm64: dts: sdm845-db845c: Add CAMSS ISP node Hi Rob, Thank you for the patch. The subject line doesn't match the patch. On Fri, Jan 08, 2021 at 01:04:28PM +0100, Robert Foss wrote: > Add regulators and camss DT node. > > Signed-off-by: Robert Foss <robert.foss@...aro.org> > --- > arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts > index a943b3f353ce..7bad0515345e 100644 > --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts > +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts > @@ -294,6 +294,9 @@ vreg_s7a_1p025: smps7 { > regulator-max-microvolt = <1028000>; > }; > > + vdda_mipi_csi0_0p9: > + vdda_mipi_csi1_0p9: > + vdda_mipi_csi2_0p9: No need for new labels, you can simply use vreg_l1a_0p875 below. > vreg_l1a_0p875: ldo1 { > regulator-min-microvolt = <880000>; > regulator-max-microvolt = <880000>; > @@ -1106,6 +1109,15 @@ &cci { > status = "okay"; > }; > > +&camss { > + vdda-csi0-supply = <&vdda_mipi_csi0_0p9>; > + vdda-csi1-supply = <&vdda_mipi_csi1_0p9>; > + vdda-csi2-supply = <&vdda_mipi_csi2_0p9>; > + > + status = "disabled"; This isn't needed. > + Extra blank line. > +}; > + > &cci_i2c0 { > camera@10 { > compatible = "ovti,ov8856"; -- Regards, Laurent Pinchart
Powered by blists - more mailing lists