diff options
author | Quan Nguyen <[email protected]> | 2023-07-26 15:00:00 +0700 |
---|---|---|
committer | Wolfram Sang <[email protected]> | 2023-08-14 15:28:33 +0200 |
commit | 49d4db3953cb9004ff94efc0c176e026c820af5a (patch) | |
tree | fa3e55d9167949c1b413666c6341878e75bba02e /lib/memory-notifier-error-inject.c | |
parent | b610c4bbd153c2cde548db48559e170905d7c369 (diff) |
i2c: designware: Correct length byte validation logic
Commit 0daede80f870 ("i2c: designware: Convert driver to using regmap API")
changes the logic to validate the whole 32-bit return value of
DW_IC_DATA_CMD register instead of 8-bit LSB without reason.
Later, commit f53f15ba5a85 ("i2c: designware: Get right data length"),
introduced partial fix but not enough because the "tmp > 0" still test
tmp as 32-bit value and is wrong in case the IC_DATA_CMD[11] is set.
Revert the logic to just before commit 0daede80f870
("i2c: designware: Convert driver to using regmap API").
Fixes: f53f15ba5a85 ("i2c: designware: Get right data length")
Fixes: 0daede80f870 ("i2c: designware: Convert driver to using regmap API")
Cc: [email protected]
Signed-off-by: Tam Nguyen <[email protected]>
Signed-off-by: Quan Nguyen <[email protected]>
Acked-by: Jarkko Nikula <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Reviewed-by: Andi Shyti <[email protected]>
Signed-off-by: Wolfram Sang <[email protected]>
Diffstat (limited to 'lib/memory-notifier-error-inject.c')
0 files changed, 0 insertions, 0 deletions