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 45672C25B76 for ; Mon, 3 Jun 2024 16:22:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=KgcIVu+oJE+ZSQnhGb7E/9buUypviyUIPN38BMDLXWc=; b=SxRIEJDMpFuRtD7U4QGjLOIwHV Y65V85Zp5O6HSX/K80GbZSRsNIN60dbSRnjh1K78ubxw/Ruey/JSdjNyepwjxg+BJ05urluhEjyzJ FFLABZxLJEF9bUklfcDKebR1sjF8NSPiyZvhKFg6/Fsetwh+WcJa/w80sbktZXz+hPzTeo4WmfVx1 hHfuvrxdxLTgJ3gi/A/DqGjYC4U2sLkdZaLSMayZPCa9VYnlkyrQbM2FsSLQqwV8ebhAi4Rxe3vmG SfeWB5jihynpQsv7wr5Vot5wFub17FiP+GyxtLBQE1ds376x56vFBln21TwtGCiBW/g5OB+JYVgpg 70HdmU3g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sEARn-0000000HMTU-0nrw; Mon, 03 Jun 2024 16:22:11 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sEARl-0000000HMSi-13IW; Mon, 03 Jun 2024 16:22:10 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 45E7B60ED6; Mon, 3 Jun 2024 16:22:08 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 75055C32786; Mon, 3 Jun 2024 16:22:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1717431728; bh=J9BtwLiH5m/8yi2TzzknIpSb4dbESwBD7BOkieFUAmo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Bu+lz/1XWvQlr8sMasYuBapx64iWXKhYYuE32EKqwkwTOey+HseXz0BB7MFLlI85O kPe8OuoeQLtatj+0dQnBSE11+aHY/uHissu22mA7QPGEup2Hllay2U8y+LkCtZp4DG 3aefv6kQfoqF4xAaZe7pVlIWmWF9Y+GsLK2MFHFI5c7JgUKtJ0WrFs/K8XnjqwOkkr QQUfznd4Do66OiqXK5PtilHsFIryvPsWf1JLlvAjJh9pUXy6R4D1bjcz5BAa/bb2oq 5pJ2MF5f1AcMYkEi5sQdEjOy3hr3+xaolC+z5Rg7kItsSw7297BoxsKLfJ0vkFmy3K V+pk2YU6AvVRw== Date: Mon, 3 Jun 2024 18:22:05 +0200 From: Maxime Ripard To: Heiko Stuebner Cc: neil.armstrong@linaro.org, Cristian Ciocaltea , Andrzej Hajda , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Sandy Huang , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Mark Yao , Andy Yan , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, kernel@collabora.com, Alexandre ARNOUD , Luis de Arquer , Algea Cao Subject: Re: [PATCH 00/14] Add initial support for the Rockchip RK3588 HDMI TX Controller Message-ID: <20240603-noisy-convivial-boa-d3d862@houat> References: <20240601-b4-rk3588-bridge-upstream-v1-0-f6203753232b@collabora.com> <11359776.NyiUUSuA9g@phil> MIME-Version: 1.0 In-Reply-To: <11359776.NyiUUSuA9g@phil> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240603_092209_413400_4250BB38 X-CRM114-Status: GOOD ( 23.52 ) 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: , Content-Type: multipart/mixed; boundary="===============4334231601569859882==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============4334231601569859882== Content-Type: multipart/signed; micalg=pgp-sha384; protocol="application/pgp-signature"; boundary="hjyaj4zwc26gvppm" Content-Disposition: inline --hjyaj4zwc26gvppm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jun 03, 2024 at 03:03:12PM GMT, Heiko Stuebner wrote: > Am Montag, 3. Juni 2024, 14:14:17 CEST schrieb Andy Yan: > > Hi Neil: > >=20 > > On 6/3/24 16:55, Neil Armstrong wrote: > > > Hi Christian, > > >=20 > > > On 01/06/2024 15:12, Cristian Ciocaltea wrote: > > >> The RK3588 SoC family integrates a Quad-Pixel (QP) variant of the > > >> Synopsys DesignWare HDMI TX controller used in the previous SoCs. > > >> > > >> It is HDMI 2.1 compliant and supports the following features, among > > >> others: > > >> > > > . > > >=20 > > > .. > > >=20 > > >> * SCDC I2C DDC access > > >> * TMDS Scrambler enabling 2160p@60Hz with RGB/YCbCr4:4:4 > > >> * YCbCr4:2:0 enabling 2160p@60Hz at lower HDMI link speeds > > >> * Multi-stream audio > > >> * Enhanced Audio Return Channel (EARC) > > > -> Those features were already supported by the HDMI 2.0a compliant H= W, just > > > list the _new_ features for HDMI 2.1 > > >=20 > > > I did a quick review of your patchset and I don't understand why you = need > > > to add a separate dw-hdmi-qp.c since you only need simple variants of= the I2C > > > bus, infoframe and bridge setup. > > >=20 > > > Can you elaborate further ? isn't this Quad-Pixel (QP) TX controller = version > > > detectable at runtime ? > > >=20 > > > I would prefer to keep a single dw-hdmi driver if possible. > > > > The QP HDMI controller is a completely different variant with totally d= ifferent > > registers layout, see PATCH 13/14. > > I think make it a separate driver will be easier for development and ma= intenance. >=20 > I'm with Andy here. Trying to navigate a driver for two IP blocks really > sounds taxing especially when both are so different. If it's a completely new controller, I agree that it needs a new driver, but then why do we need to share code between the two? Maxime --hjyaj4zwc26gvppm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iJUEABMJAB0WIQTkHFbLp4ejekA/qfgnX84Zoj2+dgUCZl3tqAAKCRAnX84Zoj2+ dtR8AYCrl/MU0AvHJWAGUIyy2KX4WDHpvmSlO0r/dHofLBLuPbECLxs/Y4Xrv9aj dKaN4FEBgKcHcNQQuLzVnSmBEdP8BYecOgEiTEgRvexLG3nvcO9BpScGHjOp+i9W SoX1pl9lAQ== =Yfg6 -----END PGP SIGNATURE----- --hjyaj4zwc26gvppm-- --===============4334231601569859882== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============4334231601569859882==--