mbox series

[v4,0/2] iio: imu: inv_icm42600: add support of accel low power

Message ID 20240605195949.766677-1-inv.git-commit@tdk.com (mailing list archive)
Headers show
Series iio: imu: inv_icm42600: add support of accel low power | expand

Message

inv.git-commit@tdk.com June 5, 2024, 7:59 p.m. UTC
From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>

Add accel low power support, enabling use of lower ODR below 12.5Hz.

Add new sysfs API for controlling the power mode and enforce power mode used
when using an ODR that supports only 1 mode (ODR < 12.5Hz or ODR > 500Hz).

Changelog

V2:
- fix patch missing Signed-off tag

V3:
- make all ODRs available and switch power mode automatically for ODR supporting
  only 1 mode.
 
V4:
- prevent power mode setting when the ODR is not supported
- return the actual power mode when sensor is on, otherwise the requested one.
- add sysfs ABI documentation

Jean-Baptiste Maneyrol (1):
  iio: imu: inv_icm42600: add support of accel low-power mode

 drivers/iio/imu/inv_icm42600/inv_icm42600.h   |   4 +
 .../iio/imu/inv_icm42600/inv_icm42600_accel.c | 124 +++++++++++++++++-
 .../iio/imu/inv_icm42600/inv_icm42600_core.c  |  27 ++++
 3 files changed, 151 insertions(+), 4 deletions(-)

Comments

Jonathan Cameron June 9, 2024, 10:09 a.m. UTC | #1
On Wed,  5 Jun 2024 19:59:47 +0000
inv.git-commit@tdk.com wrote:

> From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> 
> Add accel low power support, enabling use of lower ODR below 12.5Hz.
> 
> Add new sysfs API for controlling the power mode and enforce power mode used
> when using an ODR that supports only 1 mode (ODR < 12.5Hz or ODR > 500Hz).
Applied to the togreg branch of iio.git and pushed out as testing for 0-day to
poke at it.

Thanks,

Jonathan

> 
> Changelog
> 
> V2:
> - fix patch missing Signed-off tag
> 
> V3:
> - make all ODRs available and switch power mode automatically for ODR supporting
>   only 1 mode.
>  
> V4:
> - prevent power mode setting when the ODR is not supported
> - return the actual power mode when sensor is on, otherwise the requested one.
> - add sysfs ABI documentation
> 
> Jean-Baptiste Maneyrol (1):
>   iio: imu: inv_icm42600: add support of accel low-power mode
> 
>  drivers/iio/imu/inv_icm42600/inv_icm42600.h   |   4 +
>  .../iio/imu/inv_icm42600/inv_icm42600_accel.c | 124 +++++++++++++++++-
>  .../iio/imu/inv_icm42600/inv_icm42600_core.c  |  27 ++++
>  3 files changed, 151 insertions(+), 4 deletions(-)
>
Jean-Baptiste Maneyrol June 12, 2024, 12:28 p.m. UTC | #2
Hi Jonathan,

I'm sorry I am not able to find this patch in togreg branch currently.

Am I not looking at the correct place or there is a problem somewhere?

Thanks,
JB
Jonathan Cameron June 13, 2024, 6:22 p.m. UTC | #3
On Wed, 12 Jun 2024 12:28:50 +0000
Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com> wrote:

> Hi Jonathan,
> 
> I'm sorry I am not able to find this patch in togreg branch currently.
> 
> Am I not looking at the correct place or there is a problem somewhere?
As so often happens I was being dozy at end of a reviewing session
and failed to push out.
Done now.

> 
> Thanks,
> JB
> 
> ________________________________________
> From: Jonathan Cameron <jic23@kernel.org>
> Sent: Sunday, June 9, 2024 12:09
> To: INV Git Commit <INV.git-commit@tdk.com>
> Cc: lars@metafoo.de <lars@metafoo.de>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>
> Subject: Re: [PATCH v4 0/2] iio: imu: inv_icm42600: add support of accel low power
>  
> This Message Is From an External Sender
> This message came from outside your organization.
>  
> On Wed,  5 Jun 2024 19:59:47 +0000
> inv.git-commit@tdk.com wrote:
> 
> > From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> > 
> > Add accel low power support, enabling use of lower ODR below 12.5Hz.
> > 
> > Add new sysfs API for controlling the power mode and enforce power mode used
> > when using an ODR that supports only 1 mode (ODR < 12.5Hz or ODR > 500Hz).  
> Applied to the togreg branch of iio.git and pushed out as testing for 0-day to
> poke at it.
> 
> Thanks,
> 
> Jonathan
> 
> > 
> > Changelog
> > 
> > V2:
> > - fix patch missing Signed-off tag
> > 
> > V3:
> > - make all ODRs available and switch power mode automatically for ODR supporting
> >   only 1 mode.
> >  
> > V4:
> > - prevent power mode setting when the ODR is not supported
> > - return the actual power mode when sensor is on, otherwise the requested one.
> > - add sysfs ABI documentation
> > 
> > Jean-Baptiste Maneyrol (1):
> >   iio: imu: inv_icm42600: add support of accel low-power mode
> > 
> >  drivers/iio/imu/inv_icm42600/inv_icm42600.h   |   4 +
> >  .../iio/imu/inv_icm42600/inv_icm42600_accel.c | 124 +++++++++++++++++-
> >  .../iio/imu/inv_icm42600/inv_icm42600_core.c  |  27 ++++
> >  3 files changed, 151 insertions(+), 4 deletions(-)
> >   
>