linux-kernel - Re: [PATCH v2 2/8] dt-bindings: media: nxp: Add Wave6 video codec device

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: <cafba18a-5391-4d9d-aa4c-2f06f93af0f8@kernel.org>
Date: Fri, 16 May 2025 14:55:54 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Nas Chung <nas.chung@...psnmedia.com>
Cc: "mchehab@...nel.org" <mchehab@...nel.org>,
 "hverkuil@...all.nl" <hverkuil@...all.nl>,
 "sebastian.fricke@...labora.com" <sebastian.fricke@...labora.com>,
 "robh@...nel.org" <robh@...nel.org>, "krzk+dt@...nel.org"
 <krzk+dt@...nel.org>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
 "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-imx@....com" <linux-imx@....com>, "marex@...x.de" <marex@...x.de>,
 "jackson.lee" <jackson.lee@...psnmedia.com>,
 "lafley.kim" <lafley.kim@...psnmedia.com>
Subject: Re: [PATCH v2 2/8] dt-bindings: media: nxp: Add Wave6 video codec
 device

On 13/05/2025 09:39, Nas Chung wrote:
>>
>> All of above are wrong for the SoC...
>>
>>>
>>>         #include <dt-bindings/interrupt-controller/arm-gic.h>
>>>         #include <dt-bindings/clock/nxp,imx95-clock.h>
>>>
>>>         soc {
>>>           #address-cells = <2>;
>>>           #size-cells = <2>;
>>>
>>>           vpu: video-codec {
>>>             compatible = "nxp,imx95-vpu", "cnm,wave633c";
>>
>> What does this device represent? It is not "ctrl", because you made ctrl
>> separate device node. Your binding description suggests that is the VPU
>> control region.
> 
> My intention was to represent the MMIO VPU device, which includes
> both the core and control nodes.

Then what is the VPU device if not CTRL? What is the CTRL device?


Best regards,
Krzysztof