Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp5038801rdb; Sat, 30 Dec 2023 03:34:10 -0800 (PST) X-Google-Smtp-Source: AGHT+IF3YUhjrGMAKSzYfnCXViqrnBrK/hvkj71U7sMzywmcSrBfsjx2p0braIMtQNlIrTbkOO4S X-Received: by 2002:a05:6a21:18e:b0:196:c84:a441 with SMTP id le14-20020a056a21018e00b001960c84a441mr6330655pzb.10.1703936049797; Sat, 30 Dec 2023 03:34:09 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703936049; cv=none; d=google.com; s=arc-20160816; b=fiXOF8cjBUX9sZrwB8+531SpUiIwuIGNc9aSWFPix/3jT7dZhhFW/bz1NpifnRpugs sAvZ8K3fMBsFLxciHHMeAe9oI7QtmWLpIkpOxP3fAu4PISSo25QpSvzb0HZwF32CrDyj Xdm3FmKUsRmYa5GbZXCEIPROFakepfH4qgjV5wocp0+0WiBhJTJ+pCAI3UeDl3f6n8Ap ajdzM2gLBCAOn+UriWBvuJi+KDe0SCT8WHjdYMAdk+uFgeeErb3Ge/o00W0/6YHr6hLJ PyyB8ULbAS9SPs72UixDvWekB4II5FnFdJGElf70EvuFAGtFXp6EYkbItKwQJ+2l4jzN 0BGQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :subject:cc:to:from:date:dkim-signature; bh=UrhdpnI3mSzV0BkabTPJi2Iz3jQ6YOnNLH3r3hNuNiw=; fh=P+aR7BMBx7OllCTbw4jX5B/9Ll6GDcw08YVHEAn3ypU=; b=Zw8w4McNNfj8vK6qXrhKIRpPYQUC65AaaCohIfDdRTZUKkg+THKN9kthAY3g844+sE g3scc0wogCkddfgvn9wDQzQmSNgYKV0pdRuHfVlWGRlwWrBKaLKdOknV4xTJroyTzXWt ClUjft3FFHnLyldUFboEVx+2wOHzvywtEpaeTpGaGrnGxJeq16DjLz7K70Niih60ZLj1 HnWeHv2ws91b/uj066NI4fopVdA2oVp83Ar4Vp2EANVdzZt74SVUKVgR4LuyzcomXTHp JcmxGs4DwRH91l14309NHfDu8n/Rgumg1VbJvbCgTkG8FZ9lEPx7dvRnU6nHTLkBx25V nu+Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=O8UwZWxi; spf=pass (google.com: domain of linux-kernel+bounces-13402-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13402-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id v67-20020a632f46000000b005cdbebc0eedsi15405947pgv.538.2023.12.30.03.34.09 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 30 Dec 2023 03:34:09 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-13402-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=O8UwZWxi; spf=pass (google.com: domain of linux-kernel+bounces-13402-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13402-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 77EC12813D4 for ; Sat, 30 Dec 2023 11:34:09 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 630247497; Sat, 30 Dec 2023 11:34:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="O8UwZWxi" X-Original-To: linux-kernel@vger.kernel.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 936D98BE7; Sat, 30 Dec 2023 11:34:01 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3D9ADC433C7; Sat, 30 Dec 2023 11:33:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1703936040; bh=e4zIj9WN1rHOoaFgv5qNtqov46wr61rNTXnZQylXXXg=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=O8UwZWxiI8abpS9S3Lbzp3CPK0eqfMjQb8MxCb4ajF5b2Lw/b81zF9pvi64laJn7h FQjvOzNMA9nRZgsoGmd+qKq7Hiz07IfoAtI7+igco24jicf+IDTtKIypIbPLufzxwB EEk46PwTcc6kMTIios4IdViifsNwTLZS/tX9h0JuDgzw0NyPASanMUPtc9YwiPYOPw bk6gBmqQgp3qakzg+tjBMbJaQYxpPShB8DJHCmNV3GwXl05cM6RIkeqTc/bKVSr3WE /K6My5E7LrgKED24TcamFDb6b7YiJEPA1FcbX9DfRk+ICfZDsnw0G3FP/XR37xMzH5 AF8odw6CUHh6g== Date: Sat, 30 Dec 2023 11:33:54 +0000 From: Jonathan Cameron To: Petre Rodan Cc: Andy Shevchenko , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Andreas Klinger , Lars-Peter Clausen , Angel Iglesias , Matti Vaittinen Subject: Re: [PATCH v2 07/10] iio: pressure: mprls0025pa.c whitespace cleanup Message-ID: <20231230113354.0abaa6c1@jic23-huawei> In-Reply-To: References: <20231224143500.10940-1-petre.rodan@subdimension.ro> <20231224143500.10940-8-petre.rodan@subdimension.ro> X-Mailer: Claws Mail 4.2.0 (GTK 3.24.38; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 27 Dec 2023 19:39:28 +0200 Petre Rodan wrote: > On Wed, Dec 27, 2023 at 06:34:25PM +0200, Andy Shevchenko wrote: > > On Sun, Dec 24, 2023 at 04:34:52PM +0200, Petre Rodan wrote: > > > Fix indentation and whitespace in code that will not get refactored. > > > > > > Make URL inside comment copy-paste friendly. > > > > > return dev_err_probe(dev, ret, > > > - "honeywell,pmin-pascal could not be read\n"); > > > + "honeywell,pmin-pascal could not be read\n"); > > > > As done elsewhere, here and in other similar places fix the indentation > > by making first character on the latter line to be in the same column as > > the first character after the opening parenthesis. > > I triple-checked that I am following the max 80 column rule, the parenthesis > rule and the 'do not split printk messages' rules in all my code in these 10 patches. > precisely so I don't get feedback like this one. > if the parenthesis rule makes the line longer then 80 chars I right-align to > column 80 as seen above. I'm not aware of (and can't immediately see) anything about right aligning to 80 columns. It's fine to align it less if line length is long but normally people go with aligning to one tab more than the start of the block. > that is what I understand from the latest coding style document and that is what > I will follow. > > in this particular case if I were to ignore the 80 column rule we would end up on > column 90 if I were to follow your feedback (open parenthesis is at column 45 > and the error takes 45 chars more). It's fine to do this in the interests of readability. People differ in opinion on what constitutes 'significant readability' and I'd be happy with either a shorter alignment (single tab more than line above) or going over 80 chars in this case. Jonathan > > peter >