From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5965043380; Mon, 20 Nov 2023 17:51:00 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E585842DDE; Mon, 20 Nov 2023 17:50:59 +0100 (CET) Received: from wout5-smtp.messagingengine.com (wout5-smtp.messagingengine.com [64.147.123.21]) by mails.dpdk.org (Postfix) with ESMTP id 8DDA842DD2 for ; Mon, 20 Nov 2023 17:50:58 +0100 (CET) Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id 147CC3200A7D; Mon, 20 Nov 2023 11:50:53 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Mon, 20 Nov 2023 11:50:54 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1700499053; x=1700585453; bh=KYK0Kd1Os8DUaXqrCi9rfluAx58aw5qpK1z Q4ARlkbo=; b=Cxbe5ZXrb5YSQP76T8vZghL/UU+u9QdXcQ+mwUWGI6yODI8TKd6 LXDCw8MspMpni3fZNupo0SVsMaqAdL0GFOopdhrKktJvIFemps1sM1aeEEPuYo+Z X91qYcVyW+aCLlac0uRJlrzErE97eUMY4GFy0LnFmbQCMI+WH2vXdDx3Eb6MJf5f U6BR+bKbkcg81IPiAjdC3iJKiA85MNrDiw5o0oXOO0B2aSgl9mvkYwacU79hlVP9 8ZHq9sn8D73gD6nrmtfkKHPqAOV6lAjf8EnBF5TrtJGNto5PYLGI8RmLPAUzbLhR 8v9MDH/8Fvg+GIaC/SSgwdziiCs/jhjnBdQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t= 1700499053; x=1700585453; bh=KYK0Kd1Os8DUaXqrCi9rfluAx58aw5qpK1z Q4ARlkbo=; b=f211bny68wOknMei8ALQxJjAeu7skH/2zPBo/yL5EHCvpgljTXn SJJLRvct5U8+nTMvS2tJ8tLnWbRm2Ku0jOYHPVW5wwnECeKoznjGjVQGBJ6L3xSV efxZHuT4uxH36epbQ3fS5hhQUQaD8SxpEKiF3BHhKX66tMYr1QPoLEODmAParboE fIEfJYftRA87Czg8+DWWC08bVoaL1KwPIjIse/4aY9XvrbjQHnRB6bvXavONbpwG V/2eg0T5lDUaQX7ipYiojFChvp7w9ifDhvIVS9gAFhSUU6+BK4dppso9/hB3Jvcd 8jUTzvhergIBDsH4VBvPsfpq7oKHUtAjEqA== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrudegjedgieejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvvefufffkjghfggfgtgesthhqredttddtjeenucfhrhhomhepvfhhohhm rghsucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenuc ggtffrrghtthgvrhhnpeegtddtleejjeegffekkeektdejvedtheevtdekiedvueeuvdei uddvleevjeeujeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfh hrohhmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Feedback-ID: i47234305:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 20 Nov 2023 11:50:51 -0500 (EST) From: Thomas Monjalon To: Juraj =?utf-8?B?TGlua2XFoQ==?= Cc: Honnappa.Nagarahalli@arm.com, bruce.richardson@intel.com, jspewock@iol.unh.edu, probb@iol.unh.edu, paul.szczepanek@arm.com, yoan.picchi@foss.arm.com, stephen@networkplumber.org, dev@dpdk.org Subject: Re: [PATCH v6 2/2] dts: reformat to 100 line length Date: Mon, 20 Nov 2023 17:50:50 +0100 Message-ID: <2170702.Mh6RI2rZIc@thomas> In-Reply-To: <20231120123646.43994-2-juraj.linkes@pantheon.tech> References: <20231016064559.17609-2-juraj.linkes@pantheon.tech> <20231120123646.43994-1-juraj.linkes@pantheon.tech> <20231120123646.43994-2-juraj.linkes@pantheon.tech> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 20/11/2023 13:36, Juraj Linke=C5=A1: > Reformat to 100 from the previous 88 to unify with C recommendations. >=20 > The C recommendation is the maximum with the ideal being 80. The Python > tools are not suitable for this flexibility. >=20 > We require all patches with DTS code to be validated with the > devtools/dts-check-format.sh script, part of which is the black > formatting tool. We've set up black to format all of the codebase and > the reformat is needed so that future submitters are not affected. >=20 > Signed-off-by: Juraj Linke=C5=A1 > Acked-by: Jeremy Spewock In general, I don't like doing large cosmetic changes, but it looks mandatory to allow automatic formatting with black. Applied, thanks. Note that my pylama is still emitting warning when it goes longer than 79. It may be a problem in my environment but something to check. By the way, why are we using pylama in the script instead of directly calling the linters we are interested in?