Re: [PATCH 2/4] perf vendor events arm64: Add common and uarch event JSON

On 28/01/2021 05:30, nakamura.shun@fujitsu.com wrote:

Hi ,john

I would like to make a comment because there is excess or deficiency in comma.
There are no differences in pmu-events.c.

quoted

+    {
+        "PublicDescription": "Attributable Bus access",
+        "EventCode": "0x19",
+        "EventName": "BUS_ACCESS",
+        "BriefDescription": "Attributable Bus access",
+    },

Comma is not required at the end of BriefDescription.

quoted

+    {
+        "PublicDescription": "Bus cycle"
+        "EventCode": "0x1D",
+        "EventName": "BUS_CYCLES",
+        "BriefDescription": "Bus cycle"
+    },

Missing comma at end of PublicDescription.

quoted

+    {
+        "PublicDescription": "Attributable Last level cache memory read"
+        "EventCode": "0x36",
+        "EventName": "LL_CACHE_RD",
+        "BriefDescription": "Attributable Last level cache memory read"
+    },
+    {
+        "PublicDescription": "Last level cache miss, read"
+        "EventCode": "0x37",
+        "EventName": "LL_CACHE_MISS_RD",
+        "BriefDescription": "Last level cache miss, read"
+    },

Missing comma at end of PublicDescription.

quoted

+    },
+]

Commas are not required.

Best Regards
Shunsuke

Fine, I can fix those up and re-post today.

Thanks,
john

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel