Hi Kishore, > kishore.p@xxxxxxxxxxxx hat am 23. Dezember 2017 um 16:06 geschrieben: > > > From: Kishore KP <kishore.p@xxxxxxxxxxxx> > > Use BUG_ON instead of if condition followed by BUG. > Pointed out by Coccinelle. > > Signed-off-by: Kishore KP <kishore.p@xxxxxxxxxxxx> > Signed-off-by: Suniel Mahesh <sunil.m@xxxxxxxxxxxx> > --- > Note: > - Patch was compile tested and built(ARCH=arm) on linux-next > (latest). > - No build issues reported. > --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c > index 315b49c..7116f61 100644 > --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c > +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c > @@ -224,8 +224,7 @@ int vchiq_platform_init(struct platform_device *pdev, VCHIQ_STATE_T *state) > > platform_state = (struct vchiq_2835_state *)state->platform_state; > > - if (!platform_state->inited) > - BUG(); > + BUG_ON(!platform_state->inited); > vchiq isn't critical so i prefer WARN_ON_ONCE() here. Thanks Stefan _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
