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

[Yongxing Mou <quic_yongmou@quicinc.com>]

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@quicinc.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.