Messages in this thread | | | From | Shawn Sung (宋孝謙) <> | Subject | Re: [PATCH 13/15] drm/mediatek: Add missing plane settings when async update | Date | Mon, 18 Sep 2023 08:58:29 +0000 |
| |
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@mediatek.com> > > > > > > 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
| |