[PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description

Krzysztof Kozlowski posted 1 patch 3 months, 1 week ago
.../devicetree/bindings/media/qcom,x1e80100-camss.yaml          | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Krzysztof Kozlowski 3 months, 1 week ago
Correct description of the CSIPHY 1.2 V supply
("vdd-csiphy-1p2-supply"), because it supplies 1.2 V, confirmed with DTS
on the mailing lists.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/media/qcom,x1e80100-camss.yaml          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/media/qcom,x1e80100-camss.yaml b/Documentation/devicetree/bindings/media/qcom,x1e80100-camss.yaml
index b075341caafc..b87a13479a4b 100644
--- a/Documentation/devicetree/bindings/media/qcom,x1e80100-camss.yaml
+++ b/Documentation/devicetree/bindings/media/qcom,x1e80100-camss.yaml
@@ -124,7 +124,7 @@ properties:
 
   vdd-csiphy-1p2-supply:
     description:
-      Phandle to 1.8V regulator supply to a PHY.
+      Phandle to 1.2V regulator supply to a PHY.
 
   ports:
     $ref: /schemas/graph.yaml#/properties/ports
-- 
2.48.1
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago
On 28/10/2025 12:11, Krzysztof Kozlowski wrote:
> Correct description of the CSIPHY 1.2 V supply
> ("vdd-csiphy-1p2-supply"), because it supplies 1.2 V, confirmed with DTS
> on the mailing lists.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/media/qcom,x1e80100-camss.yaml          | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)


Ping! It has been 1.5 months already. This also got other DT ack - TWO
DAYS after the posting - so why is this still not addressed by camss
maintainers?

Best regards,
Krzysztof
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago
On 17/12/2025 16:41, Krzysztof Kozlowski wrote:
> On 28/10/2025 12:11, Krzysztof Kozlowski wrote:
>> Correct description of the CSIPHY 1.2 V supply
>> ("vdd-csiphy-1p2-supply"), because it supplies 1.2 V, confirmed with DTS
>> on the mailing lists.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>  .../devicetree/bindings/media/qcom,x1e80100-camss.yaml          | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> 
> Ping! It has been 1.5 months already. This also got other DT ack - TWO
> DAYS after the posting - so why is this still not addressed by camss
> maintainers?
> 

Huh, my bad, this was applied. There is just no trace for it here, so I
missed that. Apologies for the noise.

There is awesome tool, btw, called b4, and with one-command setup you
can then later generate nice thank you letters (and with few more
commands setup they can be nicely customized)...

Best regards,
Krzysztof
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Bryan O'Donoghue 1 month, 3 weeks ago
On 17/12/2025 16:00, Krzysztof Kozlowski wrote:
>> Ping! It has been 1.5 months already. This also got other DT ack - TWO
>> DAYS after the posting - so why is this still not addressed by camss
>> maintainers?
>>
> Huh, my bad, this was applied. There is just no trace for it here, so I
> missed that. Apologies for the noise.
> 
> There is awesome tool, btw, called b4, and with one-command setup you
> can then later generate nice thank you letters (and with few more
> commands setup they can be nicely customized)...
> 
> Best regards,
> Krzysztof
> 

I think you _should_ have received a b4 ty when that hit media-next ..

---
bod
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago
On 18/12/2025 07:22, Bryan O'Donoghue wrote:
> On 17/12/2025 16:00, Krzysztof Kozlowski wrote:
>>> Ping! It has been 1.5 months already. This also got other DT ack - TWO
>>> DAYS after the posting - so why is this still not addressed by camss
>>> maintainers?
>>>
>> Huh, my bad, this was applied. There is just no trace for it here, so I
>> missed that. Apologies for the noise.
>>
>> There is awesome tool, btw, called b4, and with one-command setup you
>> can then later generate nice thank you letters (and with few more
>> commands setup they can be nicely customized)...
>>
>> Best regards,
>> Krzysztof
>>
> 
> I think you _should_ have received a b4 ty when that hit media-next ..

I checked before pinging, look:

https://lore.kernel.org/all/730230b1-d753-4c64-9c38-0e86a184c0fa@kernel.org/

Nothing that this was applied.

Best regards,
Krzysztof
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Bryan O'Donoghue 1 month, 3 weeks ago
On 18/12/2025 06:44, Krzysztof Kozlowski wrote:
>> I think you_should_ have received a b4 ty when that hit media-next ..
> I checked before pinging, look:
> 
> https://lore.kernel.org/all/730230b1-d753-4c64-9c38-0e86a184c0fa@kernel.org/
> 
> Nothing that this was applied.
> 
> Best regards,
> Krzysztof

Strange.

I'll look into why this doesn't happen, my understanding was that it 
does even though I don't do the b4 ty's myself.

---
bod
Re: [PATCH] media: dt-bindings: qcom,x1e80100-camss: Fix typo in CSIPHY supply description
Posted by Rob Herring (Arm) 3 months, 1 week ago
On Tue, 28 Oct 2025 12:11:16 +0100, Krzysztof Kozlowski wrote:
> Correct description of the CSIPHY 1.2 V supply
> ("vdd-csiphy-1p2-supply"), because it supplies 1.2 V, confirmed with DTS
> on the mailing lists.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/media/qcom,x1e80100-camss.yaml          | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>