On Tue, Dec 16, 2025 at 01:24:50PM +0100, David Heidelberg wrote:
> Usually, the supply is around 1.2 V, not 1.8 V. Rather remove mention of
> voltage from the description.
You did not remove the voltage. Anyway, both commits should be squashed.
>
> Fixes: 849139d46d09 ("media: dt-bindings: media: camss: Fixup vdda regulator
> descriptions sdm845")
> Signed-off-by: David Heidelberg <[email protected]>
> ---
> 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..4986d18d1a2da 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.
> + 1.2V regulator supply to CSIPHY IP blocks.
"...that supply is a regulator supply" so you still decided to go that
way. Why? I think I could not write this clearer.
Best regards,
Krzysztof
