Re: [PATCH v9 7/7] arm64: dts: qcom: sdm670-google-sargo: add imx355 front camera

David Heidelberg
2026-03-13T18:26:51Z

On 17/02/2026 01:27, Richard Acayan wrote:
[...]

> +&cci_i2c1 {
> + camera@1a {
> + compatible = "sony,imx355";
> + reg = <0x1a>;
> +
> + clocks = <&camcc CAM_CC_MCLK2_CLK>;
> +
> + assigned-clocks = <&camcc CAM_CC_MCLK2_CLK>;
> + assigned-clock-rates = <19200000>;

Extract from #sdm670-mainline:erebion.eu discussion:
The imx355 can operate on 24 MHz (on both Pixel 3 and 3a), but Linux kernel
driver can operate only with 19.2 MHz.

I assume it would be worth it mention at least by comment here.

David

> +
> + reset-gpios = <&tlmm 9 GPIO_ACTIVE_LOW>;
> +
> + avdd-supply = <&cam_front_ldo>;
> + dvdd-supply = <&cam_front_ldo>;
> + dovdd-supply = <&cam_vio_ldo>;
> +
> + pinctrl-0 = <&cam_mclk2_default>;
> + pinctrl-names = "default";
> +
> + rotation = <270>;
> + orientation = <0>;
> +
> + port {
> + cam_front_endpoint: endpoint {
> + link-frequencies = /bits/ 64 <360000000>;
> + remote-endpoint = <&camss_endpoint1>;
> + };
> + };
> + };
> +};
> +
> &gcc {
> protected-clocks = <GCC_QSPI_CORE_CLK>,
> <GCC_QSPI_CORE_CLK_SRC>,
> @@ -490,6 +571,14 @@ &pm660_charger {
> status = "okay";
> };
>
> +&pm660_gpios {
> + cam_vio_pin: cam-vio-state {
> + pins = "gpio13";
> + function = "normal";
> + power-source = <0>;
> + };
> +};
> +
> &pm660_rradc {
> status = "okay";
> };
> @@ -508,6 +597,12 @@ led-0 {
> };
>
> &pm660l_gpios {
> + cam_front_ldo_pin: cam-front-state {
> + pins = "gpio4";
> + function = "normal";
> + power-source = <0>;
> + };
> +
> vol_up_pin: vol-up-state {
> pins = "gpio7";
> function = "normal";

--
David Heidelberg