From: James Clark <james.clark@arm.com>
To: Anshuman Khandual <anshuman.khandual@arm.com>,
Suzuki K Poulose <suzuki.poulose@arm.com>
Cc: Lorenzo Pieralisi <lpieralisi@kernel.org>,
Sudeep Holla <sudeep.holla@arm.com>,
Mike Leach <mike.leach@linaro.org>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
coresight@lists.linaro.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH V5 00/11] coresight: Move remaining AMBA ACPI devices into platform driver
Date: Mon, 11 Mar 2024 09:25:02 +0000 [thread overview]
Message-ID: <828d2109-7413-ffe5-da6a-56f15ed2f562@arm.com> (raw)
In-Reply-To: <379bf6df-3568-43c0-9a68-4a5693bf5ccc@arm.com>
On 11/03/2024 06:04, Anshuman Khandual wrote:
>
>
> On 3/5/24 23:55, Suzuki K Poulose wrote:
>> 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).
>
> I did rebase the series (which required some rebase related changes for some) on
> coresight next i.e with the following commit as HEAD.
>
> a4f3057d19ff ("coresight-tpda: Change qcom,dsb-element-size to qcom,dsb-elem-bits")
>
> Although did not see any warning either with = m or = y based coresight options.
> Is there any other particular config which needs to be enabled for these warnings
> to come up ?
>
It doesn't apply cleanly on a4f305 for me, maybe you sent an old version
after rebasing?
This change in patch 5 is a warning for example, because not all members
of the struct are initialised. No special config is required:
+ {"ARMHC98D", 0}, /* ARM CoreSight Dynamic Replicator */"
>>
>>
>> Suzuki
>>
>>>
>>> https://git.gitlab.arm.com/linux-arm/linux-anshuman.git (amba_other_acpi_migration_v5)
>>>
>>> Cc: Lorenzo Pieralisi <lpieralisi@kernel.org>
>>> Cc: Sudeep Holla <sudeep.holla@arm.com>
>>> Cc: Suzuki K Poulose <suzuki.poulose@arm.com>
>>> Cc: Mike Leach <mike.leach@linaro.org>
>>> Cc: James Clark <james.clark@arm.com>
>>> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
>>> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
>>> Cc: linux-acpi@vger.kernel.org
>>> Cc: linux-arm-kernel@lists.infradead.org
>>> Cc: linux-kernel@vger.kernel.org
>>> Cc: coresight@lists.linaro.org
>>> Cc: linux-stm32@st-md-mailman.stormreply.com
>>>
>>> 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@arm.com/
>>>
>>> - 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@arm.com/
>>>
>>> - 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@arm.com/
>>>
>>> - 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@arm.com/
>>>
>>> - 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@arm.com/
>>>
>>> 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(-)
>>>
>>
next prev parent reply other threads:[~2024-03-11 9:25 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-22 8:21 [PATCH V5 00/11] coresight: Move remaining AMBA ACPI devices into platform driver Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 01/11] coresight: etm4x: Fix unbalanced pm_runtime_enable() Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 02/11] coresight: stm: Extract device name from AMBA pid based table lookup Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 03/11] coresight: tmc: Extract device properties " Anshuman Khandual
2024-03-05 14:36 ` Suzuki K Poulose
2024-03-06 3:54 ` Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 04/11] coresight: Add helpers registering/removing both AMBA and platform drivers Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 05/11] coresight: replicator: Move ACPI support from AMBA driver to platform driver Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 06/11] coresight: funnel: " Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 07/11] coresight: catu: " Anshuman Khandual
2024-03-05 17:32 ` Suzuki K Poulose
2024-03-06 6:14 ` Anshuman Khandual
2024-03-06 17:21 ` Suzuki K Poulose
2024-03-08 4:25 ` Anshuman Khandual
2024-03-08 10:29 ` Suzuki K Poulose
2024-02-22 8:21 ` [PATCH V5 08/11] coresight: tpiu: " Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 09/11] coresight: tmc: " Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 10/11] coresight: stm: " Anshuman Khandual
2024-02-22 8:21 ` [PATCH V5 11/11] coresight: debug: " Anshuman Khandual
2024-03-05 18:25 ` [PATCH V5 00/11] coresight: Move remaining AMBA ACPI devices into " Suzuki K Poulose
2024-03-11 6:04 ` Anshuman Khandual
2024-03-11 9:25 ` James Clark [this message]
2024-03-11 9:33 ` Anshuman Khandual
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=828d2109-7413-ffe5-da6a-56f15ed2f562@arm.com \
--to=james.clark@arm.com \
--cc=alexandre.torgue@foss.st.com \
--cc=anshuman.khandual@arm.com \
--cc=coresight@lists.linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=lpieralisi@kernel.org \
--cc=mcoquelin.stm32@gmail.com \
--cc=mike.leach@linaro.org \
--cc=sudeep.holla@arm.com \
--cc=suzuki.poulose@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).