Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932116AbbLGKKu (ORCPT ); Mon, 7 Dec 2015 05:10:50 -0500 Received: from mail-wm0-f52.google.com ([74.125.82.52]:38078 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754618AbbLGKKt (ORCPT ); Mon, 7 Dec 2015 05:10:49 -0500 MIME-Version: 1.0 In-Reply-To: <1449094703-31159-1-git-send-email-geyslan@gmail.com> References: <1449094703-31159-1-git-send-email-geyslan@gmail.com> Date: Mon, 7 Dec 2015 11:10:47 +0100 Message-ID: Subject: Re: [PATCH] usb: use BUG_ON() instead of BUG() From: Peter Senna Tschudin To: "Geyslan G. Bem" Cc: Greg Kroah-Hartman , Sergei Shtylyov , Julia Lawall , Saurabh Karajgaonkar , "Rafael J. Wysocki" , Felipe Balbi , linux-usb@vger.kernel.org, "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1734 Lines: 52 On Wed, Dec 2, 2015 at 11:18 PM, Geyslan G. Bem wrote: > Replace BUG() with BUG_ON(). > > Caught by coccinelle. I would mention the path to the .cocci script, but other than that: Acked-by: Peter Senna Tschudin > > Signed-off-by: Geyslan G. Bem > --- > drivers/usb/host/oxu210hp-hcd.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/usb/host/oxu210hp-hcd.c b/drivers/usb/host/oxu210hp-hcd.c > index 1f139d8..bc74aca 100644 > --- a/drivers/usb/host/oxu210hp-hcd.c > +++ b/drivers/usb/host/oxu210hp-hcd.c > @@ -394,8 +394,7 @@ static void ehci_quiesce(struct oxu_hcd *oxu) > u32 temp; > > #ifdef DEBUG > - if (!HC_IS_RUNNING(oxu_to_hcd(oxu)->state)) > - BUG(); > + BUG_ON(!HC_IS_RUNNING(oxu_to_hcd(oxu)->state)); > #endif > > /* wait for any schedule enables/disables to take effect */ > @@ -1709,9 +1708,8 @@ static void start_unlink_async(struct oxu_hcd *oxu, struct ehci_qh *qh) > > #ifdef DEBUG > assert_spin_locked(&oxu->lock); > - if (oxu->reclaim || (qh->qh_state != QH_STATE_LINKED > - && qh->qh_state != QH_STATE_UNLINK_WAIT)) > - BUG(); > + BUG_ON(oxu->reclaim || (qh->qh_state != QH_STATE_LINKED > + && qh->qh_state != QH_STATE_UNLINK_WAIT)); > #endif > > /* stop async schedule right now? */ > -- > 2.6.3 > -- Peter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/