Subject: Re: [PATCH v12 5/6] rtc: mt6397: Add support for the MediaTek MT6358 RTC

From: Lee Jones <[email-masked]>
Date: Mon, 20 Apr 2020 08:36:39 +0100

On Fri, 17 Apr 2020, Ran Bi wrote:

> On Thu, 2020-04-16 at 10:14 +0100, Lee Jones wrote:
> > On Wed, 08 Apr 2020, Hsin-Hsiung Wang wrote:
> > 
> > > From: Ran Bi <[email-masked]>
> > > 
> > > This add support for the MediaTek MT6358 RTC. Driver using
> > > compatible data to store different RTC_WRTGR address offset.
> > > This replace RTC_WRTGR to RTC_WRTGR_MT6323 in mt6323-poweroff
> > > driver which only needed by armv7 CPU without ATF.
> > > 
> > > Reviewed-by: Nicolas Boichat <[email-masked]>
> > > Reviewed-by: Yingjoe Chen <[email-masked]>
> > > Acked-by: Alexandre Belloni <[email-masked]>
> > > Acked-by: Sebastian Reichel <[email-masked]>
> > > Signed-off-by: Ran Bi <[email-masked]>
> > > Signed-off-by: Hsin-Hsiung Wang <[email-masked]>
> > 
> > Please place these in chronological order.  They should provide some
> > history, rather than a unordered slab list of random sign-offs.
> > 
> 
> I suppose that you mean the order should be like below, right?
> Reviewed-by: Yingjoe Chen <[email-masked]>
> Acked-by: Sebastian Reichel <[email-masked]>
> Acked-by: Alexandre Belloni <[email-masked]>
> Reviewed-by: Nicolas Boichat <[email-masked]>
> Signed-off-by: Hsin-Hsiung Wang <[email-masked]>
> Signed-off-by: Ran Bi <[email-masked]>

This would imply that it was reviewed before it was written, which
would subsequently imply time-travel, so I suggest not.

Author(s)
Review(s)/Acks(s)/Tested(s) /* ideally in the order they were received */
Sub-maintainer sign-off     /* if applicable */
Maintainer sign-off

> > > ---
> > >  drivers/power/reset/mt6323-poweroff.c |  2 +-
> > >  drivers/rtc/rtc-mt6397.c              | 18 +++++++++++++++---
> > >  include/linux/mfd/mt6397/rtc.h        |  9 ++++++++-
> > >  3 files changed, 24 insertions(+), 5 deletions(-)

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel