From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (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 9B3B52F5313 for ; Tue, 23 Jun 2026 15:50:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782229825; cv=none; b=ezltgRzzei5BYsnkMG8bLVbvtWZx7wB2VOVdAi2x1XCjBdwpXsWvTySwsqDKdQWorHjTi+whMeatetD/uEnAe8xCTMIchu6V8uMn43QMZizV820tej5NXRuXtVGEWpCgc15hg6NhY7C79WittI8848p5gUv1qRtB1AXjVhcKZmw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782229825; c=relaxed/simple; bh=q7Jc1YY3GrhcgpyOUs/OqzKf0RPP3KjpisQthhPq/3Y=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=b89jkFUCWJNoNE3Dc47cW0EDrcTLmF4vMvmPHkCl4MdLc0Angb5bEiXjfXcjwfWU3AwqXn06vb89mUH8joG3mk0mftmdhymDKCE7idBMMH1vbdiUMbpkAvxqTonwGmvBKCKbS3P7is5bd7AL5v8mAp/soRPuHvF2iHyP5rIwtro= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=OD3Wuu9R; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=PtjLiWUc; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="OD3Wuu9R"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="PtjLiWUc" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 65NC6lx53744787 for ; Tue, 23 Jun 2026 15:50:24 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= Ym3P2gofeeAjlRIrGWhUFZ53iCiesVQBX5oel0MUm0k=; b=OD3Wuu9RehYOeWG5 bVAJExNQBqYVK7WKimu8zDrs5acO+dqSR484E7mTf83B8QfCuu++cnDHuiQb137h slxiyjceb0ZgqgSIJQ78vIBLFEpaRHEirTsX/jJONZeLMprlV1U3MfgpuEkHG9KC 0RJMz8uYKLADBxv2DRtBC9y44X002FiqsSEWe/kaAyUJLVyH9N0ayIl2WbAf+UXO 7OtcvSiU70021CbFo+IAp+kcj6pLAG15QSBPTJ0/Xu/eeBSFGsGy8ght9iGJhKFa 6WrpZS81aFmwFl1jqRmxjuoQ7uVLL8T8A8Ohj2otvR9pSDCMUk7aguiw3RBBoKAg Tpx0Ow== Received: from mail-qv1-f71.google.com (mail-qv1-f71.google.com [209.85.219.71]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4eysv3gyks-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 23 Jun 2026 15:50:23 +0000 (GMT) Received: by mail-qv1-f71.google.com with SMTP id 6a1803df08f44-8dd70b510a3so185996d6.1 for ; Tue, 23 Jun 2026 08:50:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1782229823; x=1782834623; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Ym3P2gofeeAjlRIrGWhUFZ53iCiesVQBX5oel0MUm0k=; b=PtjLiWUcap99AMdj0FklGyHJy8koBU6m1ulHOVxQOB1dxlcPU0F3xAttejIiTu3ttr xujSaxB9Jv21Xlxpdm/AzHPC3YcZXIqZgMzvUo82VTteEsAFlZeVL2G8GB9E8VKkBVlt pUnHgr7Inm/GY0FiQLqus7bYYliPc0T4Yzx1hIWQj9qL0D4IJRiuwuezf+ocZH4WBLaz FIgwbYz+Toc1xeWZEey5Xvv1uD7qORJE5xein4Mk5JRMwAjBCGoqjxUzEAnquNuYNm8P bmi/8guy9TlW0Ge005M+2iSmzJL3oYvyGoPk/BCWxAkuXQyloTi9TIfU4UXbkWjPs24X AD5w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782229823; x=1782834623; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Ym3P2gofeeAjlRIrGWhUFZ53iCiesVQBX5oel0MUm0k=; b=LP3SpAehdMSLwHuyBr9tfoTw2fojYP2zfWAAtphf8XnkOnN8ieipZVCgFNhC6PZ2F9 z5Hdt35bq5eEsUOvLPPl1D4kobW8zEIibxxPtzTB8dIJH9YlGjbcbgsabV9LkUli9adO y7S56dpQ4Tqau8BVYMGtMvO6b4QsLARRj/K19GaFY+v2MEkHV+GJnP8CLQ5icsPoQ8J6 1FTNFmP6wiGOKMQgaQKX3ofEQ2tpGWgumX5mQ2E8q3cg5pXXuUInE/AHupAcIPo0J631 ay9VoLr0gE/q1NyiN5+N8TolFw3THIuusfbJDNtSGTb01M0eszu6brOKakLXl5Q33NHI L8gw== X-Forwarded-Encrypted: i=1; AFNElJ8ohru/96FIZh8GCcl8fpkOUlSkJ/uI0wFaZNDuVIEWGDmllg3u9NQqT2Tlr+zCFdAG0TJZszuDi1FM@vger.kernel.org X-Gm-Message-State: AOJu0YwWSqPTaYsziH7C2yZ0NqDTCi8DDaNYPtz7x7NyWHnKU0n6TTRW 4pb4NURYKqmV+nVsEO4uW9knLnWacPX+rqPKV9r8+/ZdI8KvBy/wKd1Urn7OsURjSqPUPV+6C18 KXxVGT8v1hJW7nUVLSpEgUs8bp3ZUPjx1OxjM66sJvIwyGCkJAFDx/fsyn28+Fmxf X-Gm-Gg: AfdE7ckW/sjgCYZABMHc0lZNUDF9iPCAtl3j926vf/w2Pk1b99Ep0b3VmmS45e/Xicj 9qjxAzc1S5qp165qNIcP1nrzIYK0iFbVyD+kTUlR1U8TtrHbp0i7b0+TfoQjxTV7V8f8+p6AfRN j66EHH+JHMTXaHLip8uFw4SyH+8AqWEb1t0fUtrkanS0I4pQ1e/eB8sU5ehcDO6LkhWu0tSKgwj eDyF4mj8OR+9pX4VggoWh7VPEeuMJ1GZ2KUQ5NQntitbjbvVjYeDB/iFFPxFGlhzDMoxphmUmXv 0qLkkMzwAPxO+yH7TgemTt/rr9M1XkcMVxxN/g/yxbuGd6xF3brkV6z3Fzc4LK5w37In8abDk9j WKDd9ZYUInilXHXOlnlBGG4Vort+mmMCiq4I= X-Received: by 2002:ac8:7e8a:0:b0:50b:5286:f756 with SMTP id d75a77b69052e-51a4f4d4c47mr51048401cf.6.1782229822764; Tue, 23 Jun 2026 08:50:22 -0700 (PDT) X-Received: by 2002:ac8:7e8a:0:b0:50b:5286:f756 with SMTP id d75a77b69052e-51a4f4d4c47mr51047741cf.6.1782229822025; Tue, 23 Jun 2026 08:50:22 -0700 (PDT) Received: from [192.168.120.170] ([178.235.128.140]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-697ee52f216sm32857a12.3.2026.06.23.08.50.19 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 23 Jun 2026 08:50:21 -0700 (PDT) Message-ID: <65873506-1a9a-40ec-ac67-60f61a0b4b4c@oss.qualcomm.com> Date: Tue, 23 Jun 2026 17:50:18 +0200 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/8] clk: qcom: dispcc-sm8450: Fix mdss clocks To: esteuwu@proton.me, Bjorn Andersson , Michael Turquette , Stephen Boyd , Brian Masney , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Rob Clark , Will Deacon , Robin Murphy , "Joerg Roedel (AMD)" , Vinod Koul , Neil Armstrong Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, iommu@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org References: <20260622-sm8450-qol-v1-0-37e2ee8df9da@proton.me> <20260622-sm8450-qol-v1-1-37e2ee8df9da@proton.me> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20260622-sm8450-qol-v1-1-37e2ee8df9da@proton.me> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Info: AW1haW4tMjYwNjIzMDEzMCBTYWx0ZWRfXwPaBtLejWevS eLnVfO3Sq2KuyfP8DiMlmLiO+e6dj8IhM/9lRHhr5stGqxxTSX7YH33W/hmwwfcnp0Bc12Y7FrF oDkLQXqWlOJVvCtMPjLUQm0QLMQUXJw= X-Authority-Analysis: v=2.4 cv=WtAb99fv c=1 sm=1 tr=0 ts=6a3aab3f cx=c_pps a=UgVkIMxJMSkC9lv97toC5g==:117 a=PRfkaYvzSr8QmIIGAkY2Sg==:17 a=IkcTkHD0fZMA:10 a=FelO9ux0wxsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=DJpcGTmdVt4CTyJn9g5Z:22 a=1aW_qPit1WSy-zhaZFYA:9 a=QEXdDO2ut3YA:10 a=1HOtulTD9v-eNWfpl4qZ:22 X-Proofpoint-ORIG-GUID: sF3Iy-P-UIFr9UVTH23UBzWalnBaOpXD X-Proofpoint-GUID: sF3Iy-P-UIFr9UVTH23UBzWalnBaOpXD X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNjIzMDEzMCBTYWx0ZWRfX7GGhHGiKW9tf gwRhPZfwzIKbsDn+nRTokWtNSJ329KK9/ABzSoR6jKnAlHpoc6mqUeXgBTg95e3zVy1iBliZfxG AVdfLx9pe6VRC5SjkLV8v22+y904UWBLLZ18ou6k0A26ipVfuBlo5VZ+1A/f10cdkB3/low5qfW ndCVZaqqgrU/DtNoxnBZp5/o3pNJQQ21y24hyXAVKTsgbJvD7e/tyEOldGF4jnJ+WvkJGGO6Bfg tJrk3UQHbzBJEzKJgKUp2/h2u/82drcHZ+xTk51SqtDBEhR+A1de0qwfMSVQzIVmYi4wCoRxkFy gh+6EYRaOTT5pgRvg2+v9HzmHuxlhoalfiyha3jeJWvYbiKq83m1fNhvKxmRFKy3AK2FtQ1qyxP wPaIKP4z3vL+PwPUmYjxH1hBUNQlLOSVn4sXTSO+Wby7no40yKjtsI4Mi6R5uBMbmuogm6e8VXa 60BzGUO+5/Uv821mFNQ== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.125,FMLib:17.12.100.49 definitions=2026-06-23_03,2026-06-23_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 lowpriorityscore=0 clxscore=1015 spamscore=0 adultscore=0 priorityscore=1501 suspectscore=0 phishscore=0 malwarescore=0 impostorscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2606230130 On 6/23/26 2:54 AM, Esteban Urrutia via B4 Relay wrote: > From: Esteban Urrutia > > Both of these changes allow the framebuffer to show upon boot and let > the mdss driver take over afterwards. > Before, none of these actions were possible. Only mdss takeover was > possible, but screen had to be turned off first. > > OLE configuration may have been a misinterpretation... that's not > something that's done on the downstream driver. > > Changing disp_cc_mdss_mdp_clk_src from clk_rcg2_shared_ops to > clk_rcg2_shared_no_init_park_ops fixes this warning as well: [...] > drivers/clk/qcom/dispcc-sm8450.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/clk/qcom/dispcc-sm8450.c b/drivers/clk/qcom/dispcc-sm8450.c > index 2e91332dd92a..b99d3eb5e195 100644 > --- a/drivers/clk/qcom/dispcc-sm8450.c > +++ b/drivers/clk/qcom/dispcc-sm8450.c > @@ -614,7 +614,7 @@ static struct clk_rcg2 disp_cc_mdss_mdp_clk_src = { > .parent_data = disp_cc_parent_data_5, > .num_parents = ARRAY_SIZE(disp_cc_parent_data_5), > .flags = CLK_SET_RATE_PARENT, > - .ops = &clk_rcg2_shared_ops, > + .ops = &clk_rcg2_shared_no_init_park_ops, > }, > }; > > @@ -1824,8 +1824,8 @@ static int disp_cc_sm8450_probe(struct platform_device *pdev) > disp_cc_pll1.regs = clk_alpha_pll_regs[CLK_ALPHA_PLL_TYPE_LUCID_OLE]; > disp_cc_pll1.clkr.hw.init = &sm8475_disp_cc_pll1_init; > > - clk_lucid_ole_pll_configure(&disp_cc_pll0, regmap, &sm8475_disp_cc_pll0_config); > - clk_lucid_ole_pll_configure(&disp_cc_pll1, regmap, &sm8475_disp_cc_pll1_config); > + clk_lucid_evo_pll_configure(&disp_cc_pll0, regmap, &sm8475_disp_cc_pll0_config); > + clk_lucid_evo_pll_configure(&disp_cc_pll1, regmap, &sm8475_disp_cc_pll1_config); > } else { > clk_lucid_evo_pll_configure(&disp_cc_pll0, regmap, &disp_cc_pll0_config); > clk_lucid_evo_pll_configure(&disp_cc_pll1, regmap, &disp_cc_pll1_config); This can also be fixed by migrating to use qcom_cc_driver_data, which takes a list of alpha PLLs to be configured, and thenthere's a switch-statement in clk-alpha-pll.c that always assigns the correct function Konrad