Re: [PATCH v6 0/2] perf: ARM CoreSight PMU support

[Will Deacon]

On Mon, Nov 07, 2022 at 05:11:07PM +0000, Besar Wicaksono wrote:
> Hi Will,
> 
> > -----Original Message-----
> > From: Will Deacon <will@xxxxxxxxxx>
> > Sent: Monday, November 7, 2022 9:53 AM
> > To: Besar Wicaksono <bwicaksono@xxxxxxxxxx>
> > Cc: suzuki.poulose@xxxxxxx; robin.murphy@xxxxxxx;
> > catalin.marinas@xxxxxxx; mark.rutland@xxxxxxx; linux-arm-
> > kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-
> > tegra@xxxxxxxxxxxxxxx; sudeep.holla@xxxxxxx;
> > thanu.rangarajan@xxxxxxx; Michael.Williams@xxxxxxx; Thierry Reding
> > <treding@xxxxxxxxxx>; Jonathan Hunter <jonathanh@xxxxxxxxxx>; Vikram
> > Sethi <vsethi@xxxxxxxxxx>; mathieu.poirier@xxxxxxxxxx;
> > mike.leach@xxxxxxxxxx; leo.yan@xxxxxxxxxx
> > Subject: Re: [PATCH v6 0/2] perf: ARM CoreSight PMU support
> > 
> > External email: Use caution opening links or attachments
> > 
> > 
> > On Thu, Sep 29, 2022 at 09:34:41PM -0500, Besar Wicaksono wrote:
> > > Add driver support for ARM CoreSight PMU device and event attributes for
> > NVIDIA
> > > implementation. The code is based on ARM Coresight PMU architecture
> > and ACPI ARM
> > > Performance Monitoring Unit table (APMT) specification below:
> > >  * ARM Coresight PMU:
> > >         https://developer.arm.com/documentation/ihi0091/latest
> > >  * APMT: https://developer.arm.com/documentation/den0117/latest
> > >
> > > The patchset applies on top of
> > >   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> > >   master next-20220524
> > >
> > > For APMT support, please see patchset:
> > https://lkml.org/lkml/2022/4/19/1395
> > >
> > > Changes from v5:
> > >  * Default get_event/format_attrs callback now returns copy of default
> > array.
> > > Thanks to suzuki.poulose@xxxxxxx for the review comments.
> > > v5: https://lore.kernel.org/linux-arm-kernel/20220928201830.45637-1-
> > bwicaksono@xxxxxxxxxx/
> > 
> > These patches no longer apply. Please can you rebase onto -rc4 and drop the
> > defconfig change from the first patch?
> 
> Why does the defconfig change need to be dropped ?

Oh, just because that file is a big source of conflicts so I'd prefer to
keep defconfig changes separate from patches which add new functionality.

Will