Message ID | 20230829135405.1159449-1-anshuman.khandual@arm.com (mailing list archive) |
---|---|
Headers | show |
Series | coresight: trbe: Enable ACPI based devices | expand |
Hi Will On 29/08/2023 14:54, Anshuman Khandual wrote: > These are remaining coresight patches after fixing the merge conflict which > applies on coresight/next coresight-next-v6.6. > > Changes in V6: > > - Fixed te merge conflict > Please could you confirm if the commits [0] in your for-next/perf branch are stable ? Accordingly I could merge these patches on top of your commits and send them to Greg. [0] https://git.kernel.org/will/c/1aa3d0274a4a Suzuki > Changes in V5: > > https://lore.kernel.org/all/20230817055405.249630-1-anshuman.khandual@arm.com/ > > - Detected zeroed parsed GSI as a mismatch but handled all zero scenario > - Changed condition check from 'if (ret < 0)' into a 'if (ret)' > - Dropped pr_warn() message after platform_device_register() > > Changes in V4: > > https://lore.kernel.org/all/20230808082247.383405-1-anshuman.khandual@arm.com/ > > - Added in-code comment for arm_trbe_device_probe() > - Reverted back using IS_ENABLED() for SPE PMU platform device > - Replaced #ifdef with IS_ENABLED() for TRBE platform device > - Protected arm_trbe_acpi_match with ACPI_PTR() - preventing a build failure > when CONFIG_ACPI is not enabled > - Added __maybe_unused for arm_acpi_register_pmu_device() and dropped config > checks with IS_ENABLED() > > Changes in V3: > > https://lore.kernel.org/all/20230803055652.1322801-1-anshuman.khandual@arm.com/ > > - Changed ARMV8_TRBE_PDEV_NAME from "arm-trbe-acpi" into "arm,trbe" > - Dropped local variable 'matched' > - Replaced 'matched' with 'valid gsi' as being already matched once > - Moved find_acpi_cpu_topology_hetero_id() outside conditional check > > Changes in V2: > > https://lore.kernel.org/all/20230801094052.750416-1-anshuman.khandual@arm.com/ > > - Refactored arm_spe_acpi_register_device() in a separate patch > - Renamed trbe_acpi_resources as trbe_resources > - Renamed trbe_acpi_dev as trbe_dev > > Changes in V1: > > https://lore.kernel.org/all/20230728112733.359620-1-anshuman.khandual@arm.com/ > > Cc: Sami Mujawar <sami.mujawar@arm.com> > Cc: Catalin Marinas <catalin.marinas@arm.com> > Cc: Will Deacon <will@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: Suzuki K Poulose <suzuki.poulose@arm.com> > Cc: Mike Leach <mike.leach@linaro.org> > Cc: Leo Yan <leo.yan@linaro.org> > Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> > Cc: James Clark <james.clark@arm.com> > Cc: coresight@lists.linaro.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > > Anshuman Khandual (2): > coresight: trbe: Add a representative coresight_platform_data for TRBE > coresight: trbe: Enable ACPI based TRBE devices > > drivers/hwtracing/coresight/coresight-trbe.c | 23 ++++++++++++++++++-- > drivers/hwtracing/coresight/coresight-trbe.h | 2 ++ > 2 files changed, 23 insertions(+), 2 deletions(-) >
On 8/30/23 14:32, Suzuki K Poulose wrote: > Hi Will > > On 29/08/2023 14:54, Anshuman Khandual wrote: >> These are remaining coresight patches after fixing the merge conflict which >> applies on coresight/next coresight-next-v6.6. >> >> Changes in V6: >> >> - Fixed te merge conflict >> > > > Please could you confirm if the commits [0] in your for-next/perf branch > are stable ? Accordingly I could merge these patches on top of your > commits and send them to Greg. > > [0] https://git.kernel.org/will/c/1aa3d0274a4a Both these patches are already in the mainline kernel. commit 1aa3d0274a4aac338ee45a3dfc3b17c944bcc2bc Author: Anshuman Khandual <anshuman.khandual@arm.com> Date: Thu Aug 17 11:24:03 2023 +0530 arm_pmu: acpi: Add a representative platform device for TRBE ACPI TRBE does not have a HID for identification which could create and add a platform device into the platform bus. Also without a platform device, it cannot be probed and bound to a platform driver. This creates a dummy platform device for TRBE after ascertaining that ACPI provides required interrupts uniformly across all cpus on the system. This device gets created inside drivers/perf/arm_pmu_acpi.c to accommodate TRBE being built as a module. Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com> Link: https://lore.kernel.org/r/20230817055405.249630-3-anshuman.khandual@arm.com Signed-off-by: Will Deacon <will@kernel.org> commit 81e5ee471609848ee1ebf3beb2a46788113fe0eb Author: Anshuman Khandual <anshuman.khandual@arm.com> Date: Thu Aug 17 11:24:02 2023 +0530 arm_pmu: acpi: Refactor arm_spe_acpi_register_device() Sanity checking all the GICC tables for same interrupt number, and ensuring a homogeneous ACPI based machine, could be used for other platform devices as well. Hence this refactors arm_spe_acpi_register_device() into a common helper arm_acpi_register_pmu_device(). Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <will@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Co-developed-by: Will Deacon <will@kernel.org> Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com> Link: https://lore.kernel.org/r/20230817055405.249630-2-anshuman.khandual@arm.com Signed-off-by: Will Deacon <will@kernel.org>