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

[Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1488 bytes --]

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-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
> Cc: Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
> Cc: Valentin Yakovenkov <yakovenkov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  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 #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]