diff options
author | Vitor Soares <vitor.soares@synopsys.com> | 2019-03-27 17:49:19 -0400 |
---|---|---|
committer | Boris Brezillon <boris.brezillon@collabora.com> | 2019-03-30 03:41:44 -0400 |
commit | 124dbd750da4a9bf494ae9d8cbed364058140731 (patch) | |
tree | 34fbbbddfb604c2c3f294ce518077f5206ff4a49 | |
parent | 9e98c678c2d6ae3a17cb2de55d17f69dddaa231b (diff) |
i3c: master: dw: remove dead code from dw_i3c_master_*_xfers()
Detected by CoverityScan (Event result_independent_of_operands):
"(i3c_xfers + i).len > 65536" is always false regardless of the values
of its operands. This occurs as the logical operand of "if"
"(i2c_xfers + i).len > 65536" is always false regardless of the values
of its operands. This occurs as the logical operand of "if"
Signed-off-by: Vitor Soares <vitor.soares@synopsys.com>
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
-rw-r--r-- | drivers/i3c/master/dw-i3c-master.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c index 59279224e07f..e5a288069feb 100644 --- a/drivers/i3c/master/dw-i3c-master.c +++ b/drivers/i3c/master/dw-i3c-master.c | |||
@@ -841,11 +841,6 @@ static int dw_i3c_master_priv_xfers(struct i3c_dev_desc *dev, | |||
841 | return -ENOTSUPP; | 841 | return -ENOTSUPP; |
842 | 842 | ||
843 | for (i = 0; i < i3c_nxfers; i++) { | 843 | for (i = 0; i < i3c_nxfers; i++) { |
844 | if (i3c_xfers[i].len > COMMAND_PORT_ARG_DATA_LEN_MAX) | ||
845 | return -ENOTSUPP; | ||
846 | } | ||
847 | |||
848 | for (i = 0; i < i3c_nxfers; i++) { | ||
849 | if (i3c_xfers[i].rnw) | 844 | if (i3c_xfers[i].rnw) |
850 | nrxwords += DIV_ROUND_UP(i3c_xfers[i].len, 4); | 845 | nrxwords += DIV_ROUND_UP(i3c_xfers[i].len, 4); |
851 | else | 846 | else |
@@ -974,11 +969,6 @@ static int dw_i3c_master_i2c_xfers(struct i2c_dev_desc *dev, | |||
974 | return -ENOTSUPP; | 969 | return -ENOTSUPP; |
975 | 970 | ||
976 | for (i = 0; i < i2c_nxfers; i++) { | 971 | for (i = 0; i < i2c_nxfers; i++) { |
977 | if (i2c_xfers[i].len > COMMAND_PORT_ARG_DATA_LEN_MAX) | ||
978 | return -ENOTSUPP; | ||
979 | } | ||
980 | |||
981 | for (i = 0; i < i2c_nxfers; i++) { | ||
982 | if (i2c_xfers[i].flags & I2C_M_RD) | 972 | if (i2c_xfers[i].flags & I2C_M_RD) |
983 | nrxwords += DIV_ROUND_UP(i2c_xfers[i].len, 4); | 973 | nrxwords += DIV_ROUND_UP(i2c_xfers[i].len, 4); |
984 | else | 974 | else |