linux-kernel - Re: [PATCH v3 1/7] media: dt-bindings: Add qcom,qcs8300-camss compatible

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0519a8ce-9edf-4510-ada2-f8ac997142cf@quicinc.com>
Date: Wed, 10 Sep 2025 14:21:52 +0530
From: Vikram Sharma <quic_vikramsa@...cinc.com>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
        Krzysztof Kozlowski
	<krzk@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        <rfoss@...nel.org>, <todor.too@...il.com>, <mchehab@...nel.org>,
        <robh@...nel.org>, <krzk+dt@...nel.org>, <conor+dt@...nel.org>,
        <andersson@...nel.org>, <konradybcio@...nel.org>,
        <hverkuil-cisco@...all.nl>, <cros-qcom-dts-watchers@...omium.org>,
        <catalin.marinas@....com>, <will@...nel.org>
CC: <linux-arm-kernel@...ts.infradead.org>, <linux-media@...r.kernel.org>,
        <linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 1/7] media: dt-bindings: Add qcom,qcs8300-camss
 compatible


On 9/10/2025 2:12 PM, Bryan O'Donoghue wrote:
> On 10/09/2025 08:42, Krzysztof Kozlowski wrote:
>> On 10/09/2025 09:26, Vikram Sharma wrote:
>>>
>>> On 9/10/2025 12:35 PM, Krzysztof Kozlowski wrote:
>>>> On 13/08/2025 07:37, Vikram Sharma wrote:
>>>>> Add the compatible string "qcom,qcs8300-camss" to support the
>>>>> Camera Subsystem (CAMSS) on the Qualcomm QCS8300 platform.
>>>>>
>>>>> The QCS8300 platform provides:
>>>>> - 2 x VFE (version 690), each with 3 RDI
>>>>> - 5 x VFE Lite (version 690), each with 6 RDI
>>>>> - 2 x CSID (version 690)
>>>>> - 5 x CSID Lite (version 690)
>>>>> - 3 x CSIPHY (version 690)
>>>>> - 3 x TPG
>>>>>
>>>>> Signed-off-by: Vikram Sharma <quic_vikramsa@...cinc.com>
>>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>>>>> ---
>>>>>    .../bindings/media/qcom,qcs8300-camss.yaml    | 336 
>>>>> ++++++++++++++++++
>>>>>    1 file changed, 336 insertions(+)
>>>> So it turns out this code is wrong and you already sent fixes for it.
>>>
>>> Hi Krzysztof,
>>> Thanks for your comments. These bindings are tested with TPG (Test 
>>> Pattern Generator), As camera sensor testing was not possible at 
>>> that point.
>>
>> You cannot test bindings with TPG. It's impossible.
>>
>>> This is because Monaco-evk kit was not ready which can be used to 
>>> enable sensor.
>>>
>>> These Bindings are complete in its own, If someone want to use TPG.
>>
>> No. You do not understand - they are not complete if you add new
>> properties to them!
>>
>> How bindings could be complete if you add now supplies!?!?
Got it, Will post an updated version of this adding supplies and 
removing your RB.
>>
>>>
>>> My latest update in these bindings are for camera sensor enablement 
>>> which needs supplies too.
>>>
>>> Please let me know if this justification works to accept these 
>>> bindings it its current form and add supplies as a separate patch.
>>
>> NAK. Read writing bindings doc. Or internal docs.
>>
>> Best regards,
>> Krzysztof
>
> @Vikram
>
> Could you send an updated dt-bindings with the supplies included ASAP 
> as a v4 of this series ?
on it, Will Remove your RB and send v3.1 for this patch
>
> The -supplies in your email here look fine
>
> 20250909114241.840842-5-quic_vikramsa@...cinc.com
>
> Otherwise we will have to drop this binding and the associated driver 
> changes.
>
> ---
> bod

Best regards,

Vikram