Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753987AbbLJKUU (ORCPT ); Thu, 10 Dec 2015 05:20:20 -0500 Received: from mail-wm0-f46.google.com ([74.125.82.46]:33590 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751054AbbLJKUR (ORCPT ); Thu, 10 Dec 2015 05:20:17 -0500 Date: Thu, 10 Dec 2015 11:20:13 +0100 From: Peter Senna Tschudin To: "Geyslan G. Bem" Cc: Alan Stern , Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 7/9v2] usb: host: ehci.h: remove macros trailing semicolon Message-ID: <20151210102013.GG4013@toshiba-peter.rsr.lip6.fr> References: <1449700359-14262-1-git-send-email-geyslan@gmail.com> <1449700359-14262-7-git-send-email-geyslan@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1449700359-14262-7-git-send-email-geyslan@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1482 Lines: 40 On Wed, Dec 09, 2015 at 07:32:36PM -0300, Geyslan G. Bem wrote: > Removes trailing semicolon from macros. > > Caught by checkpatch: > "WARNING: macros should not use a trailing semicolon" > > Signed-off-by: Geyslan G. Bem As with previous, missing the changelog, but other than that the patch looks good to me. Acked-by: Peter Senna Tschudin > --- > drivers/usb/host/ehci.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h > index ea56147..15de5bf 100644 > --- a/drivers/usb/host/ehci.h > +++ b/drivers/usb/host/ehci.h > @@ -635,10 +635,10 @@ struct ehci_tt { > /* Prepare the PORTSC wakeup flags during controller suspend/resume */ > > #define ehci_prepare_ports_for_controller_suspend(ehci, do_wakeup) \ > - ehci_adjust_port_wakeup_flags(ehci, true, do_wakeup); > + ehci_adjust_port_wakeup_flags(ehci, true, do_wakeup) > > #define ehci_prepare_ports_for_controller_resume(ehci) \ > - ehci_adjust_port_wakeup_flags(ehci, false, false); > + ehci_adjust_port_wakeup_flags(ehci, false, false) > > /*-------------------------------------------------------------------------*/ > > -- > 2.6.3 > -- 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/