Received: by 2002:ab2:2994:0:b0:1ef:ca3e:3cd5 with SMTP id n20csp65845lqb; Thu, 14 Mar 2024 05:50:56 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWEhj+fIR6JbwunrKX7EIjBv8AiczqNy/swLSVIXFLKjGQ7g+Py7J0lglRbaNljR/m71sfpHz1SSw36wSmSicBYJo/LsDk2gbhb7ig5rA== X-Google-Smtp-Source: AGHT+IFXzHgO+/pC4Ag3HusYdDAg0wnJo5lH1anWq5a2E5RStLO/9zmiJj/HBeWwUrvyUuQsAzbM X-Received: by 2002:a17:90b:915:b0:29b:5f8:1b4 with SMTP id bo21-20020a17090b091500b0029b05f801b4mr1329781pjb.49.1710420655799; Thu, 14 Mar 2024 05:50:55 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1710420655; cv=pass; d=google.com; s=arc-20160816; b=ujTk5PKuGPQpmHVlbgU2n6QLCwwMQ5H1nhiUNqRM2W+RfQieWyKofKs7gebeD77P/A U4CzwMHR6VRgve5DCl23I3OjHidNn45WHWd0akMTaLkmfV5fmXfIcMj3TObPq71KCO1S Wn54LnmBR+ipy6l3BX08WCWqNNvjGPBAeHeImIuZFtZhEFn1x4AgTtOF8hLOdj2YJZG4 manELWbnTK6boIyD/mWs9z2aDIcQFkW6ubGdaMuCiWRw7kDr4VxjkzvDq0FOueeO1sWA HNwXXDrxFpVg+4KoQP7zGcYl/spqwflsyp7PdB90tzPI71NTMIjfzvyral3EoxahNlGn Ss5A== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:list-unsubscribe:list-subscribe:list-id:precedence :dkim-signature; bh=ZghsaZNCzPLJdRj6WkVSNSS+VYpXaG51PVVgNZx+0NE=; fh=TdzahW/ULr8N4Osgbufv6tb9NQUtwdml91B762UwcJE=; b=jZb68YvrW/wWUyXN1cj9kCNKX4kSeeBHmmRCYxH6fkRORh7NAd8dIhbjeLnonzctdJ WhtB62lhItB7skQYT1hbJRk1CKMW6/K+KfAk+XjhxtPe8gzQWM3BsE4pPYqDINDPL1Iu THJHT+//vuYSKk8weizfbcbKU0Nz/zJZPJuatLwiqZrKCGHfvoGmz8XegEnxP9l67vVU En/nMdIU4V4Ewp+eT5lOY7dJ/WXtGwuwrJmSK31nMI7GnWomFmQrs3c6teBqqFY8tPda OGRYyupdj+UQLAaFyIK1XR1R3XUgHgaZKOY4QbNgoUvdY1LBDIpofTQBRIdVE99od2SA Yuww==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=krkfoaVp; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-103254-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-103254-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [2604:1380:40f1:3f00::1]) by mx.google.com with ESMTPS id sb8-20020a17090b50c800b0029bb297c136si580619pjb.67.2024.03.14.05.50.55 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 14 Mar 2024 05:50:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-103254-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=krkfoaVp; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-103254-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-103254-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id EC119B22A9E for ; Thu, 14 Mar 2024 12:50:48 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 610715A790; Thu, 14 Mar 2024 12:50:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="krkfoaVp" Received: from mail-yw1-f178.google.com (mail-yw1-f178.google.com [209.85.128.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C64095A4CB for ; Thu, 14 Mar 2024 12:50:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710420634; cv=none; b=aMz6cqk9KCs4wosWwipicOIsnkrLzg+4YYiMUVqWYIm5muowxhv1kK4zsVN33/0QPmx7bXShvIaSQeRtqbWCz1p22dcGmU7Fpzjh5IUDQG/DlwT2eM2wFUyMD9qjSZobnkeTCCFZ6U40Mv7D3dljf3rI0rNw9yGJosmJjsYo4iI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710420634; c=relaxed/simple; bh=SJYf0eYZHPK70nL/FMrYe0Mz5UbyrSCWCw2slFY0IME=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=m4pE0MJ+DRpcwyEGfuJar5iyiq0X1TfokxXjmSW7hoMKPC/EPdCSj2MFbyMuE2oRFR1Ki6I2MHKRXIqgQKAOwu80meb0ulg3svg6p0AMbI6MmtT1Lk8oX7O0B3+3OvMeaevGuF6yUMdo5Ctqmu0+OP2yvotLkxemfIjI/oebJ+M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=krkfoaVp; arc=none smtp.client-ip=209.85.128.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-yw1-f178.google.com with SMTP id 00721157ae682-60a0579a931so9392307b3.0 for ; Thu, 14 Mar 2024 05:50:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1710420632; x=1711025432; darn=vger.kernel.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=ZghsaZNCzPLJdRj6WkVSNSS+VYpXaG51PVVgNZx+0NE=; b=krkfoaVpzdmYYsbnyeydK+VkLipe+CHaD8ET3d2Mi9ggP24Ne0VbvsMtqlcK+588X9 tpAKtLzOGdv5Vwd3Y0HnG48iJvUE5IQ5DxHgJ6iwovZiqhBgBYrijU/WW+xAgOf7r6MA Mkh16o3WzYVYgWQz9YflmaE76MEi2u5DnH4GpEQZHZsci6uJqbEyBTxbgKCK8eJ20EWt xf84YNsNp5QUaXh58O7d0XIIHBAl82ggK3CG0TXUrmLJ+64jKbP4Ci0gLqdOy72q7ZdO ktcTaJntXBKdryBWCuYJ14Z609btODBbCgL7dr+PdfHmRljnxKnlZS5iPjdRQ7nhw3Ds D2Kg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710420632; x=1711025432; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ZghsaZNCzPLJdRj6WkVSNSS+VYpXaG51PVVgNZx+0NE=; b=rQbjx+i8epzkrZUzb6L+G/+lNpKOtAYQQ/jWALtlsJGiNQkwNd0bJS40Dhw+lN12SI Jxi3yLG9WzGyY3RYmZG6uguMqEtkTDT1zlF1FkVo3rcTwxFznQaBpBCe5EmI/FaWYh66 DCUbZqJaCzS/etK9PsEBww6eze18j0RrfMFFiLu9edWdV/3abx5192FKYLgoxvnU5ck1 vbefP/hZVl4ef3hbVbYtSarWOIZR9ddTNkoVyBnoB1lEELDPyeKJT+tLIUKAD7qrLVzV FT/ObC6i+enRg9RMMXwjSx/qHh3uqwGQUObSaY+Z5Hcf8/wk1pFNd3ppmH4orswDzcCO emfw== X-Forwarded-Encrypted: i=1; AJvYcCWJoIaRRgRCkLv2KrH+bFbjSiAqvBblB/gIxxhlW2dwYd+d2wrkwmZHgrkhLHyD7FunPCyWhWasQgS4PlQeR7hlGWAKNyjHagHzOnWb X-Gm-Message-State: AOJu0Yxzj5WdPpIslq5rJX5EdFow0Pfmhhf+BvwstepZLVztTlu4Lc6o uZGU+V6hE7Qk1VMY5DxmYClNU29gnaKR8TzXS0dE0BxN7CKlmW23/2L+tUsKX1Tsh3tYdiSXJVu R1EdVJDuLqvfUQe7VYlQn8Qd8/vkXzM2U7fO2nA== X-Received: by 2002:a0d:ea16:0:b0:609:fc2c:4d8c with SMTP id t22-20020a0dea16000000b00609fc2c4d8cmr632942ywe.28.1710420631754; Thu, 14 Mar 2024 05:50:31 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240313-videocc-sm8150-dt-node-v1-0-ae8ec3c822c2@quicinc.com> <20240313-videocc-sm8150-dt-node-v1-3-ae8ec3c822c2@quicinc.com> <1a2a7d7d-ed67-1c90-d3d1-4529dc07effa@quicinc.com> In-Reply-To: <1a2a7d7d-ed67-1c90-d3d1-4529dc07effa@quicinc.com> From: Dmitry Baryshkov Date: Thu, 14 Mar 2024 14:50:20 +0200 Message-ID: Subject: Re: [PATCH 3/3] arm64: dts: qcom: sm8150: Add video clock controller node To: "Satya Priya Kakitapalli (Temp)" Cc: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Michael Turquette , Stephen Boyd , Ajit Pandey , Imran Shaik , Taniya Das , Jagadeesh Kona , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Content-Type: text/plain; charset="UTF-8" On Thu, 14 Mar 2024 at 11:14, Satya Priya Kakitapalli (Temp) wrote: > > > On 3/14/2024 12:46 AM, Dmitry Baryshkov wrote: > > On Wed, 13 Mar 2024 at 13:11, Satya Priya Kakitapalli > > wrote: > >> Add device node for video clock controller on Qualcomm > >> SM8150 platform. > >> > >> Signed-off-by: Satya Priya Kakitapalli > >> --- > >> arch/arm64/boot/dts/qcom/sa8155p.dtsi | 4 ++++ > >> arch/arm64/boot/dts/qcom/sm8150.dtsi | 13 +++++++++++++ > >> 2 files changed, 17 insertions(+) > >> > >> diff --git a/arch/arm64/boot/dts/qcom/sa8155p.dtsi b/arch/arm64/boot/dts/qcom/sa8155p.dtsi > >> index ffb7ab695213..9e70effc72e1 100644 > >> --- a/arch/arm64/boot/dts/qcom/sa8155p.dtsi > >> +++ b/arch/arm64/boot/dts/qcom/sa8155p.dtsi > >> @@ -38,3 +38,7 @@ &rpmhpd { > >> */ > >> compatible = "qcom,sa8155p-rpmhpd"; > >> }; > >> + > >> +&videocc { > >> + power-domains = <&rpmhpd SA8155P_CX>; > >> +}; > >> diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi > >> index a35c0852b5a1..6573c907d7e2 100644 > >> --- a/arch/arm64/boot/dts/qcom/sm8150.dtsi > >> +++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi > >> @@ -14,6 +14,7 @@ > >> #include > >> #include > >> #include > >> +#include > >> #include > >> #include > >> #include > >> @@ -3715,6 +3716,18 @@ usb_2_dwc3: usb@a800000 { > >> }; > >> }; > >> > >> + videocc: clock-controller@ab00000 { > >> + compatible = "qcom,sm8150-videocc"; > >> + reg = <0 0x0ab00000 0 0x10000>; > >> + clocks = <&rpmhcc RPMH_CXO_CLK>, > >> + <&gcc GCC_VIDEO_AHB_CLK>; > >> + power-domains = <&rpmhpd SM8150_MMCX>; > >> + required-opps = <&rpmhpd_opp_low_svs>; > > Should not be necessary anymore. > > > Whenever the rail is turned on, we want to keep it in low_svs state > instead of retention, hence added this property , please let me know why > you think it is not needed? See https://lore.kernel.org/linux-arm-msm/20240226-rpmhpd-enable-corner-fix-v1-1-68c004cec48c@quicinc.com/ > > > >> + #clock-cells = <1>; > >> + #reset-cells = <1>; > >> + #power-domain-cells = <1>; > >> + }; > >> + > >> camnoc_virt: interconnect@ac00000 { > >> compatible = "qcom,sm8150-camnoc-virt"; > >> reg = <0 0x0ac00000 0 0x1000>; > >> > >> -- > >> 2.25.1 > >> > >> > > -- With best wishes Dmitry