From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3CBEB4EE84D for ; Fri, 2 Oct 2026 15:40:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790955635; cv=none; b=JHpJ8xNje8ttuxzd57+S7xRlHyMyGQKmT12UA1FgzmTiyhzfKzgtod/u7Jet8B4uC6tmNp4A0fvKFoRLVMPwYwCP+S3jqdiWgy6ibIHE9U9jJ6/32Yyg7+Jpy7RCwfmOSctUWacKYkcyEmbZl2l43gA6TWSlLmjlsTS45UcU6Lk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790955635; c=relaxed/simple; bh=WQsDDSMUZRhlskhbuefNaU/oDZmoaAVqCqR5bHZT1Bg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=GxiZgU7qf8pc2todTdZVQYuPnCHL4Q4RjYU2t/nWlPcnTW8j3mIAYXLQa/o+vYtBTgJJjtcldXi7pGooCpUGFcRevfcqpBAlugyk1Zru7z3VsauIh/HE/1ddFHBAo/Wj7fQLGGl1sxKRXqxFb74ZJz+7r8Refhz2M1p6KKe3lJA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=FZO9v/rA; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=ozPpu/jX; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="FZO9v/rA"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="ozPpu/jX" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1790955619; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=oiiS057kpBWsJD1BtMWJm5EOfNc4H/UurY52kp0eYCc=; b=FZO9v/rAn4UaEvJRHny4YEhKKBHnl5oMo31kpcu/VqegrioRt3sgH54ZUriRlMdADTCCFi twVtGXqvk+mKL+//E9xugJvOidMW4pcPM+mrgkIQUy2EjgfCs1ww/d1EAYx9AxSRfes80X o+yLddwbKV229/Kv08h7BR3MCRbnK3c= Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-681-WxIZf1X3NRSb_f0Ls1xXIg-1; Fri, 02 Oct 2026 11:40:18 -0400 X-MC-Unique: WxIZf1X3NRSb_f0Ls1xXIg-1 X-Mimecast-MFC-AGG-ID: WxIZf1X3NRSb_f0Ls1xXIg_1790955617 Received: by mail-qt1-f199.google.com with SMTP id d75a77b69052e-5338cfe567eso32964331cf.2 for ; Fri, 02 Oct 2026 08:40:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1790955617; x=1791560417; darn=vger.kernel.org; h=user-agent:in-reply-to:content-disposition:content-type :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to:content-type; bh=oiiS057kpBWsJD1BtMWJm5EOfNc4H/UurY52kp0eYCc=; b=ozPpu/jXAzZIo0zu/9LK+ICU9zXJodlaZQRzNX5FgATAdHaZ+qAvKmTF6t9jvZfiJ9 AHmLMmZ6SmPxUhguGshnF08hBOW7KGWy1OVge8CKnRpdkHpUs3J9Zw5U0EhYXfIXZQc9 iRCvbwKgvrEZts/PrfR5k8R9TjKm5lvrdXwtn5rddK9pQ85cjiTOKzHchZn7MZTlSvCQ UpVf6Fbk+JFqhc9u7WML8faX1PukOsM8jJvIEF7M8zQVho6PbRJNrgkS41rljzFHo/ZF 0MGK5budA4AJFsXKlr1Lpbh+mamOjcfaEWVjU8e6SCBuHWYWyw0vC8dEOp/SiCPJEbgh m6Lw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790955617; x=1791560417; h=user-agent:in-reply-to:content-disposition:content-type :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=oiiS057kpBWsJD1BtMWJm5EOfNc4H/UurY52kp0eYCc=; b=Mz9eIAlypw6rkXi6wUika+CI70dnOnJpX4yjabMOA3CY0wPPcj58gJaJ9IluELlYhz hm5U2uC/AUvvec0jmYXvd5hsy594wDHIuG3+Ub8GtoR3FvR8uq9GmKw7LSiDwTSie1RA OTztEEkfqZ4sZtMh3I9QYBkoApGUXuZSDvyfiiG/QelGhwm/jlJIZ5ov/xMg7V5CL8Th 7yEk9rMFubRe+a1mmF3YiDrXdNX3C0i+5Ks1D7d9gwnpu51xVn6r/7w+AU3gqbYIP1L4 nC037c1nt2bv+bNnbo0yCy4X2I1JNApfWw+ipPWxLcMsA08RqiDw9k9D9px31fiX6wfk Ftbw== X-Forwarded-Encrypted: i=1; AKwUvBxfbULkdJ4/9L2gZy1S/CtNP3JsbtgOn3YXaiX3ClTMuiCTK7BsJPDQTAYRNW0hJUeH5kWl2xvFBDbLrMo=@vger.kernel.org X-Gm-Message-State: AFuF++nXz+eQJDBpLgWUYlPdXFI7N+gs94CLzqkKWhXbmU2VTHBgVEk5 p9nfbKvWZa8M8Lo5IstgeScFIYU670DfwaPn9pGVhn8f+HktlMWioTDt6CDbaQGG8RIPTKqxYGT H8dA4P+ax3ap70v11OZEQ7k8G/qgP1cgjrrMnIqvlO1hmLAXH2iyeM5MbjRVvv0Hi1w== X-Gm-Gg: AYBFou3jgp+OCDZq3iqqdyeDzZMLJvG7Q8xNv+Z2Q5W5+PjnF9SoBsvhcyTrgSeqRSo yhVOjHVbqvmtkbNd881EheDuaDMhXD1VLwVHetK6BB3ux+hmYqCyf9x016LJUR110lqmw2CF4If 8NyAaiMPSnd4rtXo4G8FJ62AxZ7pkFo+8Kf3rX/5vwrHcij29A+AUWbYro36LHgLCb9gJmasCNN AUTEVayRyQvb+hixItnzr8eYaBELMXqJ/a9KiU7stImKQBxRrnOGYe+hnGf5/naYyLcDdh+XEND qmu71Arv6uk1E3HwovQwRYP/XKtEpT+M78RSWS+Q6Woi8ntuFp+T+cu3wZuHLpD2GC45ywOZH42 azjUHmj/2GwPBaNYyuJ9lghNjxXAfghk2J78= X-Received: by 2002:a05:622a:250e:b0:532:ab93:1400 with SMTP id d75a77b69052e-533bc662ddamr53425611cf.5.1790955617360; Fri, 02 Oct 2026 08:40:17 -0700 (PDT) X-Received: by 2002:a05:622a:250e:b0:532:ab93:1400 with SMTP id d75a77b69052e-533bc662ddamr53425041cf.5.1790955616765; Fri, 02 Oct 2026 08:40:16 -0700 (PDT) Received: from redhat.com (c-73-183-53-213.hsd1.pa.comcast.net. [73.183.53.213]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-53398d438ccsm28817651cf.30.2026.10.02.08.40.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 02 Oct 2026 08:40:15 -0700 (PDT) Date: Fri, 2 Oct 2026 11:40:13 -0400 From: Brian Masney To: Mikko Perttunen Cc: Stephen Boyd , Brian Masney , Jerome Brunet , Thierry Reding , Jonathan Hunter , linux-clk@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] clk: tegra: Fix Tegra114 MIPI calibration clock parent Message-ID: References: <20260930-b4-dalmore-fixes-mipical-clk-v1-1-0518be6c7979@nvidia.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260930-b4-dalmore-fixes-mipical-clk-v1-1-0518be6c7979@nvidia.com> User-Agent: Mutt/2.4.2 (2026-08-30) Hi Mikko, On Wed, Sep 30, 2026 at 02:18:34PM +0900, Mikko Perttunen wrote: > CLK_MIPI_CAL on Tegra114 is a plain clock gate with no source mux; the > fixed parent is PLL_P_OUT3, which is specified to run at 72 MHz. The > clock driver currently registers the gate with clk_m as its parent, so > the clock framework never enables PLL_P_OUT3 when MIPI_CAL is enabled, > and reports the wrong rate for it. > > Register the gate with pll_p_out3 as its parent. Also drop > CLK_SET_RATE_PARENT at the same time: the parent canonically runs at > 72 MHz so we shouldn't change the rate. > > Signed-off-by: Mikko Perttunen Should there be a Fixes tag? I see that Thierry made a change to this same clock 11 years ago: commit 07314fc108e195044c074f4b443974b5aaa2d5d7 Author: Thierry Reding Date: Wed Apr 8 16:48:26 2015 +0200 clk: tegra: Special-case mipi-cal parent on Tegra114 Starting with Tegra124, the mipi-cal clock uses the 72 MHz clock as its source. On Tegra114 this clock's parent was clk_m, so it is the one-off chip. Brian > --- > drivers/clk/tegra/clk-tegra114.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/clk/tegra/clk-tegra114.c b/drivers/clk/tegra/clk-tegra114.c > index f1bca81b7776..97f5500b13b5 100644 > --- a/drivers/clk/tegra/clk-tegra114.c > +++ b/drivers/clk/tegra/clk-tegra114.c > @@ -1050,8 +1050,8 @@ static __init void tegra114_periph_clk_init(void __iomem *clk_base, > &emc_lock); > clks[TEGRA114_CLK_MC] = clk; > > - clk = tegra_clk_register_periph_gate("mipi-cal", "clk_m", 0, clk_base, > - CLK_SET_RATE_PARENT, 56, > + clk = tegra_clk_register_periph_gate("mipi-cal", "pll_p_out3", 0, > + clk_base, 0, 56, > periph_clk_enb_refcnt); > clks[TEGRA114_CLK_MIPI_CAL] = clk; > > > --- > base-commit: 72d3fcf802c45d00b300f25b848a93c3a2bd7c7e > change-id: 20260930-b4-dalmore-fixes-mipical-clk-eb375008dd66 >