[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <175214509884.1901332.11258386296497025238.b4-ty@sntech.de>
Date: Thu, 10 Jul 2025 13:01:33 +0200
From: Heiko Stuebner <heiko@...ech.de>
To: Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Jacopo Mondi <jacopo.mondi@...asonboard.com>,
Nicholas Roth <nicholas@...hemail.net>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
Olivier Benjamin <olivier.benjamin@...tlin.com>
Cc: Heiko Stuebner <heiko@...ech.de>,
Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-rockchip@...ts.infradead.org,
linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org,
Krzysztof Kozlowski <krzk@...nel.org>,
imx@...ts.linux.dev,
~diederik/pine64-discuss@...ts.sr.ht,
Dragan Simic <dsimic@...jaro.org>,
Ondrej Jirman <megi@....cz>
Subject: Re: (subset) [PATCH v4 0/4] Describe the cameras in the PinePhone Pro dts
On Fri, 20 Jun 2025 17:21:31 +0200, Olivier Benjamin wrote:
> This series adds support for the Pine64 PinePhone Pro's rear and front
> cameras in Device Tree.
> This is based on some of Ondrej Jirman's patches hosted in his tree at
> https://codeberg.org/megi/linux, but I have also fully reviewed and
> re-written the code from the RK3399 datasheet, the PinePhone Pro
> schematic, and the IMX258-0AQH5 software reference manual.
>
> [...]
Applied, thanks!
[3/4] arm64: dts: rockchip: describe I2c Bus 1 and IMX258 world camera on PinePhone Pro
commit: b610dc822f38010508b54c8f556a3de28556d273
[4/4] arm64: dts: rockchip: describe the OV8858 user camera on PinePhone Pro
commit: 9259c81a185663c2a7cd6a5301df4b9645980528
So I found the binding-patched from the media-tree in linux-next now,
so picked up the dts patches.
I fixed up some issues, please double check:
- the pinctrl-handle in patch3 did not match the created one
(which patch 4 fixed) but I adapted patch 3 instead
- sorting of the avdd regulator (alphabetical by node-name please)
- sorting of properties, broadly speaking, alphabetical too
(caveats apply, see dt documentation)
Best regards,
--
Heiko Stuebner <heiko@...ech.de>
Powered by blists - more mailing lists