On 4/19/21 2:38 AM, Paul A. Clarke wrote:

On Sat, Apr 17, 2021 at 02:48:50PM +0530, Kajol Jain wrote:

quoted

Patch adds initial json/events for POWER10.

I was able to apply, build, and run perf with these changes,
and every new event at least ran successfully with
`perf stat`.

Pedantically, there is a lot of inconsistency as to whether
the `BriefDescription` ends with a period or not, and whether
there is an extra space at the end.

Hi Paul,
Thanks for reviewing the patch. Sure I will remove this inconsistency
and send v2 patch for the same, with your Tested-by and Reviewed-by tag.

Thanks,
Kajol Jain

Regardless, LGTM.

Tested-by: Paul A. Clarke <redacted>
Reviewed-by: Paul A. Clarke <redacted>

PC