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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99598C433F5 for ; Thu, 5 May 2022 09:05:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234487AbiEEJJc (ORCPT ); Thu, 5 May 2022 05:09:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48958 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234283AbiEEJJc (ORCPT ); Thu, 5 May 2022 05:09:32 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6E69BBF65 for ; Thu, 5 May 2022 02:05:51 -0700 (PDT) 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-Type: text/plain; charset=us-ascii 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: devicetree@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.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 |