From nobody Thu Oct 9 04:43:19 2025 Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [217.70.183.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5CD9F2C3273; Fri, 20 Jun 2025 15:21:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.197 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750432909; cv=none; b=giIsya3+YqyJk+DK6gTlDhlJlVZvZXa2IOmAT/Y82unJL3ITEVr0C5zbAEtOq78Yj+lEvNUfWGa9pM07h1aenTJBBXpp+BbztfKSU3A2bKTiy0F4zmSB0UJ+pFyIc5L/XFSu5szxzAQznMPOpxUtIMUwvkjC6xBGaJY2mmL2Qc4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750432909; c=relaxed/simple; bh=S8TwUb3GF4dU4aN3dJBpb79QCsj1j3FukZnyTuO8DKE=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=utfuhbRCFJ1KHFCrBgu/pHjhnm7bIxIGvOihiU1W4OHmA/umYF9ypLVsHmlXyYBnG5fk53IDk1/dYN8GHBddV01HTMw6WQLj+HSbnm42kO5gFWesEArR39jeNvGAtUZh+uoqxVaxMoUiZrIJkoYIoj0OFB7hqpvH62a8Fi7GWgI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=nYbAcTWH; arc=none smtp.client-ip=217.70.183.197 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="nYbAcTWH" Received: by mail.gandi.net (Postfix) with ESMTPSA id C88D742EF5; Fri, 20 Jun 2025 15:21:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1750432905; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=2cN23HEa68EM4zJrxkTl2Mnxee+wt59/NVjDVEKkiFM=; b=nYbAcTWH7BcFK0yBFLxBnIVBi8Gkl2NCbM0Bvj+rJNXhQ8GjScqr7LQHwNmY7p9YpGa1zM TBmBrPHczwnnq585r1gVlL/KPKjJ8FrwyAOebPEmmETtbfFHLFU58tD23A21WO2nuPU2TX 7y8+6r8t0gxS3wukXl4Merxk+DXAJ77K088jpn8F2XySUuVBrnmBnvyBqzfHo8U8DkWjsH pR1LT0NEHs/ULka3JgqQcmNutoWLgM8O8ex+5XqimKfsN8pa926IrlfjOpyqDNPLFep7jS mi6qEjO2CkWD2c/u18tkQLgQARFAqnZx/rtMWnz2VMP2AHEj2LYxHZyNaRK5nw== From: Olivier Benjamin Date: Fri, 20 Jun 2025 17:21:34 +0200 Subject: [PATCH v4 3/4] arm64: dts: rockchip: describe I2c Bus 1 and IMX258 world camera on PinePhone Pro Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250620-camera-v4-3-0201a8ed5fae@bootlin.com> References: <20250620-camera-v4-0-0201a8ed5fae@bootlin.com> In-Reply-To: <20250620-camera-v4-0-0201a8ed5fae@bootlin.com> To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Jacopo Mondi , Nicholas Roth , Mauro Carvalho Chehab , Sakari Ailus , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam Cc: Thomas Petazzoni , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, Krzysztof Kozlowski , imx@lists.linux.dev, Olivier Benjamin , ~diederik/pine64-discuss@lists.sr.ht, Dragan Simic , Ondrej Jirman X-Mailer: b4 0.14.2 X-GND-State: clean X-GND-Score: -100 X-GND-Cause: gggruggvucftvghtrhhoucdtuddrgeeffedrtddvgdekjeegucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuifetpfffkfdpucggtfgfnhhsuhgsshgtrhhisggvnecuuegrihhlohhuthemuceftddunecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjughrpefhfffugggtgffkfhgjvfevofesthejredtredtjeenucfhrhhomhepqfhlihhvihgvrhcuuegvnhhjrghmihhnuceoohhlihhvihgvrhdrsggvnhhjrghmihhnsegsohhothhlihhnrdgtohhmqeenucggtffrrghtthgvrhhnpeegueegiedvvdevveevvddufeejvefftdeugfffkeeileehheefieehgfelfeeileenucfkphepvdgrtddumegvfeegmegvtgefkeemvdegvgdtmehfhegtvgemfhefgedvmeeiheekjeemfheiheeinecuvehluhhsthgvrhfuihiivgepvdenucfrrghrrghmpehinhgvthepvdgrtddumegvfeegmegvtgefkeemvdegvgdtmehfhegtvgemfhefgedvmeeiheekjeemfheiheeipdhhvghloheplgduledvrdduieekrddurddvtdgnpdhmrghilhhfrhhomhepohhlihhvihgvrhdrsggvnhhjrghmihhnsegsohhothhlihhnrdgtohhmpdhnsggprhgtphhtthhopedvgedprhgtphhtthhopeimughivgguvghrihhksdhpihhnvgeigedqughishgtuhhssheslhhishhtshdrshhrrdhhthdprhgtphhtthhopehrohgshheskhgvrhhnvghlrdhorhhgpdhrtghpthhtohepnhhitghhohhlrghssehrohhth hgvmhgrihhlrdhnvghtpdhrtghpthhtohepjhgrtghophhordhmohhnughisehiuggvrghsohhnsghorghrugdrtghomhdprhgtphhtthhopeholhhivhhivghrrdgsvghnjhgrmhhinhessghoohhtlhhinhdrtghomhdprhgtphhtthhopehmtghhvghhrggssehkvghrnhgvlhdrohhrghdprhgtphhtthhopehkrhiikhdoughtsehkvghrnhgvlhdrohhrghdprhgtphhtthhopehimhigsehlihhsthhsrdhlihhnuhigrdguvghv X-GND-Sasl: olivier.benjamin@bootlin.com Add the description of the rear/world camera (IMX258) on the PinePhone Pro to the device dts file. It receives commands on the I2C Bus 1 at address 0x1a and transmits data over CSI-MIPI. The I2C address for IMX258 can be found in the IMX258-0AQH5 Software Reference Manual, page 24, section 2.3.1: 0b0011010 =3D 0x1a. Section 3 indicates the module has 4 pairs of data lines. While 4-lane mode is nominal, 2-lane mode should also be supported. The pin muxing info was extracted from the PinePhone Pro schematic v1.0 as well as the RK3399 datasheet revision 1.8. Table 2-3 in section 2.8 of the RK3399 datasheet contains the mapping of IO functions for the SoC pins. Page 52 shows GPIO1_A0, page 54 shows GPIO2_D4. For I2C power, the PinePhone Pro schematic page 11 quadrants A4 and A5: RK3399_J.AA8 and RK3399_J.Y8 get power from vcaa1v8_codec, so turn it on The IMX258 also uses the following regulators, expected by its driver: - vana (2.8V analog), called AVDD2V8_DVP on P.18 q.C1 and derived from VCC1V8_S3 on P.13 q.B2 - vdig (1.2V digital core), called DVDD_DVP on P.18 q.C1 and shown on P.18 q.D3 to be equivalent to VCC1V2_DVP derived from VCC3V3_SYS on P.13 q.B3. Note that this regulator's voltage is inconsistently labeled either 1.2V or 1.5V RK3399_J.AG1 is GPIO4_A1/I2C1_SDA, RK3399_J.Y6 is GPIO4_A2/I2C1_SCL This is the default pinctrl "i2c1_xfer" for i2c1 from rk3399-base. For the reset (RESET) signal: page 11 quadrant D2 | p.18 q.C3-4 | p.18 q.C2 RK3399_E.R25 -> GPIO1_A0 -> Camera_RST -> MIPI_RST0 -> IMX258.12 For the powerdown (PWDN) signal: page 11 quadrants B4-5 | p.18 q.C2 RK3399_G.AF8 -> GPIO2_D4 -> DVP_PDN1_H -> IMX258.14 Helped-by: Dragan Simic Co-developed-by: Ondrej Jirman Signed-off-by: Ondrej Jirman Signed-off-by: Olivier Benjamin --- .../boot/dts/rockchip/rk3399-pinephone-pro.dts | 93 ++++++++++++++++++= ++++ 1 file changed, 93 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts b/arch/a= rm64/boot/dts/rockchip/rk3399-pinephone-pro.dts index 04ba4c4565d0a205e2e46d7535c6a3190993621d..b335d044138b413359ba54269ab= 646142254a55f 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts +++ b/arch/arm64/boot/dts/rockchip/rk3399-pinephone-pro.dts @@ -114,6 +114,16 @@ vcc3v3_sys: regulator-vcc3v3-sys { vin-supply =3D <&vcc_sys>; }; =20 + avdd2v8_dvp: regulator-avdd2v8-dvp { + compatible =3D "regulator-fixed"; + regulator-name =3D "avdd2v8_dvp"; + regulator-always-on; + regulator-boot-on; + regulator-min-microvolt =3D <2800000>; + regulator-max-microvolt =3D <2800000>; + vin-supply =3D <&vcc3v3_sys>; + }; + vcca1v8_s3: regulator-vcc1v8-s3 { compatible =3D "regulator-fixed"; regulator-name =3D "vcca1v8_s3"; @@ -136,6 +146,16 @@ vcc1v8_codec: regulator-vcc1v8-codec { vin-supply =3D <&vcc3v3_sys>; }; =20 + vcc1v2_dvp: regulator-vcc1v2-dvp { + compatible =3D "regulator-fixed"; + regulator-name =3D "vcc1v2_dvp"; + regulator-always-on; + regulator-boot-on; + regulator-min-microvolt =3D <1200000>; + regulator-max-microvolt =3D <1200000>; + vin-supply =3D <&vcca1v8_s3>; + }; + wifi_pwrseq: sdio-wifi-pwrseq { compatible =3D "mmc-pwrseq-simple"; clocks =3D <&rk818 1>; @@ -312,6 +332,8 @@ vcc3v0_touch: LDO_REG2 { =20 vcca1v8_codec: LDO_REG3 { regulator-name =3D "vcca1v8_codec"; + regulator-always-on; + regulator-boot-on; regulator-min-microvolt =3D <1800000>; regulator-max-microvolt =3D <1800000>; }; @@ -420,6 +442,45 @@ regulator-state-mem { }; }; =20 +&i2c1 { + clock-frequency =3D <400000>; + pinctrl-0 =3D <&i2c1_xfer &cif_clkouta>; + assigned-clocks =3D <&cru SCLK_CIF_OUT>; + assigned-clock-rates =3D <24000000>; + status =3D "okay"; + + wcam: camera@1a { + compatible =3D "sony,imx258"; + reg =3D <0x1a>; + clocks =3D <&cru SCLK_CIF_OUT>; /* MIPI_MCLK0, derived from CIF_CLKO */ + pinctrl-names =3D "default"; + pinctrl-0 =3D <&camera_rst>; + /* Note: both cameras also depend on vcca1v8_codec to power the I2C bus.= */ + vif-supply =3D <&vcc1v8_dvp>; + vana-supply =3D <&avdd2v8_dvp>; + vdig-supply =3D <&vcc1v2_dvp>; /* DVDD_DVP is the same as VCC1V2_DVP */ + reset-gpios =3D <&gpio1 RK_PA0 GPIO_ACTIVE_LOW>; + orientation =3D <1>; /* V4L2_CAMERA_ORIENTATION_BACK */ + rotation =3D <270>; + lens-focus =3D <&wcam_lens>; + + port { + wcam_out: endpoint { + remote-endpoint =3D <&mipi_in_wcam>; + data-lanes =3D <1 2 3 4>; + link-frequencies =3D /bits/ 64 <636000000>; + }; + }; + }; + + wcam_lens: camera-lens@c { + compatible =3D "dongwoon,dw9714"; + reg =3D <0x0c>; + /* Same I2c bus as both cameras, depends on vcca1v8_codec for power. */ + vcc-supply =3D <&vcc1v8_dvp>; + }; +}; + &i2c3 { i2c-scl-rising-time-ns =3D <450>; i2c-scl-falling-time-ns =3D <15>; @@ -462,6 +523,28 @@ &io_domains { status =3D "okay"; }; =20 +&isp1 { + status =3D "okay"; + + ports { + port@0 { + mipi_in_wcam: endpoint@0 { + reg =3D <0>; + remote-endpoint =3D <&wcam_out>; + data-lanes =3D <1 2 3 4>; + }; + }; + }; +}; + +&mipi_dphy_rx0 { + status =3D "okay"; +}; + +&isp1_mmu { + status =3D "okay"; +}; + &mipi_dsi { status =3D "okay"; clock-master; @@ -495,6 +578,10 @@ mipi_in_panel: endpoint { }; }; =20 +&mipi_dsi1 { + status =3D "okay"; +}; + &pmu_io_domains { pmu1830-supply =3D <&vcc_1v8>; status =3D "okay"; @@ -507,6 +594,12 @@ pwrbtn_pin: pwrbtn-pin { }; }; =20 + camera { + camera_rst_l: camera-rst-l { + rockchip,pins =3D <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>; + }; + }; + leds { red_led_pin: red-led-pin { rockchip,pins =3D <4 RK_PD2 RK_FUNC_GPIO &pcfg_pull_none>; --=20 2.49.0