* [PATCH v2] media: dt-bindings: Correct camss VDDA PLL supply description
@ 2025-12-13 9:19 David Heidelberg via B4 Relay
2025-12-17 0:59 ` Rob Herring (Arm)
0 siblings, 1 reply; 3+ messages in thread
From: David Heidelberg via B4 Relay @ 2025-12-13 9:19 UTC (permalink / raw)
To: Robert Foss, Todor Tomov, Bryan O'Donoghue,
Vladimir Zapolskiy, Mauro Carvalho Chehab, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Vikram Sharma, Kapatrala Syed,
Hariram Purushothaman, Richard Acayan, Bryan O'Donoghue,
Hans Verkuil
Cc: linux-media, linux-arm-msm, devicetree, linux-kernel, phone-devel,
David Heidelberg
From: David Heidelberg <david@ixit.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@ixit.cz>
---
Added only Fixes tag for the initial commit, not all the copy-paste
propagated ones.
---
Changes in v2:
- Applied suggestion to clarify the description. (Krzysztof)
- Link to v1: https://lore.kernel.org/r/20251212-docs-camss-fixes-v1-1-5c011505ff59@ixit.cz
---
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..cc48c0e50853d 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.
+ PHY refclk PLL block supply.
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..c9464192373d3 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.
+ PHY refclk PLL block supply.
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..6641084449670 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.
+ PHY refclk PLL block supply.
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..26fba2c0ceca5 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.
+ PHY refclk PLL block supply.
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..34428171d6e38 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.
+ PHY refclk PLL block supply.
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..0b3a3d0e26453 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.
+ PHY refclk PLL block supply.
required:
- clock-names
---
base-commit: d9771d0dbe18dd643760431870a6abf9b0866bb0
change-id: 20251212-docs-camss-fixes-0fa525271951
Best regards,
--
David Heidelberg <david@ixit.cz>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] media: dt-bindings: Correct camss VDDA PLL supply description
2025-12-13 9:19 [PATCH v2] media: dt-bindings: Correct camss VDDA PLL supply description David Heidelberg via B4 Relay
@ 2025-12-17 0:59 ` Rob Herring (Arm)
2025-12-17 9:08 ` David Heidelberg
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring (Arm) @ 2025-12-17 0:59 UTC (permalink / raw)
To: David Heidelberg
Cc: Mauro Carvalho Chehab, Todor Tomov, Kapatrala Syed, phone-devel,
devicetree, linux-arm-msm, Hans Verkuil, Robert Foss,
Bryan O'Donoghue, linux-kernel, Conor Dooley,
Bryan O'Donoghue, Krzysztof Kozlowski, Hariram Purushothaman,
Vikram Sharma, Richard Acayan, linux-media, Vladimir Zapolskiy
On Sat, 13 Dec 2025 10:19:31 +0100, David Heidelberg wrote:
> 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@ixit.cz>
> ---
> Added only Fixes tag for the initial commit, not all the copy-paste
> propagated ones.
> ---
> Changes in v2:
> - Applied suggestion to clarify the description. (Krzysztof)
> - Link to v1: https://lore.kernel.org/r/20251212-docs-camss-fixes-v1-1-5c011505ff59@ixit.cz
> ---
> 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(-)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] media: dt-bindings: Correct camss VDDA PLL supply description
2025-12-17 0:59 ` Rob Herring (Arm)
@ 2025-12-17 9:08 ` David Heidelberg
0 siblings, 0 replies; 3+ messages in thread
From: David Heidelberg @ 2025-12-17 9:08 UTC (permalink / raw)
To: Rob Herring (Arm)
Cc: Mauro Carvalho Chehab, Todor Tomov, Kapatrala Syed, phone-devel,
devicetree, linux-arm-msm, Hans Verkuil, Robert Foss,
Bryan O'Donoghue, linux-kernel, Conor Dooley,
Bryan O'Donoghue, Krzysztof Kozlowski, Hariram Purushothaman,
Vikram Sharma, Richard Acayan, linux-media, Vladimir Zapolskiy
On 17/12/2025 01:59, Rob Herring (Arm) wrote:
>
> On Sat, 13 Dec 2025 10:19:31 +0100, David Heidelberg wrote:
>> 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@ixit.cz>
>> ---
>> Added only Fixes tag for the initial commit, not all the copy-paste
>> propagated ones.
>> ---
>> Changes in v2:
>> - Applied suggestion to clarify the description. (Krzysztof)
>> - Link to v1: https://lore.kernel.org/r/20251212-docs-camss-fixes-v1-1-5c011505ff59@ixit.cz
>> ---
>> 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(-)
>>
>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
Thanks Rob,
meanwhile I sent v3 with 2 commits narrowing both supplies (just as I
noticed with old commit msg..), it's here:
https://lore.kernel.org/all/20251216-docs-camss-fixes-v3-0-c238b6810771@ixit.cz/
David
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-12-17 9:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-13 9:19 [PATCH v2] media: dt-bindings: Correct camss VDDA PLL supply description David Heidelberg via B4 Relay
2025-12-17 0:59 ` Rob Herring (Arm)
2025-12-17 9:08 ` David Heidelberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).