Re: [PATCH 02/32] perf/hisi-pcie: Assign parent for event_source device

From: Yicong Yang
Date: Wed Apr 05 2023 - 23:56:26 EST


On 2023/4/4 21:41, Jonathan Cameron wrote:
> Currently the PMU device appears directly under /sys/devices/
> Only root busses should appear there, so instead assign the pmu->dev
> parent to be the PCI device.
>
> Link: https://lore.kernel.org/linux-cxl/ZCLI9A40PJsyqAmq@xxxxxxxxx/
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>

Reviewed-by: Yicong Yang <yangyicong@xxxxxxxxxxxxx>

> ---
> drivers/perf/hisilicon/hisi_pcie_pmu.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/perf/hisilicon/hisi_pcie_pmu.c b/drivers/perf/hisilicon/hisi_pcie_pmu.c
> index 6fee0b6e163b..2cc88d75b895 100644
> --- a/drivers/perf/hisilicon/hisi_pcie_pmu.c
> +++ b/drivers/perf/hisilicon/hisi_pcie_pmu.c
> @@ -793,6 +793,7 @@ static int hisi_pcie_alloc_pmu(struct pci_dev *pdev, struct hisi_pcie_pmu *pcie_
> pcie_pmu->pmu = (struct pmu) {
> .name = name,
> .module = THIS_MODULE,
> + .parent = &pdev->dev,
> .event_init = hisi_pcie_pmu_event_init,
> .pmu_enable = hisi_pcie_pmu_enable,
> .pmu_disable = hisi_pcie_pmu_disable,
>