Re: [PATCH v2 2/5] dt-bindings: media: qcom,sm8150-camss: Add SM8150 CAMSS

[Bryan O'Donoghue <bod@kernel.org>]

On 06/11/2026 11:20, Vladimir Zapolskiy wrote:
>> +maintainers:
>> +  - Andrew Gigena<mail@andrewgigena.dev>
>> +
>> +description: |
>> +  The CAMSS IP is a CSI decoder and ISP present on Qualcomm platforms.
>> +
>> +properties:
>> +  compatible:
>> +    const: qcom,sm8150-camss
>> +
>> +  clocks:
>> +    minItems: 27
>> +    maxItems: 27
>> +
>> +  clock-names:
>> +    items:
>> +      - const: camnoc_axi
>> +      - const: cpas_ahb
>> +      - const: csiphy0
>> +      - const: csiphy0_timer
>> +      - const: csiphy1
>> +      - const: csiphy1_timer
>> +      - const: csiphy2
>> +      - const: csiphy2_timer
>> +      - const: csiphy3
>> +      - const: csiphy3_timer
> I ask you to switch to the new CSI2 PHY driver, please exclude these I/O
> spaces from the CAMSS container device.
> 
> It makes no sense to deal with all-phy-combined supplies and the multitude
> of other solved problems.
> 
> The cost of changing the device to exploit the new CSIPHY driver is
> close to zero.
> 
> --
> Best wishes,
> Vladimir

So I'm fine to take the patch as is - but if Andrew _wants_ to take on 
switching to csiphy driver - that would be a welcome series ofc.

Its up to you Andrew.

---
bod