Re: [PATCH v2 0/4] Factorize timestamp module

From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>
Cc: "andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>,
	INV Git Commit <INV.git-commit@tdk.com>,
	"jic23@kernel.org" <jic23@kernel.org>,
	"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
	"lars@metafoo.de" <lars@metafoo.de>
Subject: Re: [PATCH v2 0/4] Factorize timestamp module
Date: Wed, 7 Jun 2023 15:58:56 +0100	[thread overview]
Message-ID: <20230607155856.00006db3@Huawei.com> (raw)
In-Reply-To: <FR3P281MB17572FE25C24840A4030315ACE53A@FR3P281MB1757.DEUP281.PROD.OUTLOOK.COM>

On Wed, 7 Jun 2023 09:28:12 +0000
Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com> wrote:

> Hello Andy,
> 
> really sorry, I forgot to do that. I'm still not very familiar with all the details of the process, sorry.
> 
> I will send a new v3 with the Reviewed-by tag. Is it OK like this?

Go for it.   Other option is to reply with a whoops: forgot this tag and
include the tag.  That way b4 will pick it up for me when I apply.

Jonathan

> 
> Thanks a lot, and sorry again.
> JB
> 
> 
> From: andy.shevchenko@gmail.com <andy.shevchenko@gmail.com>
> Sent: Wednesday, June 7, 2023 08:26
> To: INV Git Commit <INV.git-commit@tdk.com>
> Cc: jic23@kernel.org <jic23@kernel.org>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; lars@metafoo.de <lars@metafoo.de>; Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>
> 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@tdk.com kirjoitti:
> > From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> > 
> > 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?
> 