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

Florian Fainelli <[email protected]>
Newsgroups org.kernel.vger.linux-perf-users,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 8/13/26 06:48, Ian Rogers wrote:
> On Thu, Aug 13, 2026 at 2:08 AM Peter Robinson <[email protected]> wrote:
>>
>> On Wed, 12 Aug 2026 at 09:28, 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.
>>
>> I'm also wondering why this is called rpi_axi_pmu where it very much
>> looks like Broadcom IP, and is possibly used in other Broadcom SoCs,
>> Florian can you confirm what this driver should be called?
> 
> I'm somewhat agnostic regarding the naming, and having a more general
> driver strikes me as a good thing. Fwiw, I based the name on a
> somewhat similarly named non-perf-compliant driver:
> https://github.com/raspberrypi/linux/blob/rpi-6.18.y/drivers/perf/raspberrypi_axi_monitor.c

Given this uses the Raspberry Pi firmware interface, having 
rpi/rasbperrypi in the name is sensible IMHO, but the IP is technically 
from Broadcom AFAICT, so using a bcm2835 prefix wouldn't be far off.

Ian, I am really having a hard time following all of your frequent 
submissions, would you mind waiting for feedback before spinning new 
versions?

Thanks!
-- 
Florian
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.