On Thu, 8 Jun 2023 11:57:40 +0100 Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> wrote: > 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. Applied to the togreg branch of iio.git. However note that these aren't going to make the coming merge window so will only be pushed out as testing until I can rebase on rc1. Thanks, Jonathan > > > 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 > > > > > > > >