Commit e8794666 authored by Gabriel Rondon's avatar Gabriel Rondon Committed by Greg Kroah-Hartman
Browse files

staging: most: dim2: replace BUG_ON() in poison_channel() 



Replace BUG_ON() range check on ch_idx with a return of -EINVAL.

BUG_ON() is deprecated as it crashes the entire kernel on assertion
failure (see Documentation/process/deprecated.rst).

Signed-off-by: default avatarGabriel Rondon <grondon@gmail.com>
Link: https://patch.msgid.link/20260330182255.75241-6-grondon@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e922cb40

drivers/staging/most/dim2/dim2.c

+2 −1
Original line number Diff line number Diff line
@@ -647,7 +647,8 @@ static int poison_channel(struct most_interface *most_iface, int ch_idx) 
	u8 hal_ret;

	int ret = 0;



	BUG_ON(ch_idx < 0 || ch_idx >= DMA_CHANNELS);

	if (ch_idx < 0 || ch_idx >= DMA_CHANNELS)

		return -EINVAL;



	if (!hdm_ch->is_initialized)

		return -EPERM;