Hi Joe, On Thu, Jul 14, 2016 at 10:47 AM, Joe Perches <joe@xxxxxxxxxxx> wrote: > On Thu, 2016-07-14 at 10:27 -0700, Tai Nguyen wrote: >> This patch adds a driver for the SoC-wide (AKA uncore) PMU hardware >> found in APM X-Gene SoCs. > > trivia: > >> diff --git a/drivers/perf/xgene_pmu.c b/drivers/perf/xgene_pmu.c > [] >> +struct xgene_pmu_dev_ctx { >> + char *name; >> + struct list_head next; >> + struct xgene_pmu_dev *pmu_dev; >> + struct hw_pmu_info inf; >> +}; > > Probably better to use something like > char name[20]; > as the kasprintf can fail and this doesn't > seem to be freed anywhere. Okay. I'll fix it shortly. [...] Thanks, -- Tai -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html