On Wed, 7 Jun 2023 17:44:39 +0300
Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote:
> On Wed, Jun 7, 2023 at 12:28 PM Jean-Baptiste Maneyrol
> <Jean-Baptiste.Maneyrol@xxxxxxx> wrote:
> >
> > Hello Andy,
> >
> > really sorry, I forgot to do that. I'm still not very familiar with all the details of the process, sorry.
>
> Understand.
>
> > I will send a new v3 with the Reviewed-by tag. Is it OK like this?
>
> I believe no need to resend right now as I can simply repeat it here
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
That works for me ;)
Thanks Andy.
> But next time be more careful.
>
> And thank you for what you are doing!
>
> > Thanks a lot, and sorry again.
>
> > From: andy.shevchenko@xxxxxxxxx <andy.shevchenko@xxxxxxxxx>
> > Sent: Wednesday, June 7, 2023 08:26
> > To: INV Git Commit <INV.git-commit@xxxxxxx>
> > Cc: jic23@xxxxxxxxxx <jic23@xxxxxxxxxx>; linux-iio@xxxxxxxxxxxxxxx <linux-iio@xxxxxxxxxxxxxxx>; lars@xxxxxxxxxx <lars@xxxxxxxxxx>; Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@xxxxxxx>
> > Subject: Re: [PATCH v2 0/4] Factorize timestamp module
> >
> > [CAUTION] This is an EXTERNAL email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
> >
> > ======================================================================
> > Tue, Jun 06, 2023 at 04:21:43PM +0000, inv.git-commit@xxxxxxx kirjoitti:
> > > From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@xxxxxxx>
> > >
> > > The purpose if this series is to make timestamping from
> > > inv_icm42600 driver an independent module and use it for both
> > > inv_icm42600 and inv_mpu6050 drivers.
> > >
> > > Create a new inv_sensors_timestamp common module based on
> > > inv_icm42600 driver and use it in the 2 drivers.
> > >
> > > WARNING: this patch requires following commit in fixes-togreg
> > > bbaae0c79ebd ("iio: imu: inv_icm42600: fix timestamp reset")
> >
> > > Changelog
> > > - v2: do some headers cleanup and add some justifications in
> > > the patches descriptions.
> >
> > What I haven't noticed is my tag. It's your responsibility to add given tag and
> > it's polite to Cc to the reviewers (but this will imply by the tag anyway in
> > this case).
> >
> > Any explanation why did you do so?
> >
> > --
> > With Best Regards,
> > Andy Shevchenko
> >
>
>
