[PATCH v7 06/10] perf: arm_pmuv3: Add support for ARM Rainier PMU

Vincenzo Frascino posted 10 patches 9 months, 4 weeks ago
[PATCH v7 06/10] perf: arm_pmuv3: Add support for ARM Rainier PMU
Posted by Vincenzo Frascino 9 months, 4 weeks ago
Add support for the ARM Rainier CPU core PMU.

Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
---
 drivers/perf/arm_pmuv3.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/perf/arm_pmuv3.c b/drivers/perf/arm_pmuv3.c
index 0e360feb3432..3785522a08e7 100644
--- a/drivers/perf/arm_pmuv3.c
+++ b/drivers/perf/arm_pmuv3.c
@@ -1369,6 +1369,7 @@ PMUV3_INIT_SIMPLE(armv8_neoverse_v1)
 PMUV3_INIT_SIMPLE(armv8_neoverse_v2)
 PMUV3_INIT_SIMPLE(armv8_neoverse_v3)
 PMUV3_INIT_SIMPLE(armv8_neoverse_v3ae)
+PMUV3_INIT_SIMPLE(armv8_rainier)
 
 PMUV3_INIT_SIMPLE(armv8_nvidia_carmel)
 PMUV3_INIT_SIMPLE(armv8_nvidia_denver)
@@ -1416,6 +1417,7 @@ static const struct of_device_id armv8_pmu_of_device_ids[] = {
 	{.compatible = "arm,neoverse-v2-pmu",	.data = armv8_neoverse_v2_pmu_init},
 	{.compatible = "arm,neoverse-v3-pmu",	.data = armv8_neoverse_v3_pmu_init},
 	{.compatible = "arm,neoverse-v3ae-pmu",	.data = armv8_neoverse_v3ae_pmu_init},
+	{.compatible = "arm,rainier-pmu",	.data = armv8_rainier_pmu_init},
 	{.compatible = "cavium,thunder-pmu",	.data = armv8_cavium_thunder_pmu_init},
 	{.compatible = "brcm,vulcan-pmu",	.data = armv8_brcm_vulcan_pmu_init},
 	{.compatible = "nvidia,carmel-pmu",	.data = armv8_nvidia_carmel_pmu_init},
-- 
2.43.0
Re: [PATCH v7 06/10] perf: arm_pmuv3: Add support for ARM Rainier PMU
Posted by Sudeep Holla 9 months, 3 weeks ago
On Fri, Feb 21, 2025 at 06:03:45PM +0000, Vincenzo Frascino wrote:
> Add support for the ARM Rainier CPU core PMU.
>

I can't take this through Arm SoC as the DTS and dt-bindings changes
are kept separate from the driver changes in their pull request to Linus.

I need to keep the bindings(05/10) and the DTS(07/10) together to avoid
DTC warnings.

So the 2 options I see are:
Defer this driver change until the bindings last upstream.
OR
Just take the perf driver change without the binding via perf tree

For now, I will take all other patches except this.

--
Regards,
Sudeep
Re: [PATCH v7 06/10] perf: arm_pmuv3: Add support for ARM Rainier PMU
Posted by Will Deacon 9 months, 3 weeks ago
On Thu, Feb 27, 2025 at 10:10:30AM +0000, Sudeep Holla wrote:
> 
> On Fri, Feb 21, 2025 at 06:03:45PM +0000, Vincenzo Frascino wrote:
> > Add support for the ARM Rainier CPU core PMU.
> >
> 
> I can't take this through Arm SoC as the DTS and dt-bindings changes
> are kept separate from the driver changes in their pull request to Linus.
> 
> I need to keep the bindings(05/10) and the DTS(07/10) together to avoid
> DTC warnings.
> 
> So the 2 options I see are:
> Defer this driver change until the bindings last upstream.
> OR
> Just take the perf driver change without the binding via perf tree
> 
> For now, I will take all other patches except this.

Works for me. I'll pick up the driver patch.

Will
Re: [PATCH v7 06/10] perf: arm_pmuv3: Add support for ARM Rainier PMU
Posted by Sudeep Holla 9 months, 3 weeks ago
On Thu, Feb 27, 2025 at 05:14:57PM +0000, Will Deacon wrote:
> On Thu, Feb 27, 2025 at 10:10:30AM +0000, Sudeep Holla wrote:
> > 
> > On Fri, Feb 21, 2025 at 06:03:45PM +0000, Vincenzo Frascino wrote:
> > > Add support for the ARM Rainier CPU core PMU.
> > >
> > 
> > I can't take this through Arm SoC as the DTS and dt-bindings changes
> > are kept separate from the driver changes in their pull request to Linus.
> > 
> > I need to keep the bindings(05/10) and the DTS(07/10) together to avoid
> > DTC warnings.
> > 
> > So the 2 options I see are:
> > Defer this driver change until the bindings last upstream.
> > OR
> > Just take the perf driver change without the binding via perf tree
> > 
> > For now, I will take all other patches except this.
> 
> Works for me. I'll pick up the driver patch.
> 

Thanks Will.

-- 
Regards,
Sudeep