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 D82DC36EAAE for ; Wed, 1 Apr 2026 11:21:22 +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=1775042484; cv=none; b=hz6DF6UQBnQWeBe95F4CFu4Y7kyhKRYxoPk281t9nWqmqou+2Ii4XtKiNhGBzailT7gl8sKS4WMcFC8T2aBCvJOKUyCA+xTJE4jmGWc8ydAZhyWAIwBu0yCDrSncG8sNHQvFJ9xTaNMjfvkvyGVGuF0vHScROMeuB8Hwc2uOR4A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775042484; c=relaxed/simple; bh=VMnhYYDUiaSfV3lX5JRhm4gQqAQh3ymN4z4y+uJ1Bss=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=OC4jXg0XYTJurMHJojqMjcnne0mswMpIkeT3adhQ24X0ZUlBy0u7WzC5j0sF6t8hk/wRCxg04EZQKMoaAuv38OjprB/tTfhy/c5475NVxLerw0XdVg6cSxPr23q5N3g9fbxmSXQRUmYkPhyssk+roidIMODT0cWaRDTXriJQp4c= 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=n3d/GjPU; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=KSCwkHTr; 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="n3d/GjPU"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="KSCwkHTr" Received: from pps.filterd (m0279865.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6317Z6PS1006686 for ; Wed, 1 Apr 2026 11:21:22 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= bSggxIu2zQD4DsBMlrluFZINoxno1QBalYiFALDNiKs=; b=n3d/GjPUdHAvz2CW 58H63oyVg70eCSDzS7dQOgfiC8USySrryu3pXHoemDi5bx38eASlTcAJsAeTPxC3 0yeC1PkMZjQ8djJXSGttCXqjQvr+pOsFHvyyO8G6AUUTB1gNhPQTtR0B4bGYE0Lr yyoljGFMQLcTwj65pvfraKcoQMhcycUvlR5gHZB7drZjH4JOatp4jJD7c64dUYoT EjpnjZRkfd+8G0X3ofKkmyJBHH3tGWVP6616I1q3F1ZhmMZkC4j6o8rAnteYbTq+ llGFuJF/LFcPsGspSs8YNDOFi2IoNL83CVqG0EvWADxoG1KbPbEhhHbubif9d53n 3LFBYQ== Received: from mail-pf1-f199.google.com (mail-pf1-f199.google.com [209.85.210.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4d8tfjj11u-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Wed, 01 Apr 2026 11:21:22 +0000 (GMT) Received: by mail-pf1-f199.google.com with SMTP id d2e1a72fcca58-82cdaf0f934so2684930b3a.0 for ; Wed, 01 Apr 2026 04:21:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1775042481; x=1775647281; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=bSggxIu2zQD4DsBMlrluFZINoxno1QBalYiFALDNiKs=; b=KSCwkHTr+UQy0WT0lMbT57libRfCbahSHcDj9kwy/vJFpVJgqDMsT4S+3RHKW5fn3T AA63vUskwsv2L47VFUvZWl27emOFglwYYyv/5V8o+2WMaDgAuPNdKnfu3Ga/Q2aI2Eqc fQT/sJpk88oPBWo33SV5PIRWJ/dfrOak7fKSvN16k5c8O9N94QGVQOMy3LvOAzliJRhQ 2qinwZW90ETAuTL0NtN1/dsCc0acNS+OCC8arcpWFC/LTSbyEaUlru5eWd9H1uCc+spF N1i/oTMvAijmtOzJVU+jN/RcsHN2uKXX8TmeIYb7BwQSrpopb/Hr7hqFxWcJkiG5LENv eDRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775042481; x=1775647281; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language: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=bSggxIu2zQD4DsBMlrluFZINoxno1QBalYiFALDNiKs=; b=JoU8tSbZl5mrUMwzJU+igTlIJETmqcOPzCgMQV/XdrgP3vCNJ7nKgnCA/YqSWi+ftk 2ckotGZVXDgrkf0SQ6OQWO/UHaaJhU8utFBWh2+ghwumN5xHj23kl8h1fpqybZnDY57t 7Fi/9EvUfTkysXlG+0TTXk54aOFw7kssdZRJj+WAnw1TU9WrYmWXsaq8Tge13aiXibuP 8GMuRqM6w47fu845yow0NOtCjijQr2aWDiRK5FAXXLCwIhIALyCSUAn81/DEFl5z3OGZ c6jUzIk2zBwvwmd200njVDCpydY8XcfRErqBZh8Nv8HkP7H8KLyfWMwbdfE29I1j58w7 Sb8A== X-Forwarded-Encrypted: i=1; AJvYcCVE10GEO6/aHWU1Ld0TIuhzhqY3gpvkGzCeKEypWRFIzHT5xOw6Y+l/nSLz+C/gaGFxVzez5IkjJLg5@vger.kernel.org X-Gm-Message-State: AOJu0Yw2WbEJILWJLFU8o1u0XGKSzXNoIiF3pxEtX284kF0h1biHyCBF maHXyhRVdJM4QGZFaXAovQhXJ27gAbNTZ8HsjVRYR4vHr+7onW7usPbrwrNMP+EJ8J3aFDXM7ZY yq7FKh3VNLYpJ2ZiABKGHinfXk0UihO65zFSg9yb3oRY5FWjpEnDKiiMcw0VBh3LR3Q3o4wTU X-Gm-Gg: ATEYQzxhpf/vLRJ7LRByiwqr7jJEsGLO+qY2y5ok0q9fJAuJ9qWjye8h9pApQ3HNerP NK61IShIUB3pYSbF5Ccq8YwBRBOMwfSVMLYkSOfE05Kw98GtI2O96LltogTUY3EIRyfNLOrJAha tCKvJ0ABkw9vJRzqVV+ZAovexcY2hkT8Y/+JVF/kQ6xsZ4BzP9RAu8R8YAkB405UJQmE1o+DCs3 WmkGfr3Au3Qg/KPKXRpAu6d45UUkf0xXCvIS8pZJaLLXlbAfEj109yEmdUtU7t+crX8IXvvLGUh E49/QSMigR6HsYPHjFfnp0wbqJfSBrtzILs5EcBBLHeGh9G+7/usWO48vY37Vv76MLATJ/2avJx O+7recSWXRfuwec13tGLtZ3ohyVDZGCqYq9gGb9ZzryRZ9+zOmAmtxQ== X-Received: by 2002:a05:6a00:22d4:b0:82c:e601:3601 with SMTP id d2e1a72fcca58-82ce8ae56d4mr3725103b3a.42.1775042481322; Wed, 01 Apr 2026 04:21:21 -0700 (PDT) X-Received: by 2002:a05:6a00:22d4:b0:82c:e601:3601 with SMTP id d2e1a72fcca58-82ce8ae56d4mr3725063b3a.42.1775042480682; Wed, 01 Apr 2026 04:21:20 -0700 (PDT) Received: from [10.0.0.3] ([106.222.233.247]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82caf97abfesm12659643b3a.1.2026.04.01.04.21.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 01 Apr 2026 04:21:20 -0700 (PDT) Message-ID: <5e1f6af3-c6d3-936e-70fe-6f7ddd472304@oss.qualcomm.com> Date: Wed, 1 Apr 2026 16:51:11 +0530 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: [PATCH v5 2/5] media: iris: scale MMCX power domain on SM8250 Content-Language: en-US To: Dmitry Baryshkov , Ulf Hansson Cc: Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Taniya Das , Jonathan Marek , "Rafael J. Wysocki" , Bryan O'Donoghue , Vikash Garodia , Mauro Carvalho Chehab , Stanimir Varbanov , Abhinav Kumar , Hans Verkuil , Stefan Schmidt , Konrad Dybcio , Bryan O'Donoghue , Dikshita Agarwal , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-media@vger.kernel.org, Mauro Carvalho Chehab References: <20260209-iris-venus-fix-sm8250-v5-0-0a22365d3585@oss.qualcomm.com> <20260209-iris-venus-fix-sm8250-v5-2-0a22365d3585@oss.qualcomm.com> <5e2635ac-35de-645b-b5e7-235923f844ce@oss.qualcomm.com> <0b41ee7c-83fe-d604-b750-8a5a0bd62bf8@oss.qualcomm.com> <6yfn6ux3gbin2dtopdxxv4okinspueryuey5vfdq633gsewc6r@ttp56brnpgvr> <4fiyjcqt5smotudsfzyqrevxxnx3sf5grbgfluzkndbp2od6pq@vlyikcvl3xkb> From: Dikshita Agarwal In-Reply-To: <4fiyjcqt5smotudsfzyqrevxxnx3sf5grbgfluzkndbp2od6pq@vlyikcvl3xkb> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-GUID: d-gQthDadVlVTFI9I4xQw0cXf37C5DO1 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDAxMDEwNCBTYWx0ZWRfXyTiFIAWNYSPT a5LyoSjo475cAqbb+en+Nb3jTlnfKIR1rkD5a+UUkeXl5SlA+e90zZ5cIy7sM5acJALL2g3a3Ox LY6TTUY88ECDP0HnuxYMQD8FkVdOa4L3FuTcPRrEucQBb3bqMzfVd4sxYanQiuQSyqzEeLSg6QP 3gSt0/NkMGCHWoqU5jBMGGllUFgXbT3ikVgaPEOcG9wyq04yqFzkoLb7izaY66RVa30Yl1LH1dq 7z45f5zg2HKoazwj+Q6c36HHc0GLi7yFjjDM6vU4kqwkrDSHqotfkTGx6PYToPVZ7t3in3XTPfZ aH1nMAf4S9VShuTeSmNUj1kO/IdzolroVL9jiXZIUGzTyaPYlLSsFE4cD0bcKO+zAOPyPfHnDjq OX8D3YXJDgExgeKrtqGR4z/UrvmjJQaIyJGdg+5uyzbI4nP844idWKbG01opnGrL0Dv/jlNwmLJ yPxUnY8fkSHSEinxQgA== X-Authority-Analysis: v=2.4 cv=fJc0HJae c=1 sm=1 tr=0 ts=69ccffb2 cx=c_pps a=WW5sKcV1LcKqjgzy2JUPuA==:117 a=5/Y9Gi2N1OwmQbPtUd2E/A==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=Um2Pa8k9VHT-vaBCBUpS:22 a=EUspDBNiAAAA:8 a=cwlDSPEQxbFMGEvUi3wA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=OpyuDcXvxspvyRM73sMx:22 X-Proofpoint-ORIG-GUID: d-gQthDadVlVTFI9I4xQw0cXf37C5DO1 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-04-01_03,2026-04-01_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 phishscore=0 suspectscore=0 lowpriorityscore=0 adultscore=0 priorityscore=1501 spamscore=0 clxscore=1015 bulkscore=0 impostorscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2603050001 definitions=main-2604010104 On 4/1/2026 4:31 PM, Dmitry Baryshkov wrote: > On Wed, Apr 01, 2026 at 12:46:01PM +0200, Ulf Hansson wrote: >> On Tue, 31 Mar 2026 at 20:46, Dmitry Baryshkov >> wrote: >>> >>> On Tue, Mar 31, 2026 at 01:33:35PM +0200, Ulf Hansson wrote: >>>> On Mon, 30 Mar 2026 at 15:06, Dikshita Agarwal >>>> wrote: >>>>> >>>>> >>>>> >>>>> On 3/30/2026 4:45 PM, Dmitry Baryshkov wrote: >>>>>> On Mon, Mar 30, 2026 at 10:55:02AM +0530, Dikshita Agarwal wrote: >>>>>>> >>>>>>> >>>>>>> On 2/9/2026 7:02 AM, Dmitry Baryshkov wrote: >>>>>>>> On SM8250 most of the video clocks are powered by the MMCX domain, while >>>>>>>> the PLL is powered on by the MX domain. Extend the driver to support >>>>>>>> scaling both power domains, while keeping compatibility with the >>>>>>>> existing DTs, which define only the MX domain. >>>>>>>> >>>>>>>> Fixes: 79865252acb6 ("media: iris: enable video driver probe of SM8250 SoC") >>>>>>>> Reviewed-by: Dikshita Agarwal >>>>>>>> Signed-off-by: Dmitry Baryshkov >>>>>>>> --- >>>>>>>> drivers/media/platform/qcom/iris/iris_platform_gen1.c | 2 +- >>>>>>>> drivers/media/platform/qcom/iris/iris_probe.c | 7 +++++++ >>>>>>>> 2 files changed, 8 insertions(+), 1 deletion(-) >>>>>>>> >>>>>>>> diff --git a/drivers/media/platform/qcom/iris/iris_platform_gen1.c b/drivers/media/platform/qcom/iris/iris_platform_gen1.c >>>>>>>> index df8e6bf9430e..aa71f7f53ee3 100644 >>>>>>>> --- a/drivers/media/platform/qcom/iris/iris_platform_gen1.c >>>>>>>> +++ b/drivers/media/platform/qcom/iris/iris_platform_gen1.c >>>>>>>> @@ -281,7 +281,7 @@ static const struct bw_info sm8250_bw_table_dec[] = { >>>>>>>> >>>>>>>> static const char * const sm8250_pmdomain_table[] = { "venus", "vcodec0" }; >>>>>>>> >>>>>>>> -static const char * const sm8250_opp_pd_table[] = { "mx" }; >>>>>>>> +static const char * const sm8250_opp_pd_table[] = { "mx", "mmcx" }; >>>>>>>> >>>>>>>> static const struct platform_clk_data sm8250_clk_table[] = { >>>>>>>> {IRIS_AXI_CLK, "iface" }, >>>>>>>> diff --git a/drivers/media/platform/qcom/iris/iris_probe.c b/drivers/media/platform/qcom/iris/iris_probe.c >>>>>>>> index 7b612ad37e4f..74ec81e3d622 100644 >>>>>>>> --- a/drivers/media/platform/qcom/iris/iris_probe.c >>>>>>>> +++ b/drivers/media/platform/qcom/iris/iris_probe.c >>>>>>>> @@ -64,6 +64,13 @@ static int iris_init_power_domains(struct iris_core *core) >>>>>>>> return ret; >>>>>>>> >>>>>>>> ret = devm_pm_domain_attach_list(core->dev, &iris_opp_pd_data, &core->opp_pmdomain_tbl); >>>>>>>> + /* backwards compatibility for incomplete ABI SM8250 */ >>>>>>>> + if (ret == -ENODEV && >>>>>>>> + of_device_is_compatible(core->dev->of_node, "qcom,sm8250-venus")) { >>>>>>>> + iris_opp_pd_data.num_pd_names--; >>>>>>>> + ret = devm_pm_domain_attach_list(core->dev, &iris_opp_pd_data, >>>>>>>> + &core->opp_pmdomain_tbl); >>>>>>>> + } >>>>>>>> if (ret < 0) >>>>>>>> return ret; >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> Hitting below compilation error on latest kernel >>>>>>> >>>>>>> drivers/media/platform/qcom/iris/iris_probe.c: In function >>>>>>> ‘iris_init_power_domains’: >>>>>>> drivers/media/platform/qcom/iris/iris_probe.c:71:46: error: decrement of >>>>>>> read-only member ‘num_pd_names’ >>>>>>> 71 | iris_opp_pd_data.num_pd_names--; >>>>>> >>>>>> See commit 7ad7f43e568b ("pmdomain: de-constify fields struct >>>>>> dev_pm_domain_attach_data") >>>> >>>> The intent was for this patch to be part of v7.0-rc1, but I failed >>>> with my pull-request to Linus. >>>> >>>> Instead this will be part of v7.1-rc1, assuming everything goes as expected. >>>> >>>> Is it possible to drop/defer these changes until v7.2? >>> >>> It would be very sad. >> >> Right. >> >> Since it's my mistake, let me reconsider. If I rebase my branch and >> share the necessary commit through an immutable branch that you can >> pull in. Would that work for you? > > I think that question goes to Vikash, Dikshita, Bryan and linux-media > maintainers. Bryan, what is the plan for this patchset? Should Ulf > rebase the branch? Or is this patchset delayed for some other reasons? Its good to go, Bryan can comment as he will be raising the PR. Thanks, Dikshita >