linux-kernel - Re: [PATCH 0/2] Add vcp driver

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] [day] [month] [year] [list]

Message-ID: <a162ce6f-bb69-4fa3-a67b-5d38ffb9e2ef@kernel.org>
Date: Sun, 6 Apr 2025 14:13:08 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Xiangzhi Tang (唐相志)
 <Xiangzhi.Tang@...iatek.com>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
 "robh@...nel.org" <robh@...nel.org>,
 "matthias.bgg@...il.com" <matthias.bgg@...il.com>,
 "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
 "andersson@...nel.org" <andersson@...nel.org>,
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 "mathieu.poirier@...aro.org" <mathieu.poirier@...aro.org>
Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
 "linux-remoteproc@...r.kernel.org" <linux-remoteproc@...r.kernel.org>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 Hailong Fan (范海龙) <Hailong.Fan@...iatek.com>,
 Project_Global_Chrome_Upstream_Group
 <Project_Global_Chrome_Upstream_Group@...iatek.com>,
 "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>,
 Jjian Zhou (周建) <Jjian.Zhou@...iatek.com>
Subject: Re: [PATCH 0/2] Add vcp driver

On 06/04/2025 10:05, Xiangzhi Tang (唐相志) wrote:
> On Wed, 2025-04-02 at 12:31 +0200, Krzysztof Kozlowski wrote:
>> External email : Please do not click links or open attachments until
>> you have verified the sender or the content.
>>
>>
>> On 02/04/2025 11:19, Xiangzhi Tang wrote:
>>> Add support MediaTek's Video Companion Processor(VCP) host driver
>>> to
>>> control the MediaTek VCP Risc-V coprocessor.
>>> The VCP host driver using rproc mechanism to load vcm firmware
>>> from filesystem, and using SMC services to request ATF to setting
>>> vcp boot sequence, and Host communicated with VCP FW depends on VCP
>>> IPC interfaces
>>>
>>> This series patches dependent on:
>>> [1]
>>>
> https://lore.kernel.org/all/20250307032942.10447-1-guangjie.song@mediatek.com/
>>
>> They should not depend. It blocks your patch. I suggest decoupling
>> from
>> dependencies.
>> I run test "make dt_binding_check
> DT_SCHEMA_FILES=Documentation/devicetree/bindings/remoteproc/mediatek,m
> t8196-vcp.yaml", And, I have to cherry-pick first "
> 

How is this related? Did you read my message? You should not depend and
you answer "I added dependency". Great, so your patch won't get accepted
and won't get tested. Lack of testing will result in lack of review.


Best regards,
Krzysztof