On 18/09/2025 13:14, Yongxing Mou wrote: > > > On 9/18/2025 9:01 AM, Krzysztof Kozlowski wrote: >> On Thu, Sep 11, 2025 at 07:24:03PM +0800, Yongxing Mou wrote: >>> Document the MDSS hardware found on the Qualcomm QCS8300 platform. >>> >>> Signed-off-by: Yongxing Mou <yongxing....@oss.qualcomm.com> >> >> Patch v11 and still basic issues. I am very dissapointed. >> >> <form letter> >> This is a friendly reminder during the review process. >> >> It looks like you received a tag and forgot to add it. >> >> If you do not know the process, here is a short explanation: >> Please add Acked-by/Reviewed-by/Tested-by tags when posting new >> versions of patchset, under or above your Signed-off-by tag, unless >> patch changed significantly (e.g. new properties added to the DT >> bindings). Tag is "received", when provided in a message replied to you >> on the mailing list. Tools like b4 can help here. However, there's no >> need to repost patches *only* to add the tags. The upstream maintainer >> will do that for tags received on the version they apply. >> >> Please read: >> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 >> >> If a tag was not added on purpose, please state why and what changed. >> </form letter> >> >> Best regards, >> Krzysztof >> > Hi, > Sorry for the confusion. I did intend to remove the Reviewed-by tag, and > I mentioned this in the cover letter, but maybe explanation in > cover-letter was probe not clear at all. > > This patch includes three changes: > > 1.In the displayport-controller compatible property, "items" was changed > to "contains". > 2.Use "qcom,sa8775p-dp" as fallback. > > These changes might not be considered significant. So I’ll be more > careful next time. Thanks~
I really do not expect v12 to receive so significant changes in the first place. If you keep sending us buggy code, which then you keep changing after review, I will just not do the review. It's easier for me to wait for v20... Best regards, Krzysztof
