[U-Boot-Users] [PATCH] fsl_i2c: Fix used uninitialized variable

[Kumar Gala]

On Apr 1, 2008, at 3:34 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> fsl_i2c.c: In function 'set_i2c_bus_speed':
> fsl_i2c.c:120: warning: 'dfsr' may be used uninitialized in this  
> function
> fsl_i2c.c:120: warning: 'fdr' may be used uninitialized in this  
> function
>
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD  
> <plagnioj at jcrosoft.com>
>
> diff --git a/drivers/i2c/fsl_i2c.c b/drivers/i2c/fsl_i2c.c
> index dde0571..917a4ba 100644
> --- a/drivers/i2c/fsl_i2c.c
> +++ b/drivers/i2c/fsl_i2c.c
> @@ -117,7 +117,8 @@ static unsigned int set_i2c_bus_speed(const  
> struct fsl_i2c *dev,
> {
> 	unsigned short divider = min(i2c_clk / speed, (unsigned short) -1);
> 	unsigned int i;
> -	u8 fdr, dfsr;
> +	u8 fdr = 0x3F;
> +	u8 dfsr = 0x3F;
>
> 	/*
> 	 * We want to choose an FDR/DFSR that generates an I2C bus speed that
> -- 
> 1.5.4.5

I posted a patch that moved fdr, dfsr inside the loop to fix the  
warnings.

- k