On 05/17/2017 12:29 AM, Wolfram Sang wrote:
+ switch (dev->master_cfg & DW_IC_CON_SPEED_MASK) {
+ case DW_IC_CON_SPEED_STD:
+ i2c_dw_set_timings_sm(dev);
+ break;
+ case DW_IC_CON_SPEED_FAST:
+ i2c_dw_set_timings_fm(dev);
+ break;
+ case DW_IC_CON_SPEED_HIGH:
+ i2c_dw_set_timings_hsm(dev);
+ break;
}
I would put same suffixes to the helper functions, i.e.
_sm -> _std
_fm -> _fast
_hsm-> _high
BTW is this a bugfix? I tend to think, yes.
Maybe in grey area but not enough for linux-stable I think. Unless log
spamming qualifies as a regression.
Lorenzo: does my patch fix the issue for you? I was waiting your comment
before sending version 2 addressing Andy's comment.
--
Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
