On Wed, Mar 23, 2022 at 2:41 AM qinjian[覃健] <qinjian@xxxxxxxxxxx> wrote: > > On Tue, Mar 22, 2022 at 7:17 AM Qin Jian <qinjian@xxxxxxxxxxx> wrote: > > > > It was a while ago though, so I don't remember the details, if we did > > conclude the review, can you link to the lore.kernel.org thread in > > the next version? Please also check the other patches on which > > you added a 'Reviewed-by' tag from someone to ensure that they > > actually provided that tag, rather than just providing review comments. > > In patch 6, I fixed all your comments. > > I got a reply from you: https://lore.kernel.org/all/CAK8P3a1Doh8GY9iFZsvmd6wASHoPqyR+roXx0G5XidnmHNkGaA@xxxxxxxxxxxxxx/ > > This looks all good to me now, it just needs a review for the clk, > > reset and irqchip drivers. ... > > If these not means you reviewed, I'll remove your review-tag. You can only add Reviewed-by tags that have been explicitly given, see the documentation at [1] for details. What I wrote was a less formal acknowledgement that does not have the specific meaning. Arnd [1] https://www.kernel.org/doc/html/latest/translations/zh_TW/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes
