linux-kernel - Re: [PATCH 0/4] Add MST support for qcs8300 platform

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: <1b22b005-4ad3-4ba9-9f6c-b540ed45dfc7@quicinc.com>
Date: Thu, 20 Feb 2025 15:33:04 +0800
From: Yongxing Mou <quic_yongmou@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, Rob Clark <robdclark@...il.com>,
        Abhinav Kumar <quic_abhinavk@...cinc.com>,
        Dmitry Baryshkov
	<dmitry.baryshkov@...aro.org>,
        Sean Paul <sean@...rly.run>,
        Marijn Suijten
	<marijn.suijten@...ainline.org>,
        David Airlie <airlied@...il.com>, "Simona
 Vetter" <simona@...ll.ch>,
        Maarten Lankhorst
	<maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Zimmermann <tzimmermann@...e.de>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Kuogee Hsieh <quic_khsieh@...cinc.com>,
        Bjorn Andersson
	<andersson@...nel.org>,
        Konrad Dybcio <konradybcio@...nel.org>
CC: <linux-arm-msm@...r.kernel.org>, <dri-devel@...ts.freedesktop.org>,
        <freedreno@...ts.freedesktop.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/4] Add MST support for qcs8300 platform



On 2025/2/19 20:07, Krzysztof Kozlowski wrote:
> On 19/02/2025 11:02, Yongxing Mou wrote:
>>
>>
>> On 2025/2/12 17:06, Krzysztof Kozlowski wrote:
>>> On 12/02/2025 08:12, Yongxing Mou wrote:
>>>> This series of patches introduces how to enable MST functionality on
>>>> the qcs8300 platform. The qcs8300 platform uses dpu_8_4 hardware, which
>>>> is the same as the sa8775p, but it only has one DPU. So it only has one
>>>> DP0 controller, supporting 4-stream MST. This patch only enables
>>>> 2-stream MST, using intf0 and intf3. The first and second patches are
>>>> modifications to the correspond dt-bindings, third patch is the dp
>>>> controller driver after not reuse sm8650, fourth patch is the qcs8300
>>>> dts modification which add the clk support for stream 1.
>>>>
>>>> Signed-off-by: Yongxing Mou <quic_yongmou@...cinc.com>
>>>> ---
>>>> This patch depends on following series:
>>>> https://lore.kernel.org/all/20250120-mdssdt_qcs8300-v4-0-1687e7842125@quicinc.com/
>>>> https://lore.kernel.org/all/20250114-dts_qcs8300-v3-0-d114cc5e4af9@quicinc.com/
>>>> https://lore.kernel.org/all/20241202-dp_mst_bindings-v1-0-9a9a43b0624a@quicinc.com
>>>> https://lore.kernel.org/all/20241205-dp_mst-v1-0-f8618d42a99a@quicinc.com/
>>>
>>> Buggy patch cannot be the dependency. You need to fix the original patch
>>> instead. You are not supposed to wait till knowingly incorrect patches
>>> get merged so you can send a fix. You must respond to that patch
>>> stopping its merging process.
>>>
>>> Best regards,
>>> Krzysztof
>> Thanks for the reminder. I hadn't noticed that before. We will wait for
>> our dependencies to be ready before sending the new patchset.
> 
> So this means you intend the knowingly incorrect patch to be applied? If
> so, all this should be just NAKed.
> 
> Best regards,
> Krzysztof
Hi, that is not my intention. I will correct the patch after i know it 
is incorrect. Thanks.