From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f65.google.com (mail-ej1-f65.google.com [209.85.218.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8641179EA for ; Tue, 11 Mar 2025 14:35:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.65 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741703748; cv=none; b=ILXed77SuEy/cPmZE0wzwqzcZGlgXSd6D2T8kfSBsy6HMpswz2Wtm7J6ZH0hv2Z7hQkjuEo9MQfPg3nCJn2V4oknuO97Ui5LDDeIKsPC8SNEJzAm/fkkaA34w8rQ6snB2vQjhbKTWL7Faju7ttPVTeVJCEu7o4DHn4u16eBEpY4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741703748; c=relaxed/simple; bh=rty27olN5qtT2FiJsZAnVeMcI3IQWlc6A0m+F2XfCzA=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=LoTMhCy+aiJnXahBtNFH/rSS6J1jYuB8njIq1CHRJGvITQSovSUZP7EImBLCd4e4yJY9RbZq3WOzdwQsXM+OuPjLyuDWtDPfWzFj3k76mTk+UmynWWAzA9GE02r1T3Ct4OhE/I9bYnqjnl8PGYtoy9l9osd0dNdGs95J99PbiiM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=GU3jOMES; arc=none smtp.client-ip=209.85.218.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="GU3jOMES" Received: by mail-ej1-f65.google.com with SMTP id a640c23a62f3a-abec8b750ebso941290466b.0 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=vger.kernel.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=GU3jOMESekZDdTWy5z1Z3dpQgkXwFi6DXlPpnvBhmzJ7I2BBkGP4aDHIhYYp2cl/H1 Jb09vev6yW/KF5JPE0/4XtrPgauwJDhYr0OhGS12+qPy/Llo/sMH2jRnaXa/0NgpmdbM MTftcdpClpyKf54p2vFufW7gQ49ecO1YPksWebiwWsUzUnA4EMOJst1yMmb60lG1qugf ckz0zalrLXHu6UrY/RJJh9Bv/IVR9we8MEJWyYcivSaMJpky3h/JHlzjzqljeiHyBLUk 4pviQhS7SYOV1LlEHPEdK3IlpMXgZ1dPKsfSUv1gJoCNIamLzqqfbe0HEV3Vgpt6vxxH tTtg== 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=Djc9XhJhQ7zjN8Rv3KEudnhFn1T5ksuTccwZ4KrldnxOOTp/eNbBxc10A54Ga9oGqP p4TObcvJhJInteQdxuhogrmq6SSdqwbBMYLq8NhnoUTbH5fxDyCV2LiPojs6GGmnweN4 o6U9RqDy9Ml1QN0oZKmTcmjlxQO5BKVHrSRWmsnMlB6uDcXcwWIYjitG4hpKKCL5eq8S K0/dbRgRaoOhlLhnBWWokAz0jjPUBHuA+n1vHhzsWkRvCOxFoK16HcPkfPJ9E50Z5f0n QLNmW3eszcOcgcYArkBLHWKA3CIxGaEaa47xQcgptdgkZddOZgBMbmCyXed1zLl9V7hw 6DYA== X-Forwarded-Encrypted: i=1; AJvYcCWgu/EXjhtMvz46MNz+V8yqxskFhLkW06f+zKsnKdNjU3Mq1sxEmXfEYUURciM3R+MYgbhq/nmzE9c=@vger.kernel.org X-Gm-Message-State: AOJu0YwixjXPMdUyASCZLrNx6cniq/GRvIo5/QRdfWSzut9r0u9domTe RKXUhZYIxVIcj4vgNNn8F28b41D44IydQ2kRdudHGPyElDbtWc/CCVd0VKc5oGQ= X-Gm-Gg: ASbGncuoqYwEHK+3JmbwQXN2Cz5xT9tpKW5E/Eat9kAvgqUCUsOun3Mps/GEIC56f0c sO11QXh0VdgP1rSHoQ/7O5UgcxVRLk2dENm0cm7/9VdmFDTELtvh9h16FTCRbPIvjU4ijV6FjCI doM2TIT56wq20IhdIKjmqgxm+cae48eGI1rLngD/QjSRDkYw33+n0YK0tCDLWu7oeBhUiKCoIA+ uKUqfhWVqoPoSANEzln2Y4dtiBwB1HcC/3bdm86XYTnhC8puMXdmrt2cZ57MCBZKlviGJkvdwYK DrbUXGj7k/b8scSieR4VyOebhSDaZExHQlxY3k98Xy+krlDGy4NfBam/KuMU8HDPKtqo29fmImb WBtkwer3ySPfl 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> Precedence: bulk X-Mailing-List: linux-pci@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: 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