On Fri, 2022-08-26 at 09:29 -0400, Nícolas F. R. A. Prado wrote: > On Fri, Aug 26, 2022 at 02:12:00PM +0800, xinlei.lee wrote: > > On Thu, 2022-08-25 at 17:04 +0200, Matthias Brugger wrote: > > > > > > On 23/08/2022 22:17, Nícolas F. R. A. Prado wrote: > > > > On Tue, Aug 23, 2022 at 02:38:22PM +0800, > > > > xinlei.lee@xxxxxxxxxxxx > > > > wrote: > > > > > From: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx> > > > > > > > > > > Add mmsys function to manipulate dpi output format > > > > > configuration > > > > > for MT8186. > > > > > > > > > > Co-developed-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx> > > > > > Signed-off-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx> > > > > > Signed-off-by: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx> > > > > > > > > Reviewed-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx> > > > > > > > > > > Patch looks fine, I'll wait for v4 as there is still some > > > discussion > > > on the DRM > > > part. Please try to fix the threading problem you had in sending > > > this > > > series. > > > > > > Thanks, > > > Matthias > > > > Hi Matthias: > > > > Thanks for your review. > > > > Before I send the next version, do you mean the treading problem > > you > > are talking about is the problem I'm discussing in another patch in > > this series, or something else? > > Hi Xinlei, > > no, the threading problem Matthias is referring to is the fact that > this version > of your series had two cover letters (patch 0) and a patch 2 [1], > while patch 1 > was sent in a different email thread [2]. The whole series (cover > letter, > patches 1 and 2), should have been in the same email thread. Also, I > noticed > that your cover letter lists the patches with incorrect prefixes > "FROMLIST". > > So something weird happened when you sent this version of the series. > It was ok > in the previous version. So just make sure this is fixed when you > send the next > version of the series (after we finish the discussion on patch 2). > > [1] > https://lore.kernel.org/linux-mediatek/1661235517-23699-1-git-send-email-xinlei.lee@xxxxxxxxxxxx/ > [2] > https://lore.kernel.org/linux-mediatek/1661236702-30682-1-git-send-email-xinlei.lee@xxxxxxxxxxxx/ > > Thanks, > Nícolas Hi Nícolas: Thanks for the thoughtful explanation of this, I may know what caused the two patches not to be in a series. I will fix this and revise the "FROMLIST" tag in the cover letter in the next version. Best Regards! Xinlei