From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751926AbcGRI7L (ORCPT ); Mon, 18 Jul 2016 04:59:11 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:52865 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751424AbcGRI7J (ORCPT ); Mon, 18 Jul 2016 04:59:09 -0400 Date: Mon, 18 Jul 2016 10:59:05 +0200 From: Pavel Machek To: Linus Torvalds Cc: Herbert Xu , Tim Chen , Ingo Molnar , "H. Peter Anvin" , Dan Carpenter , "David S. Miller" , Thomas Gleixner , Ingo Molnar , the arch/x86 maintainers , Megha Dey , "Wang, Rui Y" , Denys Vlasenko , Xiaodong Liu , Linux Crypto Mailing List , Linux Kernel Mailing List , kernel-janitors , Andrew Morton , Peter Zijlstra Subject: Re: [patch] crypto: sha256-mb - cleanup a || vs | typo Message-ID: <20160718085905.GA5701@amd> References: <20160629144242.GE22818@mwanda> <8538242a-eab7-127e-e47e-26027fee4f6d@zytor.com> <1467319339.22178.256.camel@linux.intel.com> <20160701075559.GA11902@gmail.com> <20160701092806.GA13424@gondor.apana.org.au> <20160701101329.GA3833@gmail.com> <20160708162803.GA29111@linux.intel.com> <20160708164502.GA3064@gondor.apana.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 2016-07-08 10:19:26, Linus Torvalds wrote: > [ rare comment rant. I think I'll do this once, and then ignore the discussion ] > > On Fri, Jul 8, 2016 at 9:45 AM, Herbert Xu wrote: > > > > Nack. As I said the commenting style in the crypto API is the > > same as the network stack. So unless we decide to change both > > please stick to the current style. > > Can we please get rid of the brain-damaged stupid networking comment > syntax style, PLEASE? Please? :-). Having different comment styles between networking and the rest is confusing. And yes, this uglyness is documented for net/ and drivers/net/, but not for crypto/, so at the very least Documentation/CodingStyle should be updated. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html