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 4ADDAC282EC for ; Tue, 11 Mar 2025 14:37: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:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:Date:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hlcozNRCUn2cDnJKUvdb8kPLVFjOiAX9tvU5DyWpFbs=; b=pMLRKYe2GnE2LiJ6YCSRgMedEB wb9pV8QFoJbnsK35KBbLTV7HK7LAhUn+h1A9xM2C0dmghegDInDZiDMnS4kzsPcnoVfXRy9PAA4c2 2u4oQuMMe6YkVlVsk9lB1DIfRQc5gzA7OYtioKA2bXrqrT0zbmjYFVpyK9uBXr+uD59Fn0hbsTcKo IfyeahKg6KBgDasHVG4DJQ99odUDIForAyAUDDv4sLZgIHZzyRfscwrwgDVDAccbQZnCOOYzltozG IPvTc1YCMdGNMPcoe6X5RVDHx0taVfGXDRnywmGAipfb5KBqcmn9WSc61Yyx/egu+MWC1DU+kuyp5 JMS90K3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1ts0jY-000000060Cl-44jX; Tue, 11 Mar 2025 14:37:28 +0000 Received: from mail-ej1-x641.google.com ([2a00:1450:4864:20::641]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1ts0hu-00000005zni-1YTG for linux-arm-kernel@lists.infradead.org; Tue, 11 Mar 2025 14:35:47 +0000 Received: by mail-ej1-x641.google.com with SMTP id a640c23a62f3a-ab7430e27b2so917586166b.3 for ; Tue, 11 Mar 2025 07:35:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1741703744; x=1742308544; darn=lists.infradead.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:date:from:from:to :cc:subject:date:message-id:reply-to; bh=hlcozNRCUn2cDnJKUvdb8kPLVFjOiAX9tvU5DyWpFbs=; b=D69VVkJ1IuLwZfiKFeJD/eMz8xfXuQL3q7yunwhQgqBOPNV/LN7EMZ0TIYh7+NVvAT oz8jvC/dJE/I1b0V+a4AzA41Bz0yC3XwPhJNdotHHT1hZHzUGUss+6NgbnIuJPmSXbUF iBO6/8hb2OWy1W976vZAue2zdOPXKWYM93O641ahwAXdQEHziK/f8e1zIbWX1HmlZB3N isk4hR1YPAOVdc6TPiAf4nHmbAAyHaZy1qIfre7HScqxHeLhVpOqO9K+bDZrLqVdxLDf Ha3qikRJlH//BMeKyD84OIgLbhruD6lg0cQooTHymWquASERja5OeukgotUwTcMVp7Iz hiaQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741703744; x=1742308544; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:date:from :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=hlcozNRCUn2cDnJKUvdb8kPLVFjOiAX9tvU5DyWpFbs=; b=PVP5Mn41c910DINLjIv4Lq2sEyjXxzb2CC4IdhndSUwHz64qwVOg3DQKa+FevzRQRN kmKQV7vMRpdzpIfLBGs1/3Gswa/Q+3e5Tqbae1gxafpiaq6hefAA6ZnpVSbMmmvZ7P89 wFx3uJ/E/n+04oEfi1lhgXnF2TSQ9/cxG0ukdCUE1JtfhYUI0fpGatr4Q5cGVKMEpqnM VPU5W9FstskW456xpPFSW6GGMS7q7ZrhmFgAAwjUVJt4txufTDj/GsWewNJjBWo2RF/K WZM96+4dLmG+tZ79sZOH4f7R7/ZfT1oiUQojlQKK9+ChuscvNpI4KMpNg7kUbsFcKpUl OjNw== X-Forwarded-Encrypted: i=1; AJvYcCWGOV5A9DPDu6httBlgnk+ON52jFlyDyw1CNql3kcVwSxoTIgYCFRshKaThKkmTXYVK2dJxvgK4LG7QPu4VsP7V@lists.infradead.org X-Gm-Message-State: AOJu0YwEslcY2y01WJNMEVDgFSP0T+v0C4C2Xl8omo99YRlzqoWhVI1V YGEPQtkg6SrwDWwm8KIlq6yGSbDDim2fXNYihvSXObOXS29QcbbQ6mkxWIegzno= X-Gm-Gg: ASbGncsY7Au99I/tN4AtvZTogl09w8PWLZ8aN2IIRWHhAhA9DZlwiKArofAw/lYVMQG bvhpFuXCseZs635fh6Ju8msju2aHnJCpvGd1SUpN9hBk9w5yMonr74h0UGbGbMGaRRuCX8fz+1t ZnjtwbY5SCWcl0ugoxz1Olb2mWuouPR6K1gyLSW/Mm3qGbHvab4QV6ixJD3fGlGzv2CXO0OBlW2 mozGJVB57x+1W3NCopALm3TBjnXYKrCoN6xBqcdJ0DcSl5HJekHufcOIBtsB0Rjk8kZV8MpdvQo 4s97rtCXWKZr/9WGL6RqROgs+0o/UWyHg/lSgk5I+UrBtUgb6UKMmYHqK+hCJO2sgXlAyjX96dr AS5GgDsPWE+CZ X-Google-Smtp-Source: AGHT+IFLdM9vCnSAq2tyDNO5yPeJsNiGhv4R/i7mo9dqAzSuL9h81HaDVtVXHdqZ5ge8YgeZcopV0g== X-Received: by 2002:a17:907:d27:b0:ac2:9e1a:bf95 with SMTP id a640c23a62f3a-ac29e1ac5camr1178043966b.18.1741703743784; Tue, 11 Mar 2025 07:35:43 -0700 (PDT) Received: from localhost (host-87-14-236-98.retail.telecomitalia.it. [87.14.236.98]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ac2a96a22d8sm278255366b.158.2025.03.11.07.35.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 Mar 2025 07:35:43 -0700 (PDT) From: Andrea della Porta X-Google-Original-From: Andrea della Porta Date: Tue, 11 Mar 2025 15:36:53 +0100 To: Rob Herring Cc: Krzysztof Wilczynski , Andrea della Porta , Michael Turquette , Stephen Boyd , Krzysztof Kozlowski , Conor Dooley , Florian Fainelli , Broadcom internal kernel review list , Lorenzo Pieralisi , Manivannan Sadhasivam , Bjorn Helgaas , Linus Walleij , Catalin Marinas , Will Deacon , Bartosz Golaszewski , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Saravana Kannan , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-gpio@vger.kernel.org, Masahiro Yamada , Stefan Wahren , Herve Codina , Luca Ceresoli , Thomas Petazzoni , Andrew Lunn Subject: Re: [PATCH v7 03/11] dt-bindings: pci: Add common schema for devices accessible through PCI BARs Message-ID: References: <20250310212125.GB2377483@rocinante> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250311_073546_409205_423D3F7E X-CRM114-Status: GOOD ( 27.11 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Rob, On 08:32 Tue 11 Mar , Rob Herring wrote: > On Mon, Mar 10, 2025 at 4:21 PM Krzysztof Wilczynski wrote: > > > > Hello, > > > > [...] > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > index d45c88955072..af2e4652bf3b 100644 > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -19752,6 +19752,7 @@ RASPBERRY PI RP1 PCI DRIVER > > > M: Andrea della Porta > > > S: Maintained > > > F: Documentation/devicetree/bindings/clock/raspberrypi,rp1-clocks.yaml > > > +F: Documentation/devicetree/bindings/pci/pci-ep-bus.yaml > > > F: Documentation/devicetree/bindings/pinctrl/raspberrypi,rp1-gpio.yaml > > > F: include/dt-bindings/clock/rp1.h > > > F: include/dt-bindings/misc/rp1.h > > > > I would be happy to pick this via the PCI tree as per the standard > > operating procedure. However, the MAINTAINERS changes do not exist > > for us yet, and are added in the first patch of the series, which is > > not ideal. > > > > I can add the missing dependency manually, but that would cause issues > > for linux-next tree, which is also not ideal. > > > > I saw some review feedback, as such, when you are going to be sending > > another version, can you make MAINTAINERS changes to be the last patch, > > perhaps. Basically, something standalone that perhaps whoever will pick > > the misc patch could also pick and apply at the same time. > > > > Alternatively, someone else picking up the PCI dt-bindings would work, too. > > > > Your thoughts? > > I guess I missed this in review, but why is a common schema buried in > a device maintainer entry? Also, an entry in MAINTAINERS is redundant > anyway because get_maintainers.pl can fetch maintainers from the > schema file. Oh nice, I've added all that .yaml entries in MAINTAINERS because I saw many reference already existing there, so I was thinking that was a good behaviour. Now I guess I can get rid of all .yaml references in MAINTAINERS file from all my patches, since they will be solved automatically by get_mainatainer.pl... There's only one minor caveat though: I have a middle name, and it turns out that get_maintainer.pl is skipping my first name. Unluckily I'm not a Perl guy so I guess I have to dive a little bit in Perl regex and send a separate patch for that. Many thanks, Andrea > > Rob