Re: [PATCH] iio: adis162xx: fix low-power docs & reports

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 25 Jun 2019 15:48:40 +0300
Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx> wrote:

> All current ADIS162XX drivers have incorrect values defined via comments.
> Also, when an error is reported the printed value is incorrect.
> 
> The functionality itself isn't affected, so it's not a critical issue.
> 
> And since the change is trivial, it was included in a single patch that
> fixes these in one go. All values were correlated with the ones specified
> in the data-sheets.
> 
> Signed-off-by: Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx>
Applied to the togreg branch of iio.git and pushed out as testing
for usual reasons.

Thanks,

Jonathan

> ---
>  drivers/iio/accel/adis16201.c         | 4 ++--
>  drivers/iio/accel/adis16209.c         | 4 ++--
>  drivers/staging/iio/accel/adis16203.c | 4 ++--
>  drivers/staging/iio/accel/adis16240.c | 2 +-
>  4 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/iio/accel/adis16201.c b/drivers/iio/accel/adis16201.c
> index 4c1d482ea73a..3257d4d27015 100644
> --- a/drivers/iio/accel/adis16201.c
> +++ b/drivers/iio/accel/adis16201.c
> @@ -71,7 +71,7 @@
>  #define  ADIS16201_DIAG_STAT_FLASH_UPT_FAIL_BIT		2
>  /* Power supply above 3.625 V */
>  #define  ADIS16201_DIAG_STAT_POWER_HIGH_BIT		1
> -/* Power supply below 3.15 V */
> +/* Power supply below 2.975 V */
>  #define  ADIS16201_DIAG_STAT_POWER_LOW_BIT		0
>  
>  /* System Command Register Definition */
> @@ -231,7 +231,7 @@ static const char * const adis16201_status_error_msgs[] = {
>  	[ADIS16201_DIAG_STAT_SPI_FAIL_BIT] = "SPI failure",
>  	[ADIS16201_DIAG_STAT_FLASH_UPT_FAIL_BIT] = "Flash update failed",
>  	[ADIS16201_DIAG_STAT_POWER_HIGH_BIT] = "Power supply above 3.625V",
> -	[ADIS16201_DIAG_STAT_POWER_LOW_BIT] = "Power supply below 3.15V",
> +	[ADIS16201_DIAG_STAT_POWER_LOW_BIT] = "Power supply below 2.975V",
>  };
>  
>  static const struct adis_data adis16201_data = {
> diff --git a/drivers/iio/accel/adis16209.c b/drivers/iio/accel/adis16209.c
> index f2dc3a5f0463..01dd02f7e1d6 100644
> --- a/drivers/iio/accel/adis16209.c
> +++ b/drivers/iio/accel/adis16209.c
> @@ -73,7 +73,7 @@
>  #define  ADIS16209_STAT_FLASH_UPT_FAIL_BIT	2
>  /* Power supply above 3.625 V */
>  #define  ADIS16209_STAT_POWER_HIGH_BIT		1
> -/* Power supply below 3.15 V */
> +/* Power supply below 2.975 V */
>  #define  ADIS16209_STAT_POWER_LOW_BIT		0
>  
>  #define ADIS16209_CMD_REG			0x3E
> @@ -241,7 +241,7 @@ static const char * const adis16209_status_error_msgs[] = {
>  	[ADIS16209_STAT_SPI_FAIL_BIT] = "SPI failure",
>  	[ADIS16209_STAT_FLASH_UPT_FAIL_BIT] = "Flash update failed",
>  	[ADIS16209_STAT_POWER_HIGH_BIT] = "Power supply above 3.625V",
> -	[ADIS16209_STAT_POWER_LOW_BIT] = "Power supply below 3.15V",
> +	[ADIS16209_STAT_POWER_LOW_BIT] = "Power supply below 2.975V",
>  };
>  
>  static const struct adis_data adis16209_data = {
> diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c
> index a5d974ac2e3b..39687139a7d3 100644
> --- a/drivers/staging/iio/accel/adis16203.c
> +++ b/drivers/staging/iio/accel/adis16203.c
> @@ -122,7 +122,7 @@
>  /* Power supply above 3.625 V */
>  #define ADIS16203_DIAG_STAT_POWER_HIGH_BIT    1
>  
> -/* Power supply below 3.15 V */
> +/* Power supply below 2.975 V */
>  #define ADIS16203_DIAG_STAT_POWER_LOW_BIT     0
>  
>  /* GLOB_CMD */
> @@ -234,7 +234,7 @@ static const char * const adis16203_status_error_msgs[] = {
>  	[ADIS16203_DIAG_STAT_SPI_FAIL_BIT] = "SPI failure",
>  	[ADIS16203_DIAG_STAT_FLASH_UPT_BIT] = "Flash update failed",
>  	[ADIS16203_DIAG_STAT_POWER_HIGH_BIT] = "Power supply above 3.625V",
> -	[ADIS16203_DIAG_STAT_POWER_LOW_BIT] = "Power supply below 3.15V",
> +	[ADIS16203_DIAG_STAT_POWER_LOW_BIT] = "Power supply below 2.975V",
>  };
>  
>  static const struct adis_data adis16203_data = {
> diff --git a/drivers/staging/iio/accel/adis16240.c b/drivers/staging/iio/accel/adis16240.c
> index b80c8529784b..62f4b3b1b457 100644
> --- a/drivers/staging/iio/accel/adis16240.c
> +++ b/drivers/staging/iio/accel/adis16240.c
> @@ -175,7 +175,7 @@
>  /* Power supply above 3.625 V */
>  #define ADIS16240_DIAG_STAT_POWER_HIGH_BIT	1
>  
> - /* Power supply below 3.15 V */
> + /* Power supply below 2.225 V */
>  #define ADIS16240_DIAG_STAT_POWER_LOW_BIT	0
>  
>  /* GLOB_CMD */




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux