Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752493AbdDCTYZ (ORCPT ); Mon, 3 Apr 2017 15:24:25 -0400 Received: from gateway31.websitewelcome.com ([192.185.143.40]:14066 "EHLO gateway31.websitewelcome.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752065AbdDCTYY (ORCPT ); Mon, 3 Apr 2017 15:24:24 -0400 X-Greylist: delayed 1423 seconds by postgrey-1.27 at vger.kernel.org; Mon, 03 Apr 2017 15:24:24 EDT Date: Mon, 03 Apr 2017 14:00:10 -0500 Message-ID: <20170403140010.Horde.wTnKeUnd6HyAwUeuoXnLOop@gator4166.hostgator.com> From: "Gustavo A. R. Silva" To: Alan Stern Cc: Greg Kroah-Hartman , Felipe Balbi , Peter Chen , Chunfeng Yun , Mathias Nyman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Peter Senna Tschudin Subject: Re: [PATCH] usb: misc: add missing continue and refactor code References: <20170403175738.GA12790@kroah.com> In-Reply-To: User-Agent: Horde Application Framework 5 Content-Type: text/plain; charset=utf-8; format=flowed; DelSp=Yes MIME-Version: 1.0 Content-Disposition: inline X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - gator4166.hostgator.com X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - embeddedor.com X-BWhitelist: no X-Source-IP: 108.167.133.22 X-Exim-ID: 1cv7D1-000Uxr-4U X-Source: X-Source-Args: X-Source-Dir: X-Source-Sender: gator4166.hostgator.com [108.167.133.22]:55261 X-Source-Auth: garsilva@embeddedor.com X-Email-Count: 2 X-Source-Cap: Z3V6aWRpbmU7Z3V6aWRpbmU7Z2F0b3I0MTY2Lmhvc3RnYXRvci5jb20= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 724 Lines: 29 Quoting Alan Stern : > On Mon, 3 Apr 2017, Greg Kroah-Hartman wrote: > >> On Mon, Apr 03, 2017 at 09:39:53AM -0500, Gustavo A. R. Silva wrote: >> > -Code refactoring to make the flow easier to follow. >> > -Add missing 'continue' for case USB_ENDPOINT_XFER_INT. >> >> Don't do multiple things in the same patch, please make these multiple >> patches. And do the "add missing continue" first, so it can be >> backported to other kernels easier please. > OK, I will send a patchset shortly. > Also, make sure your patch does not contain gratuitous whitespace > changes. > Does it have any? I ran it through checkpatch.pl before sending it and didn't see any. Thanks -- Gustavo A. R. Silva