Re: [PATCH] blackfin/acvilon: Fix timeout usage for I2C

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

 



On Sun, Apr 04, 2010 at 04:04:16PM +0200, Wolfram Sang wrote:
> The timeout value is in jiffies, so it should be using HZ, not a plain
> number. As '10000' is ambiguous, 1HZ is used as conservative default.
> 
> Signed-off-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx>
> Cc: Mike Frysinger <vapier@xxxxxxxxxx>
> Cc: Valentin Yakovenkov <yakovenkov@xxxxxxxxx>
> ---
>  arch/blackfin/mach-bf561/boards/acvilon.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/blackfin/mach-bf561/boards/acvilon.c b/arch/blackfin/mach-bf561/boards/acvilon.c
> index 5163e2c..bfcfa86 100644
> --- a/arch/blackfin/mach-bf561/boards/acvilon.c
> +++ b/arch/blackfin/mach-bf561/boards/acvilon.c
> @@ -44,6 +44,7 @@
>  #include <linux/spi/flash.h>
>  #include <linux/irq.h>
>  #include <linux/interrupt.h>
> +#include <linux/jiffies.h>
>  #include <linux/i2c-pca-platform.h>
>  #include <linux/delay.h>
>  #include <linux/io.h>
> @@ -112,7 +113,7 @@ static struct resource bfin_i2c_pca_resources[] = {
>  struct i2c_pca9564_pf_platform_data pca9564_platform_data = {
>  	.gpio = -1,
>  	.i2c_clock_speed = 330000,
> -	.timeout = 10000
> +	.timeout = HZ,
>  };
>  
>  /* PCA9564 I2C Bus driver */

Ping.

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux