* [PATCH] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 webcam @ 2026-04-09 8:36 ` Oliver White 2026-04-09 18:47 ` Bryan O'Donoghue 2026-04-09 20:17 ` [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White 0 siblings, 2 replies; 5+ messages in thread From: Oliver White @ 2026-04-09 8:36 UTC (permalink / raw) To: andersson, konradybcio, robh, krzk+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel, Oliver White Wire up the front-facing OV02C10 camera on Microsoft Romulus by enabling CAMSS, CCI1 and CSIPHY4, adding the sensor node and its endpoint, and describing the PM8010 regulator rails and pinctrl states required by the camera module. With these DT nodes in place the webcam can be probed and used through the upstream OV02C10 driver. Signed-off-by: Oliver White <oliverjwhite07@gmail.com> --- .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 127 ++++++++++++++++++ 1 file changed, 127 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi index 14b5663a4d48..9e910813fa48 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi @@ -857,6 +857,57 @@ vreg_l3j: ldo3 { regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; }; }; + + regulators-8 { + compatible = "qcom,pm8010-rpmh-regulators"; + qcom,pmic-id = "m"; + + vdd-l1-l2-supply = <&vreg_s5j>; + vdd-l3-l4-supply = <&vreg_s4c>; + vdd-l7-supply = <&vreg_bob1>; + + vreg_l1m_1p2: ldo1 { + regulator-name = "vreg_l1m_1p2"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1260000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l2m_1p2: ldo2 { + regulator-name = "vreg_l2m_1p2"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1260000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l3m_1p8: ldo3 { + regulator-name = "vreg_l3m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1900000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l4m_1p8: ldo4 { + regulator-name = "vreg_l4m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1900000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l5m_2p8: ldo5 { + regulator-name = "vreg_l5m_2p8"; + regulator-min-microvolt = <2800000>; + regulator-max-microvolt = <3072000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l7m_2p8: ldo7 { + regulator-name = "vreg_l7m_2p8"; + regulator-min-microvolt = <2800000>; + regulator-max-microvolt = <3072000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + }; }; &gpu { @@ -867,6 +918,66 @@ &gpu_zap_shader { firmware-name = "qcom/x1e80100/microsoft/qcdxkmsuc8380.mbn"; }; +&camss { + status = "okay"; + + ports { + /* + * port0 => csiphy0 + * port1 => csiphy1 + * port2 => csiphy2 + * port3 => csiphy4 + */ + port@3 { + camss_csiphy4_inep0: endpoint@0 { + clock-lanes = <7>; + data-lanes = <0 1>; + remote-endpoint = <&ov02c10_ep>; + }; + }; + }; +}; + +&cci1 { + status = "okay"; +}; + +&cci1_i2c1 { + camera@36 { + compatible = "ovti,ov02c10"; + reg = <0x36>; + + reset-gpios = <&tlmm 237 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&cam_rgb_default>; + + clocks = <&camcc CAM_CC_MCLK4_CLK>; + assigned-clocks = <&camcc CAM_CC_MCLK4_CLK>; + assigned-clock-rates = <19200000>; + + orientation = <0>; /* front facing */ + + avdd-supply = <&vreg_l5m_2p8>; + dvdd-supply = <&vreg_l1m_1p2>; + dovdd-supply = <&vreg_l3m_1p8>; + + port { + ov02c10_ep: endpoint { + data-lanes = <1 2>; + link-frequencies = /bits/ 64 <400000000>; + remote-endpoint = <&camss_csiphy4_inep0>; + }; + }; + }; +}; + +&csiphy4 { + vdda-0p8-supply = <&vreg_l2c>; + vdda-1p2-supply = <&vreg_l1c>; + + status = "okay"; +}; + &i2c0 { clock-frequency = <100000>; @@ -1441,6 +1552,22 @@ wcn_sw_en: wcn-sw-en-state { bias-disable; }; + cam_rgb_default: cam-rgb-default-state { + mclk-pins { + pins = "gpio100"; + function = "cam_aon"; + drive-strength = <16>; + bias-disable; + }; + + reset-n-pins { + pins = "gpio237"; + function = "gpio"; + drive-strength = <2>; + bias-disable; + }; + }; + cam_indicator_en: cam-indicator-en-state { pins = "gpio225"; function = "gpio"; -- 2.51.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 webcam 2026-04-09 8:36 ` [PATCH] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 webcam Oliver White @ 2026-04-09 18:47 ` Bryan O'Donoghue 2026-04-09 20:17 ` [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White 1 sibling, 0 replies; 5+ messages in thread From: Bryan O'Donoghue @ 2026-04-09 18:47 UTC (permalink / raw) To: Oliver White, andersson, konradybcio, robh, krzk+dt, conor+dt Cc: linux-arm-msm, devicetree, linux-kernel On 09/04/2026 09:36, Oliver White wrote: > Wire up the front-facing OV02C10 camera on Microsoft Romulus by enabling > CAMSS, CCI1 and CSIPHY4, adding the sensor node and its endpoint, and > describing the PM8010 regulator rails and pinctrl states required by the > camera module. > > With these DT nodes in place the webcam can be probed and used through > the upstream OV02C10 driver. Technically its a MIPI CSI2 camera not a webcam... > Signed-off-by: Oliver White <oliverjwhite07@gmail.com> > --- > .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 127 ++++++++++++++++++ > 1 file changed, 127 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi > index 14b5663a4d48..9e910813fa48 100644 > --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi > +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi > @@ -857,6 +857,57 @@ vreg_l3j: ldo3 { > regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > }; > }; > + > + regulators-8 { > + compatible = "qcom,pm8010-rpmh-regulators"; > + qcom,pmic-id = "m"; > + > + vdd-l1-l2-supply = <&vreg_s5j>; > + vdd-l3-l4-supply = <&vreg_s4c>; > + vdd-l7-supply = <&vreg_bob1>; > + > + vreg_l1m_1p2: ldo1 { > + regulator-name = "vreg_l1m_1p2"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1260000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l2m_1p2: ldo2 { > + regulator-name = "vreg_l2m_1p2"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1260000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l3m_1p8: ldo3 { > + regulator-name = "vreg_l3m_1p8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1900000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l4m_1p8: ldo4 { > + regulator-name = "vreg_l4m_1p8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1900000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l5m_2p8: ldo5 { > + regulator-name = "vreg_l5m_2p8"; > + regulator-min-microvolt = <2800000>; > + regulator-max-microvolt = <3072000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + > + vreg_l7m_2p8: ldo7 { > + regulator-name = "vreg_l7m_2p8"; > + regulator-min-microvolt = <2800000>; > + regulator-max-microvolt = <3072000>; > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; > + }; > + }; I would recommend breaking this patch into two parts. One adding the regulators the other adding the camera enablement. > }; > > &gpu { > @@ -867,6 +918,66 @@ &gpu_zap_shader { > firmware-name = "qcom/x1e80100/microsoft/qcdxkmsuc8380.mbn"; > }; > > +&camss { > + status = "okay"; > + > + ports { > + /* > + * port0 => csiphy0 > + * port1 => csiphy1 > + * port2 => csiphy2 > + * port3 => csiphy4 > + */ > + port@3 { > + camss_csiphy4_inep0: endpoint@0 { > + clock-lanes = <7>; > + data-lanes = <0 1>; > + remote-endpoint = <&ov02c10_ep>; > + }; > + }; > + }; > +}; > + > +&cci1 { > + status = "okay"; > +}; > + > +&cci1_i2c1 { > + camera@36 { > + compatible = "ovti,ov02c10"; > + reg = <0x36>; > + > + reset-gpios = <&tlmm 237 GPIO_ACTIVE_LOW>; > + pinctrl-names = "default"; > + pinctrl-0 = <&cam_rgb_default>; > + > + clocks = <&camcc CAM_CC_MCLK4_CLK>; > + assigned-clocks = <&camcc CAM_CC_MCLK4_CLK>; > + assigned-clock-rates = <19200000>; > + > + orientation = <0>; /* front facing */ > + > + avdd-supply = <&vreg_l5m_2p8>; > + dvdd-supply = <&vreg_l1m_1p2>; > + dovdd-supply = <&vreg_l3m_1p8>; > + > + port { > + ov02c10_ep: endpoint { > + data-lanes = <1 2>; > + link-frequencies = /bits/ 64 <400000000>; > + remote-endpoint = <&camss_csiphy4_inep0>; > + }; > + }; > + }; > +}; > + > +&csiphy4 { > + vdda-0p8-supply = <&vreg_l2c>; > + vdda-1p2-supply = <&vreg_l1c>; > + > + status = "okay"; > +}; > + +1 for using the new style but, you should make clear in your cover letter that there is a depends on the style change. > &i2c0 { > clock-frequency = <100000>; > > @@ -1441,6 +1552,22 @@ wcn_sw_en: wcn-sw-en-state { > bias-disable; > }; > > + cam_rgb_default: cam-rgb-default-state { > + mclk-pins { > + pins = "gpio100"; > + function = "cam_aon"; > + drive-strength = <16>; > + bias-disable; > + }; > + > + reset-n-pins { > + pins = "gpio237"; > + function = "gpio"; > + drive-strength = <2>; > + bias-disable; > + }; > + }; > + > cam_indicator_en: cam-indicator-en-state { > pins = "gpio225"; > function = "gpio"; > -- > 2.51.0 > Please cc me on V2 as I'd like to give RB for this. --- bod ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera 2026-04-09 8:36 ` [PATCH] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 webcam Oliver White 2026-04-09 18:47 ` Bryan O'Donoghue @ 2026-04-09 20:17 ` Oliver White 2026-04-09 20:17 ` [PATCH v2 1/2] arm64: dts: qcom: x1e80100-microsoft-romulus: add PM8010 camera regulators Oliver White 2026-04-09 20:17 ` [PATCH v2 2/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White 1 sibling, 2 replies; 5+ messages in thread From: Oliver White @ 2026-04-09 20:17 UTC (permalink / raw) To: andersson, konradybcio, robh, krzk+dt, conor+dt Cc: bod, linux-arm-msm, devicetree, linux-kernel, Oliver White This series wires up the front-facing OV02C10 MIPI CSI-2 camera on Microsoft Romulus. Patch 1 adds the PM8010 regulator outputs used by the camera module. Patch 2 enables CAMSS, CCI1 and CSIPHY4, adds the OV02C10 sensor node, and defines the pinctrl state used by the camera clock and reset lines. This depends on the CAMSS DT style conversion discussed in review, as it uses the same endpoint layout style. Changes in v2: - rename "webcam" to "MIPI CSI-2 camera" - split regulator additions from camera enablement - add a cover letter describing the series dependency Oliver White (2): arm64: dts: qcom: x1e80100-microsoft-romulus: add PM8010 camera regulators arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 127 ++++++++++++++++++ 1 file changed, 127 insertions(+) base-commit: 6fbfeca7008b379fb2cdc4f1409c139185e5a029 -- 2.51.0 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2 1/2] arm64: dts: qcom: x1e80100-microsoft-romulus: add PM8010 camera regulators 2026-04-09 20:17 ` [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White @ 2026-04-09 20:17 ` Oliver White 2026-04-09 20:17 ` [PATCH v2 2/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White 1 sibling, 0 replies; 5+ messages in thread From: Oliver White @ 2026-04-09 20:17 UTC (permalink / raw) To: andersson, konradybcio, robh, krzk+dt, conor+dt Cc: bod, linux-arm-msm, devicetree, linux-kernel, Oliver White Add the PM8010 regulator outputs used by the front-facing OV02C10 camera module on Microsoft Romulus. These rails provide the supplies referenced by the camera enablement patch. Signed-off-by: Oliver White <oliverjwhite07@gmail.com> --- .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 52 +++++++++++++++++++ 1 file changed, 52 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi index 14b5663a4d48..4427ecae423f 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi @@ -857,6 +857,57 @@ vreg_l3j: ldo3 { regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; }; }; + + regulators-8 { + compatible = "qcom,pm8010-rpmh-regulators"; + qcom,pmic-id = "m"; + + vdd-l1-l2-supply = <&vreg_s5j>; + vdd-l3-l4-supply = <&vreg_s4c>; + vdd-l7-supply = <&vreg_bob1>; + + vreg_l1m_1p2: ldo1 { + regulator-name = "vreg_l1m_1p2"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1260000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l2m_1p2: ldo2 { + regulator-name = "vreg_l2m_1p2"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1260000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l3m_1p8: ldo3 { + regulator-name = "vreg_l3m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1900000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l4m_1p8: ldo4 { + regulator-name = "vreg_l4m_1p8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1900000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l5m_2p8: ldo5 { + regulator-name = "vreg_l5m_2p8"; + regulator-min-microvolt = <2800000>; + regulator-max-microvolt = <3072000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + + vreg_l7m_2p8: ldo7 { + regulator-name = "vreg_l7m_2p8"; + regulator-min-microvolt = <2800000>; + regulator-max-microvolt = <3072000>; + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>; + }; + }; }; &gpu { @@ -867,6 +918,7 @@ &gpu_zap_shader { firmware-name = "qcom/x1e80100/microsoft/qcdxkmsuc8380.mbn"; }; + &i2c0 { clock-frequency = <100000>; -- 2.51.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2 2/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera 2026-04-09 20:17 ` [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White 2026-04-09 20:17 ` [PATCH v2 1/2] arm64: dts: qcom: x1e80100-microsoft-romulus: add PM8010 camera regulators Oliver White @ 2026-04-09 20:17 ` Oliver White 1 sibling, 0 replies; 5+ messages in thread From: Oliver White @ 2026-04-09 20:17 UTC (permalink / raw) To: andersson, konradybcio, robh, krzk+dt, conor+dt Cc: bod, linux-arm-msm, devicetree, linux-kernel, Oliver White Enable the front-facing OV02C10 MIPI CSI-2 camera on Microsoft Romulus by enabling CAMSS, CCI1 and CSIPHY4, adding the sensor node and its endpoint, and defining the pinctrl state used by the camera clock and reset lines. Signed-off-by: Oliver White <oliverjwhite07@gmail.com> --- .../dts/qcom/x1e80100-microsoft-romulus.dtsi | 75 +++++++++++++++++++ 1 file changed, 75 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi index 4427ecae423f..9e910813fa48 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi @@ -918,6 +918,65 @@ &gpu_zap_shader { firmware-name = "qcom/x1e80100/microsoft/qcdxkmsuc8380.mbn"; }; +&camss { + status = "okay"; + + ports { + /* + * port0 => csiphy0 + * port1 => csiphy1 + * port2 => csiphy2 + * port3 => csiphy4 + */ + port@3 { + camss_csiphy4_inep0: endpoint@0 { + clock-lanes = <7>; + data-lanes = <0 1>; + remote-endpoint = <&ov02c10_ep>; + }; + }; + }; +}; + +&cci1 { + status = "okay"; +}; + +&cci1_i2c1 { + camera@36 { + compatible = "ovti,ov02c10"; + reg = <0x36>; + + reset-gpios = <&tlmm 237 GPIO_ACTIVE_LOW>; + pinctrl-names = "default"; + pinctrl-0 = <&cam_rgb_default>; + + clocks = <&camcc CAM_CC_MCLK4_CLK>; + assigned-clocks = <&camcc CAM_CC_MCLK4_CLK>; + assigned-clock-rates = <19200000>; + + orientation = <0>; /* front facing */ + + avdd-supply = <&vreg_l5m_2p8>; + dvdd-supply = <&vreg_l1m_1p2>; + dovdd-supply = <&vreg_l3m_1p8>; + + port { + ov02c10_ep: endpoint { + data-lanes = <1 2>; + link-frequencies = /bits/ 64 <400000000>; + remote-endpoint = <&camss_csiphy4_inep0>; + }; + }; + }; +}; + +&csiphy4 { + vdda-0p8-supply = <&vreg_l2c>; + vdda-1p2-supply = <&vreg_l1c>; + + status = "okay"; +}; &i2c0 { clock-frequency = <100000>; @@ -1493,6 +1552,22 @@ wcn_sw_en: wcn-sw-en-state { bias-disable; }; + cam_rgb_default: cam-rgb-default-state { + mclk-pins { + pins = "gpio100"; + function = "cam_aon"; + drive-strength = <16>; + bias-disable; + }; + + reset-n-pins { + pins = "gpio237"; + function = "gpio"; + drive-strength = <2>; + bias-disable; + }; + }; + cam_indicator_en: cam-indicator-en-state { pins = "gpio225"; function = "gpio"; -- 2.51.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-04-09 20:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <kgIcRmrN-nnKnAZJcqsZcRPp3GQVY-saH_a--KwDezB5APFRIQj7Qm2E6NMkM-6-CevWksG-StyswtMWda66NQ==@protonmail.internalid>
2026-04-09 8:36 ` [PATCH] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 webcam Oliver White
2026-04-09 18:47 ` Bryan O'Donoghue
2026-04-09 20:17 ` [PATCH v2 0/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White
2026-04-09 20:17 ` [PATCH v2 1/2] arm64: dts: qcom: x1e80100-microsoft-romulus: add PM8010 camera regulators Oliver White
2026-04-09 20:17 ` [PATCH v2 2/2] arm64: dts: qcom: x1e80100-microsoft-romulus: enable OV02C10 MIPI CSI-2 camera Oliver White
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox