Re: [PATCH v4 0/2] perf: Add Raspberry Pi AXI PMU driver

Will Deacon <[email protected]>
Newsgroups org.kernel.vger.linux-perf-users,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-kernel
Message-ID <an2DHgfQsQ6TOF44@willie-the-truck>
On Wed, Aug 12, 2026 at 06:38:24AM -0700, Ian Rogers wrote:
> On Wed, Aug 12, 2026 at 1:27 AM Will Deacon <[email protected]> wrote:
> >
> > On Tue, Aug 11, 2026 at 10:24:15PM -0700, Ian Rogers wrote:
> > > This patch series adds an uncore Performance Monitoring Unit (PMU) driver
> > > for Broadcom AXI system and VideoCore VPU performance monitors found on
> > > Raspberry Pi SoCs (BCM2835 through BCM2712 / Raspberry Pi 1 through 5).
> >
> > Why are you sending four versions of this patch series, at -rc7, each in
> > reply to the previous one? Nobody is going to review that.
> 
> See the cover letter for changes. They address the sashiko reviews,
> you may not see these reviews as they are only sent to me and
> linux-perf-users, which is somewhat customary in the sashiko setup.

Have you considered running Sashiko locally given that you work at Google?

> I don't see any relevance in rc7, I'm just mailing out a new driver
> now that I have the opportunity to look at it. Whether and when it
> gets pulled upstream is up to a maintainer.

Right, as the maintainer for drivers/perf/ and I'm just asking you to
slow down a bit. I don't need 10 versions of a patch series in two days
when I'm focussed almost entirely on the upcoming merge window, which
this is too late for. You should read
Documentation/process/submitting-patches.rst which states:

  | Wait for a minimum of one week before resubmitting

> The changes are sent in reply to each other because the entire series
> is visible in one mailbox (b4 mbox), which I find easier to work with
> and provides me better context. Others use semcode to do something
> similar, but I've not explored that.

Unfortunately, this tends to confuse b4 when applying the patches and
makes it really hard to manage on the maintainers' side. I think this
is also documented in Documentation/process/submitting-patches.rst so
please do what everybody else does.

> Let me know if I should drop you from the cc, I placed you there as
> Mark and you were the only suggested reviewers from
> scripts/get_maintainer.pl but I don't want to spam uninterested
> people.

It depends if you want the patches merged...

For now, I'm ctrl+d'ing the thread.

Will
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.