From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754747AbbGAQHs (ORCPT ); Wed, 1 Jul 2015 12:07:48 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:40280 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754386AbbGAQHl (ORCPT ); Wed, 1 Jul 2015 12:07:41 -0400 Date: Wed, 1 Jul 2015 09:07:40 -0700 From: Greg KH To: Leung Timothy Chi King Cc: fabio.falzoi84@gmail.com, joe@perches.com, lambert.quentin@gmail.com, tinajohnson.1234@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Code cleanup - else is not generally useful after a break or return Message-ID: <20150701160740.GA31389@kroah.com> References: <20150701155521.GA3613@ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150701155521.GA3613@ubuntu> User-Agent: Mutt/1.5.23+89 (0255b37be491) (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Your subject needs a hint as to what part of the kernel it is changing, please fix that up to match what other patches have looked like for this file. On Wed, Jul 01, 2015 at 08:55:21AM -0700, Leung Timothy Chi King wrote: > Ran checkpatch.pl on driver/staging/rts5208/sd.c and foudn a WARNING > 'else is not generally useful after a break or return' on line 3523. I > removed the else without affecting the logic and the WARNING is gone. I > had compiled and modprobe rts5208 to double check as well. That's good, but not needed here, please say why you did what you did, but not in a narrative like you did. > > Regards, > Tim This isn't needed either. Can you fix up and resend? thanks, greg k-h