Re: [PATCH 13/15] drm/mediatek: Add missing plane settings when async update

[Shawn Sung (宋孝謙)]

Hi Angelo,

On Thu, 2023-09-14 at 15:18 +0200, AngeloGioacchino Del Regno wrote:
> Il 11/09/23 15:45, Shawn Sung (宋孝謙) ha scritto:
> > Hi Angelo,
> > 
> > On Thu, 2023-09-07 at 14:33 +0200, AngeloGioacchino Del Regno
> > wrote:
> > > Il 23/08/23 17:13, Hsiao Chien Sung ha scritto:
> > > > Fix an issue that plane coordinate was not saved when
> > > > calling async update.
> > > > 
> > > > Signed-off-by: Hsiao Chien Sung <shawn.sung@xxxxxxxxxxxx>
> > > 
> > >   From what I understand, this is a fix for an issue that was
> > > present
> > > before
> > > your newly introduced code.
> > > 
> > > This means that you want to add a Fixes tag, and that you should
> > > also
> > > send
> > > this commit separately from the IGT series. Please do that.
> > 
> > Got it, will add a fix tag in the next version. But since without
> > this
> > patch, IGT will fail at some test cases, could we still send them
> > in
> > the same series so IGT can pass out-of-the-box?
> > 
> 
> Fixes are applied before new features, so if you send this patch
> outside
> of the IGT series, this will most likely be applied *before* that
> series.
> 
> When the IGT series will be applied, this patch will already be
> present,
> so, please send this one separately.

Got it. Had sent two fixes that are found during IGT test separately.

Thanks,

Hsiao Chien Sung