Re: [PATCH v1 03/13] iio: chemical: bme680: fix startup time

Jonathan Cameron
2024-10-12T11:45:17Z

On Fri, 11 Oct 2024 20:51:41 +0200
Vasileios Aoiridis <vassilisamir@gmail.com> wrote:

> On Fri, Oct 11, 2024 at 01:00:55PM +0300, Andy Shevchenko wrote:
> > On Thu, Oct 10, 2024 at 11:00:20PM +0200, vamoirid wrote:
> > > From: Vasileios Amoiridis <vassilisamir@gmail.com>
> > >
> > > According to datasheet's Section 1.1, Table 1, the startup time for the
> > > device is 2ms and not 5ms.
> >
> > Fixes tag?
> >
> > --
> > With Best Regards,
> > Andy Shevchenko
> >
> >
>
> Hi Andy,
>
> It is not affecting at all the operation of the driver so I was not sure
> if it was worth it to be backported to the previous versions. This is
> why I didn't put a fixes tag. You think for such a fix is necessary?
>
Not for a reduction in sleep and not a huge one at that.
It's an optimization, not a fix to my eyes!

> Cheers,
> Vasilis