leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

i3c: master: dw: remove dead code from dw_i3c_master_*_xfers()

Browse Source 
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>
This commit is contained in:
Vitor Soares 2019-03-27 22:49:19 +01:00 committed by Boris Brezillon
parent 9e98c678c2
commit 124dbd750d
1 changed files with 0 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
drivers/i3c/master/dw-i3c-master.c
View File

@ -840,11 +840,6 @@ static int dw_i3c_master_priv_xfers(struct i3c_dev_desc *dev,
if (i3c_nxfers > master->caps.cmdfifodepth) if (i3c_nxfers > master->caps.cmdfifodepth)
return -ENOTSUPP; return -ENOTSUPP;
for (i = 0; i < i3c_nxfers; i++) {
if (i3c_xfers[i].len > COMMAND_PORT_ARG_DATA_LEN_MAX)
return -ENOTSUPP;
}
for (i = 0; i < i3c_nxfers; i++) { for (i = 0; i < i3c_nxfers; i++) {
if (i3c_xfers[i].rnw) if (i3c_xfers[i].rnw)
nrxwords += DIV_ROUND_UP(i3c_xfers[i].len, 4); nrxwords += DIV_ROUND_UP(i3c_xfers[i].len, 4);
@ -973,11 +968,6 @@ static int dw_i3c_master_i2c_xfers(struct i2c_dev_desc *dev,
if (i2c_nxfers > master->caps.cmdfifodepth) if (i2c_nxfers > master->caps.cmdfifodepth)
return -ENOTSUPP; return -ENOTSUPP;
for (i = 0; i < i2c_nxfers; i++) {
if (i2c_xfers[i].len > COMMAND_PORT_ARG_DATA_LEN_MAX)
return -ENOTSUPP;
}
for (i = 0; i < i2c_nxfers; i++) { for (i = 0; i < i2c_nxfers; i++) {
if (i2c_xfers[i].flags & I2C_M_RD) if (i2c_xfers[i].flags & I2C_M_RD)
nrxwords += DIV_ROUND_UP(i2c_xfers[i].len, 4); nrxwords += DIV_ROUND_UP(i2c_xfers[i].len, 4);