linux-kernel - Re: [PATCH] media: dt-bindings: Correct camss VDDA PLL supply description

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b9b84317-a392-4f1e-aa8f-351e92c678a1@ixit.cz>
Date: Sun, 14 Dec 2025 10:35:14 +0100
From: David Heidelberg <david@...t.cz>
To: Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
 Robert Foss <rfoss@...nel.org>, Todor Tomov <todor.too@...il.com>,
 Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Vikram Sharma <quic_vikramsa@...cinc.com>,
 Kapatrala Syed <akapatra@...cinc.com>,
 Hariram Purushothaman <hariramp@...cinc.com>,
 Richard Acayan <mailingradian@...il.com>
Cc: linux-media@...r.kernel.org, linux-arm-msm@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 phone-devel@...r.kernel.org
Subject: Re: [PATCH] media: dt-bindings: Correct camss VDDA PLL supply
 description

On 14/12/2025 01:04, Vladimir Zapolskiy wrote:
> On 12/13/25 00:55, David Heidelberg via B4 Relay wrote:
>> From: David Heidelberg <david@...t.cz>
>>
>> Usually, the supply is around 1.2 V, not 1.8 V. Rather remove mention of
>> voltage from the description.
>>
>> Fixes: 849139d46d09 ("media: dt-bindings: media: camss: Fixup vdda 
>> regulator descriptions sdm845")
>> Signed-off-by: David Heidelberg <david@...t.cz>
>> ---
>> Added only Fixes tag for the initial commit, not all the copy-paste
>> propagated ones.
> 
> Generally it should be possible to specify a list of Fixes: tags.
> 
> However, despite the change is the proper one, there is an opinion that 
> Fixes
> tags are not applicable to dt bindings with a given reason that only 
> executed
> code can be fixed. So you may be asked to remove even a single Fixes tag.
> 
>>   Documentation/devicetree/bindings/media/qcom,sa8775p-camss.yaml  | 2 +-
>>   Documentation/devicetree/bindings/media/qcom,sc7280-camss.yaml   | 2 +-
>>   Documentation/devicetree/bindings/media/qcom,sc8280xp-camss.yaml | 2 +-
>>   Documentation/devicetree/bindings/media/qcom,sdm670-camss.yaml   | 2 +-
>>   Documentation/devicetree/bindings/media/qcom,sdm845-camss.yaml   | 2 +-
>>   Documentation/devicetree/bindings/media/qcom,sm8250-camss.yaml   | 2 +-
>>   6 files changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sa8775p- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sa8775p- 
>> camss.yaml
>> index 019caa2b09c32..9009cfe993d75 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sa8775p-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sa8775p-camss.yaml
>> @@ -130,7 +130,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>     ports:
>>       $ref: /schemas/graph.yaml#/properties/ports
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sc7280- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sc7280- 
>> camss.yaml
>> index ee35e3bc97ffd..cb922f90fe900 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sc7280-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sc7280-camss.yaml
>> @@ -129,7 +129,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>     ports:
>>       $ref: /schemas/graph.yaml#/properties/ports
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sc8280xp- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sc8280xp- 
>> camss.yaml
>> index c99fe4106eee9..2231d7216f62a 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sc8280xp-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sc8280xp-camss.yaml
>> @@ -268,7 +268,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>   required:
>>     - clock-names
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sdm670- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sdm670- 
>> camss.yaml
>> index 35c40fe223767..8e6ca94c88695 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sdm670-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sdm670-camss.yaml
>> @@ -95,7 +95,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>     ports:
>>       $ref: /schemas/graph.yaml#/properties/ports
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sdm845- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sdm845- 
>> camss.yaml
>> index 82bf4689d3300..d50e096b900db 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sdm845-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sdm845-camss.yaml
>> @@ -211,7 +211,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>   required:
>>     - clock-names
>> diff --git a/Documentation/devicetree/bindings/media/qcom,sm8250- 
>> camss.yaml b/Documentation/devicetree/bindings/media/qcom,sm8250- 
>> camss.yaml
>> index ebf68ff4ab961..ccd2d024bfd10 100644
>> --- a/Documentation/devicetree/bindings/media/qcom,sm8250-camss.yaml
>> +++ b/Documentation/devicetree/bindings/media/qcom,sm8250-camss.yaml
>> @@ -300,7 +300,7 @@ properties:
>>     vdda-pll-supply:
>>       description:
>> -      Phandle to 1.8V regulator supply to PHY refclk pll block.
>> +      Phandle to regulator supply to PHY refclk pll block.
>>   required:
>>     - clock-names
>>
> 
> As for the change itself it is good, I'd prefer to get a correction
> to 1.2V in the documentation rather than a complete removal though.

What I'm seeing, while the vdda-pll is usually close to 1.2V, it seems 
to me that's not everytime the case.

Can you confirm it should be exactly 1.2V, if yes, I'll clarify it in 
next version.

David>
> In any case,
> 
> Reviewed-by: Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>
> 

-- 
David Heidelberg