From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from msg-2.mailo.com (msg-2.mailo.com [213.182.54.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1656B17CE; Thu, 23 Mar 2023 04:15:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=mailo.com; s=mailo; t=1679544921; bh=amePZAswCZxnH0HeDHnJFI+gYZgT7zu0jXMkYntGVqE=; h=X-EA-Auth:Date:From:To:Cc:Subject:Message-ID:References: MIME-Version:Content-Type:In-Reply-To; b=I23woQEI4sc/sCfZ4ELvool5T+tNHo0ty69z/W+kUdjfs/7NROrQIAC4fMANAAlv+ 2X6eXvju2xkd2PtUC/ZLwYJkqNw5TgyDJXXZLfbZJRH40vu+pvjQOb1gOpis/Rvh/i BdOtEOuPlzxq1wY8WO772T0Wx+Lq5FUAuu4QYHjU= Received: by b221-3.in.mailobj.net [192.168.90.23] with ESMTP via ip-20.mailobj.net [213.182.54.20] Thu, 23 Mar 2023 05:15:21 +0100 (CET) X-EA-Auth: MqpSBc+VIot+UUjG5z9STlJgrWHjIBGzlEqbIKY79KgmtCwouDOtNFJrNhCxglWEJd+gf/ccKY2TkkvhxmO0sXsACRkL8WTt Date: Thu, 23 Mar 2023 09:44:59 +0530 From: Deepak R Varma To: Randy Dunlap Cc: Khadija Kamran , outreachy@lists.linux.dev, Parthiban Veerasooran , Christian Gromm , Greg Kroah-Hartman , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Andy Whitcroft , Joe Perches , Dwaipayan Ray , Lukas Bulwahn Subject: Re: [PATCH v2] staging: most: fix line ending with '(' Message-ID: References: <061a2fea-b3da-551d-f6d2-0da30b0ac879@infradead.org> Precedence: bulk X-Mailing-List: outreachy@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <061a2fea-b3da-551d-f6d2-0da30b0ac879@infradead.org> On Wed, Mar 22, 2023 at 08:12:23AM -0700, Randy Dunlap wrote: > Hi-- > > And FTR, I still haven't found anything in coding-style.rst that says that > lines should not end with an open parenthesis. Did I overlook it? Hi Randy, You are correct. I did not find anything clear or obvious that calls out not ending the lines with a '(' symbol. I think the other guideline from the coding style is to align the overrunning argument list in the next lines such that they match the arg1 after '('. So, I think that may be the reason why checkpatch is coded to catch such lines. Regards, Deepak. > > Thanks.