| Message ID | 20200420042816.18989-1-yanaijie@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | i2c: busses: remove unneeded conversion to bool | expand |
在 2020/4/26 16:16, Wolfram Sang 写道: > On Mon, Apr 20, 2020 at 12:28:16PM +0800, Jason Yan wrote: >> The '>' expression itself is bool, no need to convert it to bool again. >> This fixes the following coccicheck warning: >> >> drivers/i2c/busses/i2c-qup.c:960:48-53: WARNING: conversion to bool not needed here >> drivers/i2c/busses/i2c-qup.c:962:47-52: WARNING: conversion to bool not needed here >> drivers/i2c/busses/i2c-qup.c:1531:29-34: WARNING: conversion to bool not needed here >> drivers/i2c/busses/i2c-qup.c:1533:29-34: WARNING: conversion to bool not needed here >> >> Signed-off-by: Jason Yan <yanaijie@huawei.com> > > Applied to for-next, thanks! But please fix $subject to have the driver > name "qup" next time. > OK, get it. Thanks, Jason
diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c index 155dcde70fc9..25d5fe2f8316 100644 --- a/drivers/i2c/busses/i2c-qup.c +++ b/drivers/i2c/busses/i2c-qup.c @@ -956,10 +956,8 @@ static void qup_i2c_conf_v1(struct qup_i2c_dev *qup) u32 qup_config = I2C_MINI_CORE | I2C_N_VAL; u32 io_mode = QUP_REPACK_EN; - blk->is_tx_blk_mode = - blk->total_tx_len > qup->out_fifo_sz ? true : false; - blk->is_rx_blk_mode = - blk->total_rx_len > qup->in_fifo_sz ? true : false; + blk->is_tx_blk_mode = blk->total_tx_len > qup->out_fifo_sz; + blk->is_rx_blk_mode = blk->total_rx_len > qup->in_fifo_sz; if (blk->is_tx_blk_mode) { io_mode |= QUP_OUTPUT_BLK_MODE; @@ -1528,9 +1526,9 @@ qup_i2c_determine_mode_v2(struct qup_i2c_dev *qup, qup->use_dma = true; } else { qup->blk.is_tx_blk_mode = max_tx_len > qup->out_fifo_sz - - QUP_MAX_TAGS_LEN ? true : false; + QUP_MAX_TAGS_LEN; qup->blk.is_rx_blk_mode = max_rx_len > qup->in_fifo_sz - - READ_RX_TAGS_LEN ? true : false; + READ_RX_TAGS_LEN; } return 0;
The '>' expression itself is bool, no need to convert it to bool again. This fixes the following coccicheck warning: drivers/i2c/busses/i2c-qup.c:960:48-53: WARNING: conversion to bool not needed here drivers/i2c/busses/i2c-qup.c:962:47-52: WARNING: conversion to bool not needed here drivers/i2c/busses/i2c-qup.c:1531:29-34: WARNING: conversion to bool not needed here drivers/i2c/busses/i2c-qup.c:1533:29-34: WARNING: conversion to bool not needed here Signed-off-by: Jason Yan <yanaijie@huawei.com> --- drivers/i2c/busses/i2c-qup.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-)