On Saturday, September 25, 2021 10:04:30 PM CEST Michael Estner wrote: > To avoid chrashing the kernel I use WARN_ON instead. "[] I use WARN_ON instead". Instead of what? Changelogs must be self-contained. I had to look at the diff to complete that sentence in my mind. Please don't ask reviewers to go back and forth commit messages and code in order to get the whole picture. However, my suggestion is only about the formal aspects of proper patch construction. The actual top priority is to properly solve this problem using the tips provided by Greg K-H with his review. Thanks, Fabio P.S.: "chrashing" => "crashing". > > Signed-off-by: Michael Estner <michaelestner@xxxxxx> > --- > drivers/staging/most/i2c/i2c.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/most/i2c/i2c.c b/drivers/staging/most/i2c/ i2c.c > index 7042f10887bb..e1edd892f9fd 100644 > --- a/drivers/staging/most/i2c/i2c.c > +++ b/drivers/staging/most/i2c/i2c.c > @@ -68,7 +68,7 @@ static int configure_channel(struct most_interface *most_iface, > struct hdm_i2c *dev = to_hdm(most_iface); > unsigned int delay, pr; > > - BUG_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS); > + WARN_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS);
