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 764E0C433F5 for ; Thu, 5 May 2022 09:07:06 +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-Transfer-Encoding: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-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=BCk2YseH5Ck1HGir3lILKJQ3mjA7rXX1ASeEsgHtqiQ=; b=Iw1DxMrgHKQxYz AWtq8TRC4g3g2G3j0QbB1U0dSnDqH9cObTH7Ii+5OrRl3fS5ENI6+CnPHVgs3V2L49dAEuqgQP9Sy pUNeeEWZ7rnrPsx07p+7KO/Zk9suVodW2c4jHAHFCY8lakePlpL6Rh23NUAgFHo6gwcSpp7ZiUcaP B5WlfV/1XSfcBaEsQdyCWIUN0AfHsBL0y8KYrXLTIiXjl1MI5KwAEUs78aMWEfIP2RiYuPbqlReiU iRyOXuwb5lEY1sytJOwOsFjy7v+RS94Z1jqeZoCJV4ZR+LRiNDOiSoRioS8ltdcsOOjLVKCZvMn5K vMy86T95idFee3jA+JKA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmXQl-00Euuo-7N; Thu, 05 May 2022 09:05:51 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmXQh-00Eusp-Jj for linux-arm-kernel@lists.infradead.org; Thu, 05 May 2022 09:05:49 +0000 Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nmXQe-0001Q4-00; Thu, 05 May 2022 11:05:44 +0200 Received: from sha by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1nmXQb-0007lM-D7; Thu, 05 May 2022 11:05:41 +0200 Date: Thu, 5 May 2022 11:05:41 +0200 From: Sascha Hauer To: Frank Wunderlich Cc: dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, kernel@pengutronix.de, Andy Yan , Benjamin Gaignard , Michael Riesch , Sandy Huang , Heiko =?iso-8859-15?Q?St=FCbner?= , Peter Geis Subject: Re: [PATCH v11 17/24] arm64: dts: rockchip: rk356x: Add HDMI nodes Message-ID: <20220505090541.GN4012@pengutronix.de> References: <20220422072841.2206452-1-s.hauer@pengutronix.de> <20220422072841.2206452-18-s.hauer@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 11:03:13 up 35 days, 21:32, 83 users, load average: 0.45, 0.31, 0.18 User-Agent: Mutt/1.10.1 (2018-07-13) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c0 X-SA-Exim-Mail-From: sha@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220505_020547_701682_5E839AA1 X-CRM114-Status: GOOD ( 31.70 ) 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: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, May 05, 2022 at 10:45:03AM +0200, Frank Wunderlich wrote: > Hi, > > > Gesendet: Freitag, 22. April 2022 um 09:28 Uhr > > Von: "Sascha Hauer" > > Betreff: [PATCH v11 17/24] arm64: dts: rockchip: rk356x: Add HDMI nodes > > > > Add support for the HDMI port found on RK3568. > > > > Signed-off-by: Sascha Hauer > > --- > > > > Notes: > > Changes since v7: > > - Rename hclk to niu > > clock-name no more present since v9, see below > > > Changes since v5: > > - Drop unnecessary #size-cells/#address-cells from nodes with only single endpoint > > > ... > > --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi > > @@ -620,6 +620,38 @@ vop_mmu: iommu@fe043e00 { > > status = "disabled"; > > }; > > > > + hdmi: hdmi@fe0a0000 { > > + compatible = "rockchip,rk3568-dw-hdmi"; > > + reg = <0x0 0xfe0a0000 0x0 0x20000>; > > + interrupts = ; > > + clocks = <&cru PCLK_HDMI_HOST>, > > + <&cru CLK_HDMI_SFR>, > > + <&cru CLK_HDMI_CEC>, > > + <&pmucru CLK_HDMI_REF>, > > + <&cru HCLK_VO>; > > + clock-names = "iahb", "isfr", "cec", "ref"; > > noticed there are still 5 clocks, but only 4 clock-names. So i added "niu" after ref. > maybe missing clock-name was causing my iommu page-faults...on a quick test i have not got it, > but they came not every time. The clock is not handled by the HDMI driver, so it shouldn't be the cause for any failure. It should be dropped here. Heiko, could you apply the below patch or squash it into the original one? Sascha -------------------------------8<--------------------------- >From 8e5f90273401d98b2202676aafd49a350c9c4abd Mon Sep 17 00:00:00 2001 From: Sascha Hauer Date: Thu, 5 May 2022 10:59:48 +0200 Subject: [PATCH] arm64: dts: rockchip: rk356x: remove extra hdmi clock HCLK_VO is not handled by the HDMI driver. This is a leftover from earlier VOP2 series. Remove it. Signed-off-by: Sascha Hauer --- arch/arm64/boot/dts/rockchip/rk356x.dtsi | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi index 1a359bbf65300..49eb45e23f8c9 100644 --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi @@ -627,8 +627,7 @@ hdmi: hdmi@fe0a0000 { clocks = <&cru PCLK_HDMI_HOST>, <&cru CLK_HDMI_SFR>, <&cru CLK_HDMI_CEC>, - <&pmucru CLK_HDMI_REF>, - <&cru HCLK_VO>; + <&pmucru CLK_HDMI_REF>; clock-names = "iahb", "isfr", "cec", "ref"; pinctrl-names = "default"; pinctrl-0 = <&hdmitx_scl &hdmitx_sda &hdmitxm0_cec>; -- 2.30.2 -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel