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 X-Spam-Level: X-Spam-Status: No, score=-8.5 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 949E9C433E0 for ; Thu, 25 Jun 2020 07:40:19 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 50A6120789 for ; Thu, 25 Jun 2020 07:40:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="u9j9cUL3"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Vss8WPFz" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 50A6120789 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject: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=l54jqOjn7tDYOMhw9BMRu6Y26mkaYuFFZ7JN9tyxkO0=; b=u9j9cUL3e1KvNcJMnbAinXWfb 3OtBSxA5zYmK20xGh6XwOtatA3XwAhUXPCHjPrTcMfCIXhbBVCdGclANr5fgU6EZfvHFVZRDmz2ay A4QpFSvXq9xtSn3yFp9GjYsjnMiDBQW/DZlqWxmbcjHyGaEDqketQNClhPO91Ko1Ho1y/lOXArv8X DYtkx5HH/rSLOjKHfaj4z79iFxeib085aK2YrOQn7T3suqnZwQMzzp0fA9qKeRPt3PdF9hGbuTdmY NKNpHGFmZbSAPOqQUPVyUsqdj0KXchg5KUWBpSz0YplwfvTgd5h6AlMHaKidc5xLX+f5GnZEFu6VN W6kQIWN1g==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1joMS7-0003MA-W5; Thu, 25 Jun 2020 07:37:44 +0000 Received: from mail-ed1-x541.google.com ([2a00:1450:4864:20::541]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1joMS5-0003Lp-TR for linux-arm-kernel@lists.infradead.org; Thu, 25 Jun 2020 07:37:42 +0000 Received: by mail-ed1-x541.google.com with SMTP id m21so3386139eds.13 for ; Thu, 25 Jun 2020 00:37:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=wdRm8JiRn8ZIVWS/nELlT+8THIV83UbGYn4itX5cs+M=; b=Vss8WPFzKWmlnt6jhrUa+/5u5QdUKfWPkmn4LuDPc/T7GeoY46jA2ZvO8VWUTimFji Ox1+FtsAwZMSed8fX/AkNAO7rsa5JVK1l8gmn2TEG8l/0Z4fkxTfXpuBQQeGGj6Q+sE2 b+AIcqmlhOjd/cqq2JTxgYvwdvxuupg47vSAYs1HueHkxf0S07mZBzo/+pXVMPcB2xpg 6Ge8A2eKr873fxcUCgTU3muc+/iAeUT9Fy6y2e611PIF04PRxtI8dqEGvs/htLyXXON7 WBkwOjnwtqL0UPv4BpVbfgvFN+Xksg1RqxF68vrdDEoYb4gdxHaRln246tqlt61pGZzR X2Ew== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=wdRm8JiRn8ZIVWS/nELlT+8THIV83UbGYn4itX5cs+M=; b=b0UzemKE87J29j/0O0hM63vy92XQO94edTEs3SCoD6nx6AcOWtPk2GyrWpXjIKwHzo VeN/1X1wqO2GTeJHsRc4I/CaMp6x8R98pIPkMp1oFjAjke71KpSmahZ0ZUqh4x4ieQPv 7fKziJRqIMNYyQr+W+08/vaXC9JJQkdsuXrieSBYOh7IqfWmS8U5mVJuvpCmum/4EqnJ QxYemUsRKwWbv2VlrRABwUwzD28nBjqudJLp9q+t84tZg09TNdo+7IDc5zpg+YpIMGDD sL7KAu5ZaEb/kUaRYPMCJ8/cSXnXKebUm6HGui8QiP5A42LXyK1q9KFOJaPNMSwotJyk MCbA== X-Gm-Message-State: AOAM531iMggJxCnvUWM32vDuT01O+R1CfJf6Pni5uXzTARQ4iGnVl/6o +iXIbM9HbSGc3YQ8gISl1o8= X-Google-Smtp-Source: ABdhPJxjAdTGqBrNvSrM0sM74Qmq9b1P3KorN/9GUMvi5vss0dxXpkF1uQH6q4umHRHry6yF8oy3EA== X-Received: by 2002:a05:6402:642:: with SMTP id u2mr30413461edx.230.1593070659024; Thu, 25 Jun 2020 00:37:39 -0700 (PDT) Received: from localhost ([62.96.65.119]) by smtp.gmail.com with ESMTPSA id v5sm1669545ede.35.2020.06.25.00.37.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Jun 2020 00:37:37 -0700 (PDT) Date: Thu, 25 Jun 2020 09:37:36 +0200 From: Thierry Reding To: Dmitry Osipenko Subject: Re: [PATCH 12/73] ARM: tegra: Drop display controller parent clocks on Tegra114 Message-ID: <20200625073736.GA2800220@ulmo> References: <20200616135238.3001888-1-thierry.reding@gmail.com> <20200616135238.3001888-13-thierry.reding@gmail.com> <0bc2ec0e-f863-207a-d61b-058503598139@gmail.com> MIME-Version: 1.0 In-Reply-To: <0bc2ec0e-f863-207a-d61b-058503598139@gmail.com> User-Agent: Mutt/1.14.4 (2020-06-18) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-tegra@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Jon Hunter Content-Type: multipart/mixed; boundary="===============5902308210519159024==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5902308210519159024== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="17pEHd4RhPHOinZp" Content-Disposition: inline --17pEHd4RhPHOinZp Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 24, 2020 at 07:19:26PM +0300, Dmitry Osipenko wrote: > 16.06.2020 16:51, Thierry Reding =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > From: Thierry Reding > >=20 > > The parent clocks are determined by the output that will be used, not by > > the display controller that drives the output. Drop the parent clocks > > from the display controller device tree nodes. > >=20 > > Signed-off-by: Thierry Reding > > --- > > arch/arm/boot/dts/tegra114.dtsi | 10 ++++------ > > arch/arm/boot/dts/tegra124.dtsi | 10 ++++------ > > arch/arm/boot/dts/tegra20.dtsi | 10 ++++------ > > arch/arm/boot/dts/tegra30.dtsi | 10 ++++------ > > 4 files changed, 16 insertions(+), 24 deletions(-) > >=20 > > diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra1= 14.dtsi > > index a06b88b01ef3..23df7a5f37d3 100644 > > --- a/arch/arm/boot/dts/tegra114.dtsi > > +++ b/arch/arm/boot/dts/tegra114.dtsi > > @@ -59,9 +59,8 @@ dc@54200000 { > > compatible =3D "nvidia,tegra114-dc"; > > reg =3D <0x54200000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA114_CLK_DISP1>, > > - <&tegra_car TEGRA114_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA114_CLK_DISP1>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 27>; > > reset-names =3D "dc"; > > =20 > > @@ -78,9 +77,8 @@ dc@54240000 { > > compatible =3D "nvidia,tegra114-dc"; > > reg =3D <0x54240000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA114_CLK_DISP2>, > > - <&tegra_car TEGRA114_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA114_CLK_DISP2>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 26>; > > reset-names =3D "dc"; > > =20 > > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra1= 24.dtsi > > index 1afed8496c95..2c992e8e3594 100644 > > --- a/arch/arm/boot/dts/tegra124.dtsi > > +++ b/arch/arm/boot/dts/tegra124.dtsi > > @@ -105,9 +105,8 @@ dc@54200000 { > > compatible =3D "nvidia,tegra124-dc"; > > reg =3D <0x0 0x54200000 0x0 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA124_CLK_DISP1>, > > - <&tegra_car TEGRA124_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA124_CLK_DISP1>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 27>; > > reset-names =3D "dc"; > > =20 > > @@ -120,9 +119,8 @@ dc@54240000 { > > compatible =3D "nvidia,tegra124-dc"; > > reg =3D <0x0 0x54240000 0x0 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA124_CLK_DISP2>, > > - <&tegra_car TEGRA124_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA124_CLK_DISP2>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 26>; > > reset-names =3D "dc"; > > =20 > > diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20= =2Edtsi > > index f0a172c61b26..8b6909839f59 100644 > > --- a/arch/arm/boot/dts/tegra20.dtsi > > +++ b/arch/arm/boot/dts/tegra20.dtsi > > @@ -103,9 +103,8 @@ dc@54200000 { > > compatible =3D "nvidia,tegra20-dc"; > > reg =3D <0x54200000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA20_CLK_DISP1>, > > - <&tegra_car TEGRA20_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA20_CLK_DISP1>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 27>; > > reset-names =3D "dc"; > > =20 > > @@ -120,9 +119,8 @@ dc@54240000 { > > compatible =3D "nvidia,tegra20-dc"; > > reg =3D <0x54240000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA20_CLK_DISP2>, > > - <&tegra_car TEGRA20_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA20_CLK_DISP2>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 26>; > > reset-names =3D "dc"; > > =20 > > diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30= =2Edtsi > > index 27000f0ba35b..23fedb76e5ae 100644 > > --- a/arch/arm/boot/dts/tegra30.dtsi > > +++ b/arch/arm/boot/dts/tegra30.dtsi > > @@ -200,9 +200,8 @@ dc@54200000 { > > compatible =3D "nvidia,tegra30-dc", "nvidia,tegra20-dc"; > > reg =3D <0x54200000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA30_CLK_DISP1>, > > - <&tegra_car TEGRA30_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA30_CLK_DISP1>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 27>; > > reset-names =3D "dc"; > > =20 > > @@ -219,9 +218,8 @@ dc@54240000 { > > compatible =3D "nvidia,tegra30-dc"; > > reg =3D <0x54240000 0x00040000>; > > interrupts =3D ; > > - clocks =3D <&tegra_car TEGRA30_CLK_DISP2>, > > - <&tegra_car TEGRA30_CLK_PLL_P>; > > - clock-names =3D "dc", "parent"; > > + clocks =3D <&tegra_car TEGRA30_CLK_DISP2>; > > + clock-names =3D "dc"; > > resets =3D <&tegra_car 26>; > > reset-names =3D "dc"; > > =20 > >=20 >=20 > Hello Thierry, >=20 > Tegra DRM fails to probe after this change using next-20200624 on T20/30 > (T124 also should be broken): >=20 > tegra-dc 54200000.dc: failed to get parent clock > tegra-dc 54200000.dc: failed to probe RGB output: -2 Indeed. I had completely missed that we used to have this RGB output on prior chips and therefore do need the parent clock. As of Tegra124 that RGB output is no longer present, so this isn't needed anymore. > BTW, the commit's title is misleading since the patch touches all SoCs > and not only the T114. Good catch. I've replaced this with the following: --- >8 --- commit afd92390fcaa784a6d064f3b07c8d8124e43e5d1 Author: Thierry Reding Date: Thu Jun 11 19:09:36 2020 +0200 ARM: tegra: Drop display controller parent clocks on Tegra124 =20 The parent clocks are determined by the output that will be used, not by the display controller that drives the output. On previous generations a simple RGB output used to be part of the display controller and hence an explicit parent clock needed to be assigned to the display controller to drive the RGB output. Starting with Tegra124, that RGB output has been dropped and the parent clock can therefore be removed from the display controller device tree nodes. =20 Signed-off-by: Thierry Reding diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.d= tsi index 1afed8496c95..2c992e8e3594 100644 --- a/arch/arm/boot/dts/tegra124.dtsi +++ b/arch/arm/boot/dts/tegra124.dtsi @@ -105,9 +105,8 @@ dc@54200000 { compatible =3D "nvidia,tegra124-dc"; reg =3D <0x0 0x54200000 0x0 0x00040000>; interrupts =3D ; - clocks =3D <&tegra_car TEGRA124_CLK_DISP1>, - <&tegra_car TEGRA124_CLK_PLL_P>; - clock-names =3D "dc", "parent"; + clocks =3D <&tegra_car TEGRA124_CLK_DISP1>; + clock-names =3D "dc"; resets =3D <&tegra_car 27>; reset-names =3D "dc"; =20 @@ -120,9 +119,8 @@ dc@54240000 { compatible =3D "nvidia,tegra124-dc"; reg =3D <0x0 0x54240000 0x0 0x00040000>; interrupts =3D ; - clocks =3D <&tegra_car TEGRA124_CLK_DISP2>, - <&tegra_car TEGRA124_CLK_PLL_P>; - clock-names =3D "dc", "parent"; + clocks =3D <&tegra_car TEGRA124_CLK_DISP2>; + clock-names =3D "dc"; resets =3D <&tegra_car 26>; reset-names =3D "dc"; --- >8 --- Thierry --17pEHd4RhPHOinZp Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl70VD0ACgkQ3SOs138+ s6FFzBAAo/d7tW+bWArDQmse5y8uTXq9jzqaVKBHJKPXQcDjXVIZjF9zojwpdOV/ sMp+iYRZH/18OARVC8UT0JjWx2ojDQ/z3HAey001Zxu/icnX1ossvtFJDUG/Bucf 6HFfJYJDlHvnfCvDQPrB7yUTURyVX6fOP1fBE4REKwl0cGuaPwGET4bzgsq8Ooon E/I/3jpGs/hSacDNy2Y9J1Br3W+Ws6FVQPFH50yqN4nQr+QCQ6/1IrOcbsrR4Run umDwM0YcokY1HQkHuTExa+tS2mzb1OQxrNIJdvy8Hdny5R2NNeFZgDN0lJV6/zta zYJAQXEGKKVmDslX9jJV3U5uUphatqi5oMobOUdXSbpYs62kEtkSA2sVDr69uKwi PChhy1fqeL7qt0aDj6kX6TxiLRZHN5odHFyEvisiONZx/Wqq16Z8NhkzoY/Mnu9P oysqzR6cP8vNz0Wo/wuT/rwjVoM/e0Vq8G1YrFKavCKd77+85mzgXlOrQJW1spW4 d06TMEXk7Oc98/56PTwDPPOWFS7ywQPNspS7D7N031wIdxQreVb/e9z24M1B7hWP loSF+CtLv5mI+XikVmSj4Ap1J5HFJXkd9mZwTNFadppnnHCKK6OWJXFEc6rZHuAY R2tBiT7Ou2JkNtsKDl/1UeokEamJ5EZsSk6cuzMAbYJdHXw/miU= =oiOs -----END PGP SIGNATURE----- --17pEHd4RhPHOinZp-- --===============5902308210519159024== 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 --===============5902308210519159024==--