media: dib9000: Use min() instead of doing it manually
authorzhaoxiao <zhaoxiao@uniontech.com>
Wed, 17 Nov 2021 03:06:56 +0000 (03:06 +0000)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Fri, 19 Nov 2021 16:08:37 +0000 (16:08 +0000)
Fix following coccicheck warning:
drivers/media/dvb-frontends/dib9000.c:261:10-11: WARNING opportunity for min()
drivers/media/dvb-frontends/dib9000.c:345:10-11: WARNING opportunity for min()

Link: https://lore.kernel.org/linux-media/20211117030656.14984-1-zhaoxiao@uniontech.com
Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, zhaoxiao <zhaoxiao@uniontech.com>
Signed-off-by: zhaoxiao <zhaoxiao@uniontech.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/dvb-frontends/dib9000.c

index 04d92d6142797e658ca0747f330f5c935af2a2c3..914ca820c174b05e2a110c2ba0ff7476e03b01c6 100644 (file)
@@ -258,7 +258,7 @@ static int dib9000_read16_attr(struct dib9000_state *state, u16 reg, u8 *b, u32
                state->i2c_write_buffer[0] |= (1 << 4);
 
        do {
-               l = len < chunk_size ? len : chunk_size;
+               l = min(len, chunk_size);
                state->msg[1].len = l;
                state->msg[1].buf = b;
                ret = i2c_transfer(state->i2c.i2c_adap, state->msg, 2) != 2 ? -EREMOTEIO : 0;
@@ -342,7 +342,7 @@ static int dib9000_write16_attr(struct dib9000_state *state, u16 reg, const u8 *
                state->i2c_write_buffer[0] |= (1 << 4);
 
        do {
-               l = len < chunk_size ? len : chunk_size;
+               l = min(len, chunk_size);
                state->msg[0].len = l + 2;
                memcpy(&state->i2c_write_buffer[2], buf, l);