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

[Krzysztof Kozlowski <krzk@kernel.org>]

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