Re: [PATCH 0/2] media: qcom: camss: track PHY type and scale link frequency for C-PHY
From: Anusha Arun Nandi
Date: Mon Oct 19 2026 - 18:56:39 EST
Hi Nihal, Krzyszto,
You're right. The driver itself does not depend on the DTS. Apologies
for the confusion.
The Depends-on was included purely for testing purposes. During
validation on sa8775p, qcs8300, and sc7280, we needed the corresponding
DTSI changes to instantiate the C-PHY consumer nodes so the driver could
probe and be exercised on real hardware. Without those device-tree
nodes, the driver simply wouldn't bind on those platforms in our test
environments.
Since this is a C-PHY consumer driver, the DTS patches are what enable
the PHY nodes that the driver attaches to. They are a prerequisite for
functional testing, not for compilation or correctness of the driver
code itself. The driver is self-contained and will build and function
correctly independent of those DTS changes.
I will drop the Depends-on from the next revision.
Apologies for the noise.
Thanks, Anusha
On 10/17/2026 10:35 AM, Krzysztof Kozlowski wrote:
> On 17/10/2026 19:30, Nihal Kumar Gupta wrote:
>>
>>
>> On 17-10-2026 09:22, Anusha Arun Nandi wrote:
>>> Depends-on: https://lore.kernel.org/all/20261017001116.2318647-1-anusha.nandi@xxxxxxxxxxxxxxxx/
>>
>> This Depends-on is in the wrong series.
>> The driver should not depend on the DTS.
>
> Good point. If dependency is true, then this patchset is just broken.
>
> Best regards,
> Krzysztof