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 | 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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 CAA24C433F5 for ; Thu, 5 May 2022 09:05:47 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id EF0DC10F957; Thu, 5 May 2022 09:05:46 +0000 (UTC) Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by gabe.freedesktop.org (Postfix) with ESMTPS id BDEF710F957 for ; Thu, 5 May 2022 09:05:45 +0000 (UTC) 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 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: dri-devel@lists.freedesktop.org X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Benjamin Gaignard , Peter Geis , Sandy Huang , dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, Michael Riesch , kernel@pengutronix.de, Andy Yan , linux-arm-kernel@lists.infradead.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" 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 | 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 A0E76C433EF for ; Thu, 5 May 2022 09:06:23 +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=Ycgd69mWCklzpmt77bCey+Cjq6D5Wr14ihz9jnwqUoM=; b=ZQrz//5dDhi9mG Xwuvx5rCOIPzACw92Z0wH2ViIrQn3mryDCtUqtAF4x66x69nb8BcRneYDnFFJe1hESnxWOIPd/sm9 ShJNcNb4QCQmzaMilid0xFOVIDJGS6GfdPeyPqgcDJK76G2SW8QjzCchmPIzI8sbMdlqatffQCHf9 PD3XbKg4hTNUShoFfWf8xboRBGgAF0SMKU5v83xKtj7zKP8FINGRXssl6gw1ViduNFevB3uSdvBNg X7gzfkNmFA+A+MEjLnOGOzB9Qw9fdSuNe0+mPE3wW8cpmaYwaHd0cvWO90spyorl+noXares/EjqI a/LckxqPh17I0XjgtJEg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmXQt-00Eux0-B0; Thu, 05 May 2022 09:05:59 +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-00EutD-K6 for linux-rockchip@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-rockchip@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220505_020547_701685_089B1D1C X-CRM114-Status: GOOD ( 30.80 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=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-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip 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