[PATCH v3] i2c: aspeed: fix invalid clock parameters for very large divisors
    Jae Hyun Yoo 
    jae.hyun.yoo at linux.intel.com
       
    Sat Sep 22 08:46:42 AEST 2018
    
    
  
On 9/21/2018 3:34 PM, Brendan Higgins wrote:
> @@ -142,7 +142,8 @@ struct aspeed_i2c_bus {
>   	/* Synchronizes I/O mem access to base. */
>   	spinlock_t			lock;
>   	struct completion		cmd_complete;
> -	u32				(*get_clk_reg_val)(u32 divisor);
> +	u32				(*get_clk_reg_val)(struct device *dev,
> +							   u32 divisor);
I realized that you changed *get_clk_reg_val type in v2. You probably
need to fix below code in aspeed_i2c_probe_bus() as well to remove
a sparse warning.
bus->get_clk_reg_val = (u32 (*)(u32))match->data;
-Jae
    
    
More information about the openbmc
mailing list