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 C6D06C4167B for ; Mon, 11 Dec 2023 17:35:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344735AbjLKRfA (ORCPT ); Mon, 11 Dec 2023 12:35:00 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50362 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344741AbjLKRez (ORCPT ); Mon, 11 Dec 2023 12:34:55 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6F8D7107 for ; Mon, 11 Dec 2023 09:35:00 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id EF17FC433C7; Mon, 11 Dec 2023 17:34:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702316100; bh=C+xcU0YxoxqS8wlJs7ViUOBkcKRqhoDYHPQNqHhIU7o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EgSMMg5SQJwi9ZJO7tXgJEZ8pHAb3w7OZkogzS3Vi50kwtTf5HzMgpP5nN5vWAACB Pyqh7ZY1N0JJW//s5JpHyEWGivj5qKUnPZwMuMawSeLSxZfzAWUtekk03NYwWZW7fh HCmTvpukqTsjl/PGl+tlwztLweC4X2P1pkS50mmfQlyq+yxRTcDqPLLuA4Wuo6GkD1 nUL/EFoKOQgGjFjeBMa1XyCflVMHEM/CBaI0I3ZKdIhQC7RawnnwuqhzgudYwxaj+y ovVrVLkKBqLhpce2H6hxJb4GMaIilOVFDuWXqLBcoh75FfAUTqzjwrE6WBnXcJiwXl KLmZ4++vOhNww== Received: from johan by xi.lan with local (Exim 4.96.2) (envelope-from ) id 1rCkC2-0006Jn-1R; Mon, 11 Dec 2023 18:35:47 +0100 Date: Mon, 11 Dec 2023 18:35:46 +0100 From: Johan Hovold To: Bjorn Helgaas Cc: Johan Hovold , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Andy Gross , Bjorn Andersson , Konrad Dybcio , Manivannan Sadhasivam , Rob Herring , Nirmal Patel , Jonathan Derrick , linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 0/6] PCI: Fix deadlocks when enabling ASPM Message-ID: References: <20231128081512.19387-1-johan+linaro@kernel.org> <20231208175312.GA803148@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231208175312.GA803148@bhelgaas> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Bjorn, On Fri, Dec 08, 2023 at 11:53:12AM -0600, Bjorn Helgaas wrote: > On Tue, Nov 28, 2023 at 09:15:06AM +0100, Johan Hovold wrote: > > Johan Hovold (6): > > PCI/ASPM: Add locked helper for enabling link state > > PCI: vmd: Fix deadlock when enabling ASPM > > PCI: qcom: Fix deadlock when enabling ASPM > > PCI: qcom: Clean up ASPM comment > > PCI/ASPM: Clean up disable link state parameter > > PCI/ASPM: Add lockdep assert to link state helper > Applied to for-linus for v6.7, thanks, Johan! I've noticed that you're pretty keen on amending commit messages. For this series, for example, I noticed that you added an American comma after "e.g." even though this is not expected in British English that I (try to) use. This risks introducing inconsistencies and frankly I see no reason for this kind of editing. British English is not an error. :) You also added a plus sign after the stable kernel versions in the comments after the CC-stable tags even though this is not the right notation for this (see the stable kernel rules). I'm more OK with you preferring to use function names over free text in commit messages even if that is not my preferred style. But generally I find it a bit odd that you insist on rewriting commit messages like this and would prefer if you did not (especially since there's no record of you having done this in the commits themselves). Fixing typos and grammar issues, or rewriting bad commit messages, is another thing of course. Johan