From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754061Ab1GNJFb (ORCPT ); Thu, 14 Jul 2011 05:05:31 -0400 Received: from ftcsun4.faraday-tech.com ([60.248.181.187]:39057 "EHLO ftcsun4.faraday.com.tw" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753867Ab1GNJFa convert rfc822-to-8bit (ORCPT ); Thu, 14 Jul 2011 05:05:30 -0400 X-Greylist: delayed 1726 seconds by postgrey-1.27 at vger.kernel.org; Thu, 14 Jul 2011 05:05:29 EDT X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="big5" Content-Transfer-Encoding: 8BIT Subject: RE: [PATCH 4/5] fusb300_udc: Add missing breaks in switch/case Date: Thu, 14 Jul 2011 16:36:39 +0800 Message-ID: In-Reply-To: <20110714080830.GA7149@linutronix.de> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH 4/5] fusb300_udc: Add missing breaks in switch/case thread-index: AcxB/T75vUwxQ8tcRTiXOGJT0TdkpQAASCrg References: <08f6dd600dbf588e87734b3a6b33d897cd253fe3.1310289795.git.joe@perches.com> <20110714080830.GA7149@linutronix.de> From: =?big5?B?V2VuZHkgWXVhbi1Ic2luIENoZW4os6+kuMTJKQ==?= To: "Sebastian Andrzej Siewior" Cc: "Felipe Balbi" , "Greg Kroah-Hartman" , , , "Joe Perches" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org -----Original Message----- From: Sebastian Andrzej Siewior [mailto:bigeasy@linutronix.de] Sent: Thursday, July 14, 2011 4:09 PM To: Wendy Yuan-Hsin Chen(³―€ΈΔΙ) Cc: Felipe Balbi; Greg Kroah-Hartman; linux-usb@vger.kernel.org; linux-kernel@vger.kernel.org; Joe Perches Subject: Re: [PATCH 4/5] fusb300_udc: Add missing breaks in switch/case * Joe Perches | 2011-07-10 02:28:27 [-0700]: >Signed-off-by: Joe Perches >--- > drivers/usb/gadget/fusb300_udc.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > >diff --git a/drivers/usb/gadget/fusb300_udc.c >b/drivers/usb/gadget/fusb300_udc.c >index 06353e7..453bc0f 100644 >--- a/drivers/usb/gadget/fusb300_udc.c >+++ b/drivers/usb/gadget/fusb300_udc.c >@@ -688,10 +688,13 @@ static void fusb300_cmp_dbg_fifo(struct fusb300_ep *ep, > switch (length % 4) { > case 1: > golden_value = *tmp; >+ break; > case 2: > golden_value = *tmp | *(tmp + 1) << 8; >+ break; > case 3: > golden_value = *tmp | *(tmp + 1) << 8 | *(tmp + 2) << 16; >+ break; > default: > break; > That is in an #if 0 block. Maybe we should remove the #if 0? Yuan-Hsin? It's ok to remove the #if 0. But the warning will occur while compling. Sebastian Yuan-Hsin ********************* Confidentiality Notice ************************ This electronic message and any attachments may contain confidential and legally privileged information or information which is otherwise protected from disclosure. If you are not the intended recipient,please do not disclose the contents, either in whole or in part, to anyone,and immediately delete the message and any attachments from your computer system and destroy all hard copies. Thank you for your cooperation. ***********************************************************************