Re: [PATCH v4 03/19] dt-bindings: display: mediatek: add EXDMA yaml for MT8196

["Paul-pl Chen (陳柏霖)" <Paul-pl.Chen@mediatek.com>]

On Fri, 2025-08-29 at 08:48 +0200, Krzysztof Kozlowski wrote:
> 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On Fri, Aug 29, 2025 at 08:35:23AM +0200, Krzysztof Kozlowski wrote:
> > On Thu, Aug 28, 2025 at 04:06:58PM +0800, Paul Chen wrote:
> > > From: Paul-pl Chen <paul-pl.chen@mediatek.com>
> > > 
> > > Add mediatek,exdma.yaml to support EXDMA for MT8196.
> > > The MediaTek display overlap extended DMA engine, namely
> > > OVL_EXDMA or EXDMA, primarily functions as a DMA engine
> > > for reading data from DRAM with various DRAM footprints
> > > and data formats.
> > > 
> > > Signed-off-by: Paul-pl Chen <paul-pl.chen@mediatek.com>
> > > ---
> > >  .../bindings/dma/mediatek,exdma.yaml          | 68
> > > +++++++++++++++++++
> > 
> > 
> > Your changelog says NOTHING changed here and this fails tests, so
> > does it
> > mean you received that warnings before but you keep sending same
> > broken
> > code?
> > 
> > Last two weeks of contributions from mediatek are absolutely
> > terrible.
> > Very poor code, basic in-house reviews not done, basic testing not
> > done.
> > 
> > I talked about this at OSSE 25 with some friends and got reasons
> > why
> > your setup is broken. Well, it's on you.
> > 
> > I was already raising this with Mediatek, but we are now back to
> > square
> > one.
> > 
> > NAK, because this patch WAS NEVER tested.
> 
> 
> And now I found you got EXACTLY the same error at v3, so you just
> never
> tested and ignored OUR test reports.
> 
> This is unfortunately an example how you waste reviewers' time.
> 
> Best regards,
> Krzysztof
> 

Hi Krzysztof,
 
I apologize for resubmitting an insufficiently tested patch and wasting
reviewers’ time. The repeated failure and the inadequate changelog are
entirely my responsibility—no excuses. I’m withdrawing this revision
and will only resend after I’ve reproduced and fixed the failures,
validated with comprehensive tests, completed internal review, and
included a clear “Changes since vN.”
 
Thank you for the direct feedback—I will do better.
 
Best regards, Paul