mbox series

[v5,0/2] Add CCPI2 PMU support

Message ID 20190830125436.16959-1-ganapat@localhost.localdomain (mailing list archive)
Headers show
Series Add CCPI2 PMU support | expand

Message

Ganapatrao Kulkarni Aug. 30, 2019, 12:54 p.m. UTC
From: Ganapatrao Kulkarni <gkulkarni@marvell.com>

Add Cavium Coherent Processor Interconnect (CCPI2) PMU
support in ThunderX2 Uncore driver.

v5:
	Fixed minor bug of v4 (timer callback fuction
	was getting initialized to NULL for all PMUs).

v4:
	Update with review comments [2].
	Changed Counter read to 2 word read since single dword read is misbhehaving(hw issue).

[2] https://lkml.org/lkml/2019/7/23/231

v3: Rebased to 5.3-rc1

v2: Updated with review comments [1]

[1] https://lkml.org/lkml/2019/6/14/965

v1: initial patch


Ganapatrao Kulkarni (2):
  Documentation: perf: Update documentation for ThunderX2 PMU uncore
    driver
  drivers/perf: Add CCPI2 PMU support in ThunderX2 UNCORE driver.

 .../admin-guide/perf/thunderx2-pmu.rst        |  20 +-
 drivers/perf/thunderx2_pmu.c                  | 267 +++++++++++++++---
 2 files changed, 245 insertions(+), 42 deletions(-)

Comments

Ganapatrao Kulkarni Oct. 13, 2019, 5:15 p.m. UTC | #1
Hi Will, Mark,

On Fri, Aug 30, 2019 at 6:24 PM ganapat <gklkml16@gmail.com> wrote:
>
> From: Ganapatrao Kulkarni <gkulkarni@marvell.com>
>
> Add Cavium Coherent Processor Interconnect (CCPI2) PMU
> support in ThunderX2 Uncore driver.
>
> v5:
>         Fixed minor bug of v4 (timer callback fuction
>         was getting initialized to NULL for all PMUs).
>
> v4:
>         Update with review comments [2].
>         Changed Counter read to 2 word read since single dword read is misbhehaving(hw issue).
>
> [2] https://lkml.org/lkml/2019/7/23/231
>
> v3: Rebased to 5.3-rc1
>
> v2: Updated with review comments [1]
>
> [1] https://lkml.org/lkml/2019/6/14/965
>
> v1: initial patch
>
>
> Ganapatrao Kulkarni (2):
>   Documentation: perf: Update documentation for ThunderX2 PMU uncore
>     driver
>   drivers/perf: Add CCPI2 PMU support in ThunderX2 UNCORE driver.
>
>  .../admin-guide/perf/thunderx2-pmu.rst        |  20 +-
>  drivers/perf/thunderx2_pmu.c                  | 267 +++++++++++++++---
>  2 files changed, 245 insertions(+), 42 deletions(-)
>
> --
> 2.17.1
>

Any comments on this patchset?

Thanks,
Ganapat
Ganapatrao Kulkarni Oct. 15, 2019, 9:36 a.m. UTC | #2
Hi Will,

On Sun, Oct 13, 2019 at 10:45 PM Ganapatrao Kulkarni <gklkml16@gmail.com> wrote:
>
> Hi Will, Mark,
>
> On Fri, Aug 30, 2019 at 6:24 PM ganapat <gklkml16@gmail.com> wrote:
> >
> > From: Ganapatrao Kulkarni <gkulkarni@marvell.com>
> >
> > Add Cavium Coherent Processor Interconnect (CCPI2) PMU
> > support in ThunderX2 Uncore driver.
> >
> > v5:
> >         Fixed minor bug of v4 (timer callback fuction
> >         was getting initialized to NULL for all PMUs).
> >
> > v4:
> >         Update with review comments [2].
> >         Changed Counter read to 2 word read since single dword read is misbhehaving(hw issue).
> >
> > [2] https://lkml.org/lkml/2019/7/23/231
> >
> > v3: Rebased to 5.3-rc1
> >
> > v2: Updated with review comments [1]
> >
> > [1] https://lkml.org/lkml/2019/6/14/965
> >
> > v1: initial patch
> >
> >
> > Ganapatrao Kulkarni (2):
> >   Documentation: perf: Update documentation for ThunderX2 PMU uncore
> >     driver
> >   drivers/perf: Add CCPI2 PMU support in ThunderX2 UNCORE driver.
> >
> >  .../admin-guide/perf/thunderx2-pmu.rst        |  20 +-
> >  drivers/perf/thunderx2_pmu.c                  | 267 +++++++++++++++---
> >  2 files changed, 245 insertions(+), 42 deletions(-)
> >
> > --
> > 2.17.1
> >
>
> Any comments on this patchset?

If no further comments, can you please queue it to next?

Thanks,
Ganapat

>
> Thanks,
> Ganapat