From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C1A87D47CD0 for ; Fri, 16 Jan 2026 11:07:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=dzSr+vK27hutzJjP42tJFaeLCI9FsTxETKHO6P3r+0c=; b=3yQZNSkdUVhuKW0+iN3lW/3ExS faH3GWPcRYuwdV5Dgc3SiwkDGP5KnCoFtzKUsD98mcIWVhYAAJmMhmlXICQgwCyh99/88q2qgKIkb g5T9hZDnYIP51C8AHro5CKyG7+aLqjlJ89cdHKJKb8YWZNgEV+Ui1g2vXnrQx4s0IXurElkzhAHo3 w2QU8Ddpcj7U0RAijqMUGQrtEPfL+gn9cTng1wMcyL28uuSaw3eoGAvCFkRwvWgx3Z5Lt4iQFxObI xIM6DhRS4EDsyyTpTRBFKlzGoxerRMAwqq6hfW5N1xPxYBgs50+8lNkOXDeGJ7SEXDxFb4uFOvIOf 6revG9/w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vghg5-0000000E0n3-2klV; Fri, 16 Jan 2026 11:07:41 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vghg3-0000000E0m6-1XZS; Fri, 16 Jan 2026 11:07:40 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To; bh=dzSr+vK27hutzJjP42tJFaeLCI9FsTxETKHO6P3r+0c=; b=B8+VwbhHy4P6pCRNKtlLFwEdMp lhd0dxCpyuWrkYSuFEtmsiURAqSkvL/WinKpjy116YeztW7MJT+o53mT7hYGinbXb5rz0fm+m9zJB lpJIK6pM6ETLCiX9v0Ye98LU9ruBQ2DKWyE434g2tIDq9H3jryobySsbYWr/2GIoRxH1Abn+g8hZT r5idEIyofm7OiqISbU3v7vIeP2AYgduIoNS5Wda1NnflUaC+71XIa06q52eiFHb6AQo+F6BTZxa4J ycJZXdXCjQe4yMpDNmnHmpGfL6QSHpA2JnpU4S3oUhc7a9WJGOTUixQKFBC9CE6mEqdKJuj/qRLPh BubXjh/g==; Received: from i53875a97.versanet.de ([83.135.90.151] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1vghfn-002cBj-LC; Fri, 16 Jan 2026 12:07:24 +0100 From: Heiko =?UTF-8?B?U3TDvGJuZXI=?= To: michael.riesch@collabora.com, Sakari Ailus Cc: Chaoyi Chen , Kever Yang , Frank Li , Mehdi Djait , Bryan O'Donoghue , Laurent Pinchart , Hans Verkuil , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Sebastian Reichel , Nicolas Dufresne , Collabora Kernel Team , linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/3] media: dt-bindings: add rockchip mipi csi-2 receiver Date: Fri, 16 Jan 2026 12:07:22 +0100 Message-ID: <5173450.iZASKD2KPV@diego> In-Reply-To: References: <20251114-rockchip-mipi-receiver-v4-0-a9c86fecd052@collabora.com> <20251114-rockchip-mipi-receiver-v4-1-a9c86fecd052@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260116_030739_433052_57CC1AB2 X-CRM114-Status: GOOD ( 22.94 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Freitag, 16. Januar 2026, 11:11:26 Mitteleurop=C3=A4ische Normalzeit sch= rieb Sakari Ailus: > Hi Michael, >=20 > On Thu, Jan 15, 2026 at 07:26:07PM +0100, Michael Riesch via B4 Relay wro= te: > > From: Michael Riesch > >=20 > > Add documentation for the Rockchip MIPI CSI-2 Receiver. > >=20 > > Signed-off-by: Michael Riesch > > Signed-off-by: Michael Riesch > > --- > > .../bindings/media/rockchip,rk3568-mipi-csi2.yaml | 141 +++++++++++++= ++++++++ > > MAINTAINERS | 6 + > > 2 files changed, 147 insertions(+) > >=20 > > diff --git a/Documentation/devicetree/bindings/media/rockchip,rk3568-mi= pi-csi2.yaml b/Documentation/devicetree/bindings/media/rockchip,rk3568-mipi= =2Dcsi2.yaml > > new file mode 100644 > > index 000000000000..2c2bd87582eb > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/rockchip,rk3568-mipi-csi2= =2Eyaml >=20 > I'd add a compatible string for the base IP block and name it accordingly. personally, I wouldn't do that. While the RK3568-variant is a somewhat smooth integration ... the now (temporarily) omitted RK3588 variant of a similar block is not. I.e. the RK3588 has quite a number of those CSI hosts, with a bunch of resource routing bits and bops between those CSI hosts sitting in the Rockchip "Gernal Register Files" (dumping ground for random bits and bops). So you then get a syscon accessing per-soc registers and bits. So while it is (compatible to) some Synopsis IP block, the integration to make that thing actually do something is highly soc-specific. That's also why the for example the dw-hdmi/dsi IPs don't use a common compatible [0] [1] [2] [3] [4] [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree= /Documentation/devicetree/bindings/display/amlogic,meson-dw-hdmi.yaml [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree= /Documentation/devicetree/bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree= /Documentation/devicetree/bindings/display/hisilicon/dw-dsi.txt [3] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree= /Documentation/devicetree/bindings/display/rockchip/rockchip,dw-hdmi.yaml [4] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree= /Documentation/devicetree/bindings/display/rockchip/rockchip,dw-mipi-dsi.ya= ml > > + soc { > > + interrupt-parent =3D <&gic>; > > + #address-cells =3D <2>; > > + #size-cells =3D <2>; > > + > > + csi: csi@fdfb0000 { > > + compatible =3D "rockchip,rk3568-mipi-csi2"; >=20 > This would become e.g. >=20 > compatible =3D "rockchip,rk3568-mipi-csi2", "snps,dw-mipi-csi= 2rx"; >=20 > See my comments on the driver patch as well. In the PCIe area, we have rockchip,rk3568-pcie and rockchip,rk3568-pcie-ep for a similar combo. =46or CSI the receiver is the vastly more common thing to do. So if anythin= g, I'd go with "foo-csi2" vs. "foo-csi2-device", if somebody really develops a "camera" SoC, with a fully featured DT-based OS in the future ;-) . Heiko