Eric Engestrom <eric.engest...@imgtec.com> writes: > The assert was missing a `!` to become active, but since that would only turn > it into a complicated codepath for a pr_err(), let's simply replace it.
The skd assert macro prints out file and line number, which would have been nice to keep. Sorry I didn't explicitly mention that last time. -Jeff > > CC: Jeff Moyer <jmo...@redhat.com> > Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com> > --- > drivers/block/skd_main.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/block/skd_main.c b/drivers/block/skd_main.c > index 3822eae..c04e92a 100644 > --- a/drivers/block/skd_main.c > +++ b/drivers/block/skd_main.c > @@ -1905,7 +1905,7 @@ static void skd_send_internal_skspcl(struct skd_device > *skdev, > break; > > default: > - SKD_ASSERT("Don't know what to send"); > + pr_err("Don't know what to send"); > return; > > } > @@ -2105,7 +2105,7 @@ static void skd_complete_internal(struct skd_device > *skdev, > break; > > default: > - SKD_ASSERT("we didn't send this"); > + pr_err("we didn't send this"); > } > }