mbox series

[i2c-next,0/4] i2c: mlxcpld: Add fixes and features for transaction polling time setting

Message ID 20210823144504.1249203-1-vadimp@nvidia.com
Headers show
Series i2c: mlxcpld: Add fixes and features for transaction polling time setting | expand

Message

Vadim Pasternak Aug. 23, 2021, 2:45 p.m. UTC 
This patch set contains:
Patches #1-#2: Fixes for frequency setting.
Patch #3: Reducing driver's transaction polling time.
Patch #4: Flexible setting of transaction polling time according to I2C
	  bus frequency

Vadim Pasternak (4):
  i2c: mlxcpld: Fix criteria for frequency setting
  i2c: mlxcpld: Modify register setting for 400KHz frequency
  i2c: mlxcpld: Reduce polling time for performance improvement
  i2c: mlxcpld: Allow flexible polling time setting for I2C transactions

 drivers/i2c/busses/i2c-mlxcpld.c | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

Comments

Wolfram Sang Oct. 4, 2021, 7:58 p.m. UTC | #1 
On Mon, Aug 23, 2021 at 05:45:01PM +0300, Vadim Pasternak wrote:
> Value for getting frequency capability wrongly has been taken from

> register offset instead of register value.

> 

> Fixes: 66b0c2846ba8 ("i2c: mlxcpld: Add support for I2C bus frequency setting")

> Signed-off-by: Vadim Pasternak <vadimp@nvidia.com>


Applied to for-current, thanks!
Wolfram Sang Oct. 4, 2021, 7:59 p.m. UTC | #2 
On Mon, Aug 23, 2021 at 05:45:03PM +0300, Vadim Pasternak wrote:
> Decrease polling time 'MLXCPLD_I2C_POLL_TIME' from 400 usec to 200

> usec. It improves performance of I2C transactions.

> 

> Reliability of setting polling time to 200 usec has been validated

> across all the supported systems.

> 

> Signed-off-by: Vadim Pasternak <vadimp@nvidia.com>


Applied to for-next, thanks!