Re: [PATCH v3 3/8] iio: imu: inv_icm42607: Update _odr_to_period_us formatting

[Jonathan Cameron]

> Update the function formatting for the inv_icm42607_odr_to_period_us
> function. This enhances the readability of the function and drop the
> unnecessary comments in the array elements.
The description doesn't mention the change I don't think is
beneficial.  Which makes it even easier to drop that change.

Thanks,

Jonathan

> 
> Signed-off-by: Kanak Shilledar <kanak.shilledar@xxxxxxxx>
>
> diff --git a/drivers/iio/imu/inv_icm42607/inv_icm42607.h b/drivers/iio/imu/inv_icm42607/inv_icm42607.h
> index 4d51b0da1aa1..fa85cf738cc0 100644
> --- a/drivers/iio/imu/inv_icm42607/inv_icm42607.h
> +++ b/drivers/iio/imu/inv_icm42607/inv_icm42607.h
> @@ -62,16 +62,16 @@ enum inv_icm42607_accel_fs {
>  /* ODR values  - Note Gyro does not support ODR less than 12.5Hz */
>  enum inv_icm42607_odr {
>  	INV_ICM42607_ODR_1600HZ = 5,
> -	INV_ICM42607_ODR_800HZ = 6,
> -	INV_ICM42607_ODR_400HZ = 7,
> -	INV_ICM42607_ODR_200HZ = 8,
> -	INV_ICM42607_ODR_100HZ = 9,
> -	INV_ICM42607_ODR_50HZ = 10,
> -	INV_ICM42607_ODR_25HZ = 11,
> -	INV_ICM42607_ODR_12_5HZ = 12,
> -	INV_ICM42607_ODR_6_25HZ_LP = 13,
> -	INV_ICM42607_ODR_3_125HZ_LP = 14,
> -	INV_ICM42607_ODR_1_5625HZ_LP = 15,

Why?  We often put all the values when they are mapping to hardware
register values simply because it makes them easier to compare to
a datasheet.

-- 
Jonathan Cameron <jonathan.cameron@xxxxxxxxxxxxxxxx>