mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	i2c: designware: Call i2c_dw_clk_rate() only once in i2c_dw_init_master()
This is rather readability update than micro-optimization, or if not optimization at all. We take the input clock rate to a variable and pass that to SCL timing parameter calculation functions. Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
This commit is contained in:
		
							parent
							
								
									83b2cb48cb
								
							
						
					
					
						commit
						3bd4f27727
					
				
					 1 changed files with 5 additions and 4 deletions
				
			
		| 
						 | 
				
			
			@ -55,6 +55,7 @@ static void i2c_dw_configure_fifo_master(struct dw_i2c_dev *dev)
 | 
			
		|||
 */
 | 
			
		||||
static int i2c_dw_init_master(struct dw_i2c_dev *dev)
 | 
			
		||||
{
 | 
			
		||||
	u32 ic_clk = i2c_dw_clk_rate(dev);
 | 
			
		||||
	u32 hcnt, lcnt;
 | 
			
		||||
	u32 reg, comp_param1;
 | 
			
		||||
	u32 sda_falling_time, scl_falling_time;
 | 
			
		||||
| 
						 | 
				
			
			@ -79,12 +80,12 @@ static int i2c_dw_init_master(struct dw_i2c_dev *dev)
 | 
			
		|||
		hcnt = dev->ss_hcnt;
 | 
			
		||||
		lcnt = dev->ss_lcnt;
 | 
			
		||||
	} else {
 | 
			
		||||
		hcnt = i2c_dw_scl_hcnt(i2c_dw_clk_rate(dev),
 | 
			
		||||
		hcnt = i2c_dw_scl_hcnt(ic_clk,
 | 
			
		||||
					4000,	/* tHD;STA = tHIGH = 4.0 us */
 | 
			
		||||
					sda_falling_time,
 | 
			
		||||
					0,	/* 0: DW default, 1: Ideal */
 | 
			
		||||
					0);	/* No offset */
 | 
			
		||||
		lcnt = i2c_dw_scl_lcnt(i2c_dw_clk_rate(dev),
 | 
			
		||||
		lcnt = i2c_dw_scl_lcnt(ic_clk,
 | 
			
		||||
					4700,	/* tLOW = 4.7 us */
 | 
			
		||||
					scl_falling_time,
 | 
			
		||||
					0);	/* No offset */
 | 
			
		||||
| 
						 | 
				
			
			@ -101,12 +102,12 @@ static int i2c_dw_init_master(struct dw_i2c_dev *dev)
 | 
			
		|||
		hcnt = dev->fs_hcnt;
 | 
			
		||||
		lcnt = dev->fs_lcnt;
 | 
			
		||||
	} else {
 | 
			
		||||
		hcnt = i2c_dw_scl_hcnt(i2c_dw_clk_rate(dev),
 | 
			
		||||
		hcnt = i2c_dw_scl_hcnt(ic_clk,
 | 
			
		||||
					600,	/* tHD;STA = tHIGH = 0.6 us */
 | 
			
		||||
					sda_falling_time,
 | 
			
		||||
					0,	/* 0: DW default, 1: Ideal */
 | 
			
		||||
					0);	/* No offset */
 | 
			
		||||
		lcnt = i2c_dw_scl_lcnt(i2c_dw_clk_rate(dev),
 | 
			
		||||
		lcnt = i2c_dw_scl_lcnt(ic_clk,
 | 
			
		||||
					1300,	/* tLOW = 1.3 us */
 | 
			
		||||
					scl_falling_time,
 | 
			
		||||
					0);	/* No offset */
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue