From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f44.google.com (mail-lf1-f44.google.com [209.85.167.44]) (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 361A8221299 for ; Thu, 16 Oct 2025 08:03:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760601828; cv=none; b=ohKtlO3KA1T8Li/pH70v3H2HTRR/dTXn+sSm67V4ir5fRaDzWAQT1HFYlXpZlPAMcqgEgOVmG24rAch+Vrlv0xfD3ldMzlpmHKKHoVaEKmxEgP1R4Cfu0BVNDaPAjQZ9x1l8qQ/MYQJ9OHcn5rkFIT+bURhNcGnYOk2gW0kbj7s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760601828; c=relaxed/simple; bh=Upf+vZPf5FtAc5nFbo/GsuIEVrSiRNpKkQuMnHoDkKY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ChEHVvqUwlTYySByL73yTwBzQKr/FFSeSTryoJdkR3iNzba/voDAChC6l3ZeITlIiWvEx38dId/s4oLPwXRULajvz4/Y+nKkRgdPWuQl/GWcXuS2yNvvGcsheAqxHB8B2awZ609aGrAt6Z54ovHPqP0m8f8+CBd2DWIVa3tJleU= 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=gWlGH7Os; arc=none smtp.client-ip=209.85.167.44 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 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="gWlGH7Os" Received: by mail-lf1-f44.google.com with SMTP id 2adb3069b0e04-57bd7f0e2daso72993e87.0 for ; Thu, 16 Oct 2025 01:03:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1760601824; x=1761206624; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to:subject :user-agent:mime-version:date:message-id:from:to:cc:subject:date :message-id:reply-to; bh=SI3iJaG6AehsZaDVHen+J6EpKqRu7s9HPBybFYzEdgw=; b=gWlGH7Os+/ZnhKC7HSZP/PbQmY20AL8hDGE96MbwYi57o/couwbx84q3/gZyYBA4dJ optOTlN5VLfwO+ZaAjT7MX8446a/jfp0CBtJO0LTADsCzz7PvIS3bitSYM6vAviYn8OX SfxXF0zzCqCLrbD3/ARobYWmVM0NU02IqvP2bb87vWi4soAfHwHhHu9c/5ifAkhgnuXg WJI0miFxLcfCFGkMKPQFIJWWIbgluWUPyS34r/wStU7eyf4fvT4Sd4zOg7rSxAiHp7Ah ZZq6EYUUocgvij1x3Vq3SX9KOnTzFSOEfU6Vm3Q1lEEhcqCHaosbJSoJoXM8oy16AYsg mbhg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760601824; x=1761206624; h=content-transfer-encoding:in-reply-to:from:references:cc:to:subject :user-agent:mime-version:date:message-id:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=SI3iJaG6AehsZaDVHen+J6EpKqRu7s9HPBybFYzEdgw=; b=mHmXimSR1so+X5xTpNGzUvAgvxIU/Lch1TaFvseBXdV3N9Y6I5MByeNow5MDGkhcaK X7wZqYj5ozlbm7SNfaK+Z2QntUFWNtKk8c8GekuZfuSFNfEDZkEk4yvhZSit0c4wsGnP +MYLPNFXjM0R2++FY1mp8JjgBoOWdPV1/4BnCvbw18iqr947eDb4lURYwazTdZ8B4aAv 6nA9b/Lhdtc94BSGmH5R7jDwKSvDWwVzo18H2a3PwF8erIXO6XDTCQLVo9zDN/hmk0pK i73UxdDPdv2Lj9ysJ9TsrCOwZjlFb5ctekf+RcmmVbkKriBKcKUvJ/ZhG4BeWVUgdAyi xaBA== X-Gm-Message-State: AOJu0YxagNI3Vo86Z31WXPktkCqStTocqikhV/Inlua8Wr27/jvfb5Hz FZryl/KtebzCjZFhKv957tukfkdcZeuTC98zOI9fUqb/McC/jxHJBkQ5nCGJScD3w9s= X-Gm-Gg: ASbGncvdqI+81P8LPRKw1Jun3Mg6+YhpvVl4epUO4T7ewoN3wF+Cv4kVg/hC4vJoGZ4 QYTVojMxKrCrdnnP02dj/IOq7kf/2ynwJR5X4CorVtwAj5Mr9LRj5MfN1LlGdJW7k1HRWthaSP0 SnF4HniL5IZtmIDXesgBGapKcFXhajzNKCH1yO0iQLQ96m8bTGXGjXli5LZ1kR/cI0mv8cZ7/96 5Q0L3fY6j0zhpFPVOGIFn6S7WovMuhqV0wZgvVA0aP8j55/PyGXVgsZyvgf6U/Z8sZ0sTlfx8nA jyOU4JRivEMMj9ua00PC43ryATfzfZdGjuNxaV+qTW81hEmF32mCfn7xwHxXK15tV8ieF1kfJAX gqZBm/RdKZGKVcuD26JsGcacfwwKOxkyYysfEO0nsa8GJAsKdhSGE8KBP6oHvdBq7FziOapdycr SVUOlx9xT9DRBEb2NuNZLonah4WP86v9Y+Lc3jCF5giNN8EGM8WpLxNY8eWxNxJ95MWV+jjg== X-Google-Smtp-Source: AGHT+IHcAxdLhGzdW1NI2UBF0IVQHMl6HFpUd/jJYBtOzICKf88GKwgE5spfrmYtn/8eULnHs4lGZg== X-Received: by 2002:a05:6512:3dac:b0:57a:8ae1:6a8e with SMTP id 2adb3069b0e04-5906d8ec20bmr5447249e87.6.1760601824219; Thu, 16 Oct 2025 01:03:44 -0700 (PDT) Received: from [192.168.1.100] (91-159-24-186.elisa-laajakaista.fi. [91.159.24.186]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-590881f88ddsm6922497e87.31.2025.10.16.01.03.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 16 Oct 2025 01:03:43 -0700 (PDT) Message-ID: <3202275a-25ec-435a-93ff-80a7ff8a8153@linaro.org> Date: Thu, 16 Oct 2025 11:03:42 +0300 Precedence: bulk X-Mailing-List: linux-i2c@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/6] dt-bindings: media: camss: Add qcom,kaanapali-camss binding To: Vijay Kumar Tumati , Bryan O'Donoghue , Hangxiang Ma , Loic Poulain , Robert Foss , Andi Shyti , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Todor Tomov , Mauro Carvalho Chehab Cc: linux-i2c@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, aiqun.yu@oss.qualcomm.com, tingwei.zhang@oss.qualcomm.com, trilok.soni@oss.qualcomm.com, yijie.yang@oss.qualcomm.com, Jingyi Wang , Atiya Kailany References: <20251014-add-support-for-camss-on-kaanapali-v2-0-f5745ba2dff9@oss.qualcomm.com> <20251014-add-support-for-camss-on-kaanapali-v2-2-f5745ba2dff9@oss.qualcomm.com> <0b6c157a-3d8d-4251-a704-31f8369f6a4e@linaro.org> <62a3f09b-50d6-4ace-8229-d71585378ae1@oss.qualcomm.com> From: Vladimir Zapolskiy In-Reply-To: <62a3f09b-50d6-4ace-8229-d71585378ae1@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 10/16/25 06:46, Vijay Kumar Tumati wrote: > > On 10/15/2025 4:32 PM, Vladimir Zapolskiy wrote: >> On 10/16/25 00:43, Bryan O'Donoghue wrote: >>> On 15/10/2025 20:45, Vladimir Zapolskiy wrote: >>>>> +  power-domains: >>>>> +    items: >>>>> +      - description: >>>>> +          TFE0 GDSC - Thin Front End, Global Distributed Switch >>>>> Controller. >>>>> +      - description: >>>>> +          TFE1 GDSC - Thin Front End, Global Distributed Switch >>>>> Controller. >>>>> +      - description: >>>>> +          TFE2 GDSC - Thin Front End, Global Distributed Switch >>>>> Controller. >>>>> +      - description: >>>>> +          Titan GDSC - Titan ISP Block Global Distributed Switch >>>>> Controller. >>>>> + >>>>> +  power-domain-names: >>>>> +    items: >>>>> +      - const: tfe0 >>>>> +      - const: tfe1 >>>>> +      - const: tfe2 >>>> >>>> Please remove all 'tfeX' power domains, they are not going to be >>>> utilized >>>> any time soon. >>>> >>>> When 'power-domains' list is just a single Titan GDSC, >>>> 'power-domain-names' >>>> property is not needed. >>> >>> Each one of these TFEs powers an individually power-collapsible TFEs. >>> >>> This is also so with the other xFE power-domains on previous SoC >>> generations. >> >> This is false, for instance there is no management of SFEx power domains >> in SM8550 or X1E80100 CAMSS in the upstrem, neither there is no >> management >> of SBI, IPE, BPS, CPP and so on GDSC power domans given by CAMCCs. >> >> TFEx is no more special, if it's unused, then it should not be added. > I agree with Bryan, if I understood the original comment correctly. This > is no different to the IFE0/1/2 GDSCs on SM8550. All the other modules > listed above (SFE, IPE, BPS etc.) are not supported by the CAMSS driver > and hence there is no management. However, we need to manage the TOP and > TFE0/1/2 GDSCs for the real time RDI paths. Agreed, at first glance I didn't find the management of the TFE power domains in the driver's change 3/6 from this series, however now I see it was added along with the 6/6 change. This my comment is answered, thank you. >> >>> >>> You'll need the TFEx power-domain to process any data on TFEx with the >>> 'lite' versions being tied to the TOP GDSC. >> >> When it is needed, the documentation will be updated accordingly, >> right now >> it is unknown what a data processing on TFEx looks like, it might happen >> that there will be separate device tree nodes for TFEx. >> >> TFEx power domains shall be removed right now, unti; a usecase in the >> upstream >> CAMSS appears to use them, I haven't seen such code at the moment. >> > We attach these power power domains by name in the corresponding driver. > For instance, the VFE driver attaches the TFE power domains mentioned > here and are exercised from vfe_set_power() -> vfe_get() > ->vfe_pm_domain_on(). You can also see the related codes with '.has_pd' > and '.pd_name' properties in the CAMSS subdev resource structures. Hope > this clarifies. > -- Best wishes, Vladimir