From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0CC5AC433FE for ; Mon, 4 Apr 2022 15:59:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1378799AbiDDQB2 (ORCPT ); Mon, 4 Apr 2022 12:01:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60772 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242759AbiDDQB0 (ORCPT ); Mon, 4 Apr 2022 12:01:26 -0400 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C66401AD85; Mon, 4 Apr 2022 08:59:28 -0700 (PDT) Received: by verein.lst.de (Postfix, from userid 2407) id AB6CB68AFE; Mon, 4 Apr 2022 17:59:24 +0200 (CEST) Date: Mon, 4 Apr 2022 17:59:24 +0200 From: Christoph Hellwig To: Hector Martin Cc: Christoph Hellwig , Sven Peter , Keith Busch , Jens Axboe , Sagi Grimberg , Alyssa Rosenzweig , Rob Herring , Arnd Bergmann , Marc Zyngier , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org Subject: Re: [PATCH 6/9] nvme-apple: Add initial Apple SoC NVMe driver Message-ID: <20220404155924.GA9764@lst.de> References: <20220321165049.35985-1-sven@svenpeter.dev> <20220321165049.35985-7-sven@svenpeter.dev> <20220324061620.GA12330@lst.de> <18f3a7e6-16d1-2037-90e7-1c0b1b2fbb1c@marcan.st> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <18f3a7e6-16d1-2037-90e7-1c0b1b2fbb1c@marcan.st> User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 05, 2022 at 12:57:33AM +0900, Hector Martin wrote: > The kernel hard limit is 100-character lines, not 80-character lines. > Maintainers for existing drivers are certainly free to stick to 80 chars > if they like it that way, but I don't see why we should still be > enforcing that for new code. See bdc48fa11e46. Because 100 is completely utterly unreadable if is not for individual lines like strings, and that is actually how Linus stated it in CodingStyle. Your code as-is is completely unreadable and will not go into drivers/nvme/ in that form. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AD08CC433F5 for ; Mon, 4 Apr 2022 16:00:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8FNDn0QgFK7EkoPtEjI+56nWc6YVrOUMxsNTFJsjzwI=; b=Zlf/TIQtIiUFLC jizn7F+m/9fCTmYvLp/cSBGZkD3CmciGkv3CciV2q37SvZNZbYzxkp7EMDyySVte6BkHuU5aZu+rT ei4f68rWGYZ/88ZZ2vpfMC8hzo4SPo+0x97dSS06A1xcKLG5gSFJUQ1VNDpV4AHTt8YQDXd02NVPa S2RnuW9FlWrue70jp35jeI4N+Q3P96QSRQ5fb6Zgdl0uX+7tTd1nDaT8Y200N1vZbaB4XGU8eriQu 88+sEJH04lr/in2B6h8zj9oufDU3k8uhQjXGMnXIQy0J4gceBzWb5N3KxVlPqD9dxn1Hvk4Xyod1p e04etRcdAPFAFkjk8wHQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nbP7A-00FeuN-R4; Mon, 04 Apr 2022 15:59:36 +0000 Received: from verein.lst.de ([213.95.11.211]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nbP76-00FeqJ-70; Mon, 04 Apr 2022 15:59:34 +0000 Received: by verein.lst.de (Postfix, from userid 2407) id AB6CB68AFE; Mon, 4 Apr 2022 17:59:24 +0200 (CEST) Date: Mon, 4 Apr 2022 17:59:24 +0200 From: Christoph Hellwig To: Hector Martin Cc: Christoph Hellwig , Sven Peter , Keith Busch , Jens Axboe , Sagi Grimberg , Alyssa Rosenzweig , Rob Herring , Arnd Bergmann , Marc Zyngier , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org Subject: Re: [PATCH 6/9] nvme-apple: Add initial Apple SoC NVMe driver Message-ID: <20220404155924.GA9764@lst.de> References: <20220321165049.35985-1-sven@svenpeter.dev> <20220321165049.35985-7-sven@svenpeter.dev> <20220324061620.GA12330@lst.de> <18f3a7e6-16d1-2037-90e7-1c0b1b2fbb1c@marcan.st> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <18f3a7e6-16d1-2037-90e7-1c0b1b2fbb1c@marcan.st> User-Agent: Mutt/1.5.17 (2007-11-01) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220404_085932_453388_B20552DE X-CRM114-Status: GOOD ( 14.89 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Apr 05, 2022 at 12:57:33AM +0900, Hector Martin wrote: > The kernel hard limit is 100-character lines, not 80-character lines. > Maintainers for existing drivers are certainly free to stick to 80 chars > if they like it that way, but I don't see why we should still be > enforcing that for new code. See bdc48fa11e46. Because 100 is completely utterly unreadable if is not for individual lines like strings, and that is actually how Linus stated it in CodingStyle. Your code as-is is completely unreadable and will not go into drivers/nvme/ in that form. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel