Re: [PATCH V5 00/11] coresight: Move remaining AMBA ACPI devices into platform driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 22/02/2024 08:21, Anshuman Khandual wrote:
This moves remaining AMBA ACPI devices into respective platform drivers for
enabling ACPI based power management support. This series applies on kernel
v6.8-rc5 release. This series has been built, and boot tested on a DT based
(RB5) and ACPI supported coresight platform (N1SDP).

Please rebase your series on coresight next and fix build failures with the extra warnings turned ON (should be on by default with next).


Suzuki


https://git.gitlab.arm.com/linux-arm/linux-anshuman.git (amba_other_acpi_migration_v5)

Cc: Lorenzo Pieralisi <lpieralisi@xxxxxxxxxx>
Cc: Sudeep Holla <sudeep.holla@xxxxxxx>
Cc: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
Cc: Mike Leach <mike.leach@xxxxxxxxxx>
Cc: James Clark <james.clark@xxxxxxx>
Cc: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>
Cc: Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>
Cc: linux-acpi@xxxxxxxxxxxxxxx
Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
Cc: linux-kernel@xxxxxxxxxxxxxxx
Cc: coresight@xxxxxxxxxxxxxxxx
Cc: linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx

Changes in V5:

- Used table->mask to filter out bits from pid in coresight_get_uci_data_from_amba()
- Dropped custom masks such as STM_AMBA_MASK and TMC_AMBA_MASK
- Modified tmc_etr_setup_caps() to accept struct csdev_access argument
- Reverted back tmc_etr_setup_caps() call site position in tmc_probe()
- Changed replicator and funnel devices to use the new helpers earlier in series
- Updated the commit messages regarding xxx_probe() refactoring and renaming

Changes in V4:

https://lore.kernel.org/all/20240123054608.1790189-1-anshuman.khandual@xxxxxxx/

- Fixed PM imbalance in etm4_probe() error path with pm_runtime_disable()
- Restored back the pm_runtime_disable() on platform probe error paths
   in replicator, funnel, catu, tpiu, tmc and stm devices
- Dropped dev_caps argument from __tmc_probe()
- Changed xxxx_platform_remove() for platform_driver->remove_new() callback

Changes in V3:

https://lore.kernel.org/all/20231208053939.42901-1-anshuman.khandual@xxxxxxx/

- Split coresight_init_driver/remove_driver() helpers into a separate patch
- Added 'drvdata->pclk' comments in replicator, funnel, tpiu, tmc, and stm devices
- Updated funnel, and replicator drivers to use these new helpers
- Check for drvdata instead of drvdata->pclk in suspend and resume paths in catu,
   tmc and debug devices
- Added patch to extract device name from AMBA pid based table lookup for stm
- Added patch to extract device properties from AMBA pid based table look for tmc
- Dropped pm_runtime_put() from common __probe() functions
- Handled pm_runtime_put() in AMBA driver in success path
- Handled pm_runtime_put() in platform driver in both success and error paths

Changes in V2:

https://lore.kernel.org/all/20231201062053.1268492-1-anshuman.khandual@xxxxxxx/

- Dropped redundant devm_ioremap_resource() hunk from tmc_platform_probe()
- Defined coresight_[init|remove]_driver() for both AMBA/platform drivers
- Changed catu, tmc, tpiu, stm and debug coresight drivers to use the new
   helpers avoiding build issues arising from module_amba_driver(), and
   module_platform_driver() being on the same file

Changes in V1:

https://lore.kernel.org/all/20231027072943.3418997-1-anshuman.khandual@xxxxxxx/

- Replaced all IS_ERR() instances with IS_ERR_OR_NULL() as per Suzuki

Changes in RFC:

https://lore.kernel.org/all/20230921042040.1334641-1-anshuman.khandual@xxxxxxx/

Anshuman Khandual (11):
   coresight: etm4x: Fix unbalanced pm_runtime_enable()
   coresight: stm: Extract device name from AMBA pid based table lookup
   coresight: tmc: Extract device properties from AMBA pid based table lookup
   coresight: Add helpers registering/removing both AMBA and platform drivers
   coresight: replicator: Move ACPI support from AMBA driver to platform driver
   coresight: funnel: Move ACPI support from AMBA driver to platform driver
   coresight: catu: Move ACPI support from AMBA driver to platform driver
   coresight: tpiu: Move ACPI support from AMBA driver to platform driver
   coresight: tmc: Move ACPI support from AMBA driver to platform driver
   coresight: stm: Move ACPI support from AMBA driver to platform driver
   coresight: debug: Move ACPI support from AMBA driver to platform driver

  drivers/acpi/arm64/amba.c                     |   8 -
  drivers/hwtracing/coresight/coresight-catu.c  | 142 +++++++++++++---
  drivers/hwtracing/coresight/coresight-catu.h  |   1 +
  drivers/hwtracing/coresight/coresight-core.c  |  29 ++++
  .../hwtracing/coresight/coresight-cpu-debug.c | 141 ++++++++++++++--
  .../coresight/coresight-etm4x-core.c          |   3 +
  .../hwtracing/coresight/coresight-funnel.c    |  86 +++++-----
  drivers/hwtracing/coresight/coresight-priv.h  |  10 ++
  .../coresight/coresight-replicator.c          |  81 +++++-----
  drivers/hwtracing/coresight/coresight-stm.c   | 115 +++++++++++--
  .../hwtracing/coresight/coresight-tmc-core.c  | 153 +++++++++++++++---
  drivers/hwtracing/coresight/coresight-tmc.h   |   2 +
  drivers/hwtracing/coresight/coresight-tpiu.c  | 102 ++++++++++--
  include/linux/coresight.h                     |   7 +
  14 files changed, 721 insertions(+), 159 deletions(-)






[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux