From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.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 5981A3A6B66 for ; Fri, 22 May 2026 09:09:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779440976; cv=none; b=AlTnSKyFUKmzZkKlFx3gHaYEdxqhE8x28sMFTwvTF0lW/neAR5H+Fpie0KJdjhDwQ9wyp+ruG3Dy84or+KmJ+7sU9lQxtLS7pkssckbM5zhnZQ5FDOpZPBmmfaNfFKjC2MK/kVlfF2IE5ZR89sIYGd1m4+EiTfSCcFrY0zTjpQg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779440976; c=relaxed/simple; bh=tvjnb+Uet6RnFiJb1YynFAL6Q40GRLj1mRo6yq1TEdg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=uLebHNiGpcsIJmmCd7+GBNtFqojzlrVuuT7FHL6bXLGzf3BreVj4UgSyu6saM4yhGk51zAH+d7IrG/uUHJ+y/GQyY+Z3tu9w7XmWt07H2BsR0eq5BHJt+JslGQktoJz3UxXLfE7T9meiqi6VErk9oHvcXP/hDbR6lnF2qvhOFR4= 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=Kt2PDFcW; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=JVOH+J9I; arc=none smtp.client-ip=205.220.180.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="Kt2PDFcW"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="JVOH+J9I" Received: from pps.filterd (m0279872.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 64M8o99o1816077 for ; Fri, 22 May 2026 09:09:34 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= QE39ajGQ40BKpXOOWM/P7CokESwSbt3jOVzNBooRXFU=; b=Kt2PDFcWVOG+KN9i /EDTBKBhXa2iF0r2ZCfF05sTkCyXC3LEu7HfJhosoS4lkqq4tq00j5osv0jFwD4W kc5NyHNbe9trU310IH9AP5jVGAPdeyrKqqi4xNJFtTH4oEd8I3SyidER282OmtbA orHkfNJ6y3psHYYdkaJrKL1Yn4GNYFc8ATFi0sE1UYT5J+1DTkZ+0Gvw8vDNYwcS lVqsYxUQ+0Y68CQaG303nMczNHwe2GBXCx3R+j3/gyLNuSunvqvUtkgvaMtYYaRQ b5EtNPly7WX2feSuicLf4/+ttJg7/j2AZS6IDwP0RqVs8j2ITDQGyWAz0TOBz+8K Xg/12g== Received: from mail-pj1-f72.google.com (mail-pj1-f72.google.com [209.85.216.72]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4eafrm17ub-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 22 May 2026 09:09:34 +0000 (GMT) Received: by mail-pj1-f72.google.com with SMTP id 98e67ed59e1d1-3663cbff31cso17250388a91.2 for ; Fri, 22 May 2026 02:09:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1779440973; x=1780045773; 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=QE39ajGQ40BKpXOOWM/P7CokESwSbt3jOVzNBooRXFU=; b=JVOH+J9IJrM+MT9khwiwlNkyvYAaU3OY4ksnfaIYiTpaRxvzmQdmGtVz6REhkyGAEG G1Fvhzy8gm9uJvfgz5ZVQzWAV4pSnRUoZKt9x5u710TTR+/dELLC1kIrEITd43KKxQfs +WhstFPD4T/sy31F/81QqctPpbYITI8mdZXlbFA7eaaotx5SnB+dth4ZhlNMtGt3eaVH uyI6zjduXPOWIGZNEDUjX44dZJ4OZOZUDYyfVqdERHAS/c5jkrZj2cQAYBOg9OZvs4FB yJ41uXuW6gGN+yhFHziEXGCpvMmIPfIjbWazBZsgms5sQ9uvp0cKk45qM4cKY5JdqzmJ ASGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779440973; x=1780045773; 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=QE39ajGQ40BKpXOOWM/P7CokESwSbt3jOVzNBooRXFU=; b=W55nkMR9sJ9X8MQ73z5EIZv6CCA+qqp6hKauQ/IG4OtnGIv/FGr2TnX9aAa0FWxOLs Eb4rzIfmQUeOda3aq9tEI+8/yKaCYBOQykXfbysF9KExKCkMh2TbjcqDDuwxDQ1x0IoS dNDZCSizswvHhUDSAcMjFHvsakTO7d3QFmgeU1+f8BZU1SyXuT7/YAO0G2gJPNL/Hrld FFnI+6oPwNEP8xbEmSDTAmXuHOAIUeVsBb/oLwJA7MRl0IZovx29V56AaDYSapsWIqew 8QFWieAbmHLyr0DVrvgPDZmF/zDQPORFtstKkggsynvJP1Q7DtFRTJg77dQtfuWcjnFF wwqw== X-Forwarded-Encrypted: i=1; AFNElJ9g8a7ogHNA5+FY8BRgWyHmShhYK1d7PWvwLxGv79pgxLj/3bcXSj1F7RWoLyDu5eCdv02IkDQrY/cbvSc=@vger.kernel.org X-Gm-Message-State: AOJu0YxUAyxMVbH2OZX3h8Pp3Nmxzhicb+xtBZTmifZNSAf/zmW2Dq4m BlrxiEeb2Y6Yf7sC2e5vUrzvxSf45OnEK95AW6D/zmuxoN+Sfx8WG8E4pr1GfMvu7hEps9SDam6 58F7jO+p8+kN2ivm+BH7VQRyNaUokkwmp5reZ6JsyQzhGrLJXHILDy3WI/0nXCBJCkx8= X-Gm-Gg: Acq92OEY5JKP/B242hilqG0HYKNuqKGxCb1QDeVXJZUXXH5HRq8YGbmgXVMvfyjnOFs 7qNvz4UQ2xn9rfgcM5P6MnEkEndC8T3kt2NiwT6vUBkKC93DmFrUFo42M8MzKjJJV5gn5XlkXFB 3dyJH9QSCpy1wDfc6oSx2wmu6fm1zFBET/WVCCvpoG8m4CIB8wXvMr2N1k5dxU0jLFQCcDCgMj4 +XObuYM2CJYX1mqutQUQFDefaOb9zp50Kp0EG88uqsjj7266MObMLMKPLEvqoFIUR2QM8tghPHQ bU8c2d8UGFSnOC5EzcFghhcE4Ye0QlO5K0rzntxibCSnGhYANCXEuTxijg32yHyUIS3BZnS9ju0 ej5Cj12UtYNi+EhVSvVlzPsQ/JIbJgGPYHxrPjEm9RWFeIRRl X-Received: by 2002:a17:90b:564f:b0:368:ed92:6f3 with SMTP id 98e67ed59e1d1-36a677ce506mr2573581a91.18.1779440973259; Fri, 22 May 2026 02:09:33 -0700 (PDT) X-Received: by 2002:a17:90b:564f:b0:368:ed92:6f3 with SMTP id 98e67ed59e1d1-36a677ce506mr2573549a91.18.1779440972660; Fri, 22 May 2026 02:09:32 -0700 (PDT) Received: from [10.218.5.114] ([202.46.22.19]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-36a72188499sm717523a91.4.2026.05.22.02.09.22 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 22 May 2026 02:09:32 -0700 (PDT) Message-ID: <143e095c-471b-4cd7-9395-1fa06fa50625@oss.qualcomm.com> Date: Fri, 22 May 2026 14:39:01 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 3/4] clk: qcom: Add support for Display Clock Controller on Shikra To: Dmitry Baryshkov Cc: Konrad Dybcio , Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Konrad Dybcio , Ajit Pandey , Taniya Das , Jagadeesh Kona , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260513-shikra-dispcc-gpucc-v1-0-5fd673146ab2@oss.qualcomm.com> <20260513-shikra-dispcc-gpucc-v1-3-5fd673146ab2@oss.qualcomm.com> <1cf064d0-14b9-489f-964f-614aeed4fcbd@oss.qualcomm.com> <1c0bb8ff-eaca-4779-9b2f-4d564abd8670@oss.qualcomm.com> Content-Language: en-US From: Imran Shaik In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Proofpoint-ORIG-GUID: joigKJhjyBIJUK8nh2zkGtxZmcU-DoZY X-Authority-Analysis: v=2.4 cv=Zekt8MVA c=1 sm=1 tr=0 ts=6a101d4e cx=c_pps a=RP+M6JBNLl+fLTcSJhASfg==:117 a=fChuTYTh2wq5r3m49p7fHw==:17 a=IkcTkHD0fZMA:10 a=NGcC8JguVDcA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=yx91gb_oNiZeI1HMLzn7:22 a=EUspDBNiAAAA:8 a=0XXBx609mjK7aBirURsA:9 a=QEXdDO2ut3YA:10 a=iS9zxrgQBfv6-_F4QbHw:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNTIyMDA4OSBTYWx0ZWRfX5WTzyjJfISfb Uc0MYQO7TG7/NDrkG9NYrwxELnUOyRDwwuJ+ct3QC0rLqK8oHc8+FBSf+jbJdcPI8xeZGsOVsbL UdUaVyIsGncvAhIeXL0Z5VgUz1BzxUplozaroMBgSyQcEtkBSx+ApEwTOOqY0QXDucbuZSP0k99 OZ9uE3/6pvJvSAjoY+axZ3zSEgyId3lTI8eJrm7EryIzL+nRFIMi/KBTolOa3KaKbtHKzpsMweH 7k0iU0MlnZM+cWRh9uIEdPk/RC9SqsWmlzs8Y0ZWo4yGUi2lijOFUYYtAQtEmc4XTdIzSe3rD+g TcJ3SfvwYrc9dGCbXtufCOc2r4mERjqRjVWzFJRGbclQCoBi9eR+2IpWgwrnjUegoKcPwCpj3KD CrVHuYHImiIRx8du5Pstne3VgWp4kWNGR3WUdQ0hnTFAZcLmvqR+sZJafDBkRvxsygiSvR5txl+ mj00P97/IrYqoMFcbqA== X-Proofpoint-GUID: joigKJhjyBIJUK8nh2zkGtxZmcU-DoZY X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-05-22_02,2026-05-18_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 malwarescore=0 adultscore=0 lowpriorityscore=0 phishscore=0 suspectscore=0 spamscore=0 bulkscore=0 clxscore=1015 priorityscore=1501 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2605130000 definitions=main-2605220089 On 20-05-2026 09:59 pm, Dmitry Baryshkov wrote: > On Tue, May 19, 2026 at 09:34:09AM +0530, Imran Shaik wrote: >> >> >> On 13-05-2026 08:38 pm, Dmitry Baryshkov wrote: >>> On Wed, May 13, 2026 at 04:51:03PM +0200, Konrad Dybcio wrote: >>>> On 5/13/26 4:06 PM, Dmitry Baryshkov wrote: >>>>> On Wed, May 13, 2026 at 05:01:16PM +0300, Dmitry Baryshkov wrote: >>>>>> On Wed, May 13, 2026 at 07:10:38PM +0530, Imran Shaik wrote: >>>>>>> Add a driver for the Display clock controller on Qualcomm Shikra SoC. >>>>>>> >>>>>>> Signed-off-by: Imran Shaik >>>>>>> --- >>>>>>> drivers/clk/qcom/Kconfig | 10 + >>>>>>> drivers/clk/qcom/Makefile | 1 + >>>>>>> drivers/clk/qcom/dispcc-shikra.c | 565 +++++++++++++++++++++++++++++++++++++++ >>>>>>> 3 files changed, 576 insertions(+) >>>>>>> >>>>>> >>>>>> Reviewed-by: Dmitry Baryshkov >>>>> >>>>> After comparing the files... >>>>> >>>>> Can we use dispcc-qcm2290.c instead? It uses clock-names instead of >>>>> clock-indices, but I think it should be fine to use clock-names as a >>>>> one-off. >>>> >>>> Or we can convert it to use indices, since those are stable for agatti >>>> too - the names would remain in the binding, just unused by the driver >>> >>> Either is fine for me. >>> >> >> Hi, >> >> In Agatti, apart from the clock-names difference, I see that the AHB/XO >> clocks are not handled as always-on via the probe and instead rely on pm_clk > > There is no pm_clk handling in Agatti driver. > >> style handling, whereas Shikra follows the newer pattern by marking required >> CBCRs as critical during probe. I think that attempting to modify this >> approach into Agatti may introduce unnecessary complexity. > > Well, you can start by explaining what caused the difference and the > result of those differences. > >> >> And the Agatti DISPCC doesn't have the DT_DSI1 bindings exposed, and >> updating this might break the ABI with respect to bindings, and DT. > > You can add Shikra-specific bindings. See how it's handled for other > dispcc drivers. > >> Given these and considering that Agatti is already stable, keeping the >> Shikra as separate GPUCC/DISPCC drivers is better to avoid the risk of >> regressions and complexity. > > I think you've provided arguments for merging two drivers. It would > allow us to modernize Agatti driver and also to make sure that both > platforms use the well-tested code pattern. > Sure Dmitry, but we would like to proceed with Shikra as-is now since it already follows the latest upstream conventions, and will handle Agatti modernization as a follow-up series to align and reuse Shikra drivers. Thanks, Imran