From: Krzysztof Kozlowski Subject: Re: [PATCH 1/2] crypto: s5p-sss - Minor coding cleanups Date: Mon, 07 Mar 2016 13:05:49 +0900 Message-ID: <56DCFE1D.7000106@samsung.com> References: <1457259436-32560-1-git-send-email-krzk@kernel.org> <56DCD301.4030401@mleia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit To: Vladimir Zapolskiy , Krzysztof Kozlowski , Herbert Xu , "David S. Miller" , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org Return-path: Received: from mailout3.w1.samsung.com ([210.118.77.13]:46050 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751840AbcCGEF5 (ORCPT ); Sun, 6 Mar 2016 23:05:57 -0500 In-reply-to: <56DCD301.4030401@mleia.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On 07.03.2016 10:01, Vladimir Zapolskiy wrote: > Hi Krzysztof, > > On 06.03.2016 12:17, Krzysztof Kozlowski wrote: >> Remove unneeded inclusion of delay.h and get rid of indentation from >> labels. > > just in case are you aware of this discussion regarding goto labels > http://lkml.iu.edu/hypermail/linux/kernel/0706.0/0809.html ? > > Practically there is no big need (neither it is a clean-up IMHO) > to change column 1 indentation of goto labels, however I do agree that > ^[a-z0-9_]\+:$ style of goto labels is 10 times more commonly used, > and that means it's a good time to update my ~/.emacs config. > > Someone may want to fix the style of another 5000+ goto labels one day, > though it is better to have an officially agreed policy beforehand. > > Anyway I do not strictly object against the change, see one my comment > below and feel free to add my > > Acked-by: Vladimir Zapolskiy Actually I was not aware of that discussion, thanks for pointing this out. Mostly I encountered the style without indentation but of course that does not mean it is the only one. > >> Signed-off-by: Krzysztof Kozlowski >> --- >> drivers/crypto/s5p-sss.c | 15 +++++++-------- >> 1 file changed, 7 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/crypto/s5p-sss.c b/drivers/crypto/s5p-sss.c >> index 5f161a9777e3..60f835455a41 100644 >> --- a/drivers/crypto/s5p-sss.c >> +++ b/drivers/crypto/s5p-sss.c >> @@ -11,7 +11,6 @@ >> * >> */ >> >> -#include >> #include >> #include >> #include > > Since you are here, can you sort headers out, please? You mean sort by name (or by christmas tree)? I can... I am not sure about benefits of such patch (which should be the motivation for change). > And I'm pretty sure some more inclusions can be safely removed > from the list. I checked them while removing delay - all of them seem to be used. Best regards, Krzysztof