Commit e59a9e50 authored by zhaoxiao's avatar zhaoxiao Committed by Mauro Carvalho Chehab
Browse files

media: dib9000: Use min() instead of doing it manually 

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: default avatarzhaoxiao <zhaoxiao@uniontech.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent 98a1ca29

drivers/media/dvb-frontends/dib9000.c

+2 −2
Original line number Diff line number Diff line
@@ -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);