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: <8c1163ad-6e65-450b-ae44-c9a71a045333@kernel.org>
Date: Wed, 10 Sep 2025 09:42:40 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Vikram Sharma <quic_vikramsa@...cinc.com>,
 Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>, rfoss@...nel.org,
 todor.too@...il.com, bryan.odonoghue@...aro.org, 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 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!?!?

> 
> 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