From: Mathieu Poirier <mathieu.poirier@linaro.org>
To: Suzuki K Poulose <suzuki.poulose@arm.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, robert.walker@arm.com,
mike.leach@arm.com
Subject: Re: [PATCH 09/13] coresight: etmx: Claim devices before use
Date: Tue, 14 Aug 2018 17:43:02 -0600 [thread overview]
Message-ID: <20180814234302.GA8196@xps15> (raw)
In-Reply-To: <1533562915-21558-10-git-send-email-suzuki.poulose@arm.com>
On Mon, Aug 06, 2018 at 02:41:51PM +0100, Suzuki K Poulose wrote:
> Use the CLAIM tags to grab the device for self-hosted usage.
>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
> ---
> drivers/hwtracing/coresight/coresight-etm3x.c | 18 +++++++++++++++---
> drivers/hwtracing/coresight/coresight-etm4x.c | 15 ++++++++++++---
> 2 files changed, 27 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/hwtracing/coresight/coresight-etm3x.c b/drivers/hwtracing/coresight/coresight-etm3x.c
> index 771691c..b810bbc 100644
> --- a/drivers/hwtracing/coresight/coresight-etm3x.c
> +++ b/drivers/hwtracing/coresight/coresight-etm3x.c
> @@ -357,7 +357,7 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata,
>
> static int etm_enable_hw(struct etm_drvdata *drvdata)
> {
> - int i;
> + int i, rc;
> u32 etmcr;
> struct etm_config *config = &drvdata->config;
>
> @@ -369,6 +369,9 @@ static int etm_enable_hw(struct etm_drvdata *drvdata)
> etm_set_pwrup(drvdata);
> /* Make sure all registers are accessible */
> etm_os_unlock(drvdata);
> + rc = coresight_claim_device_unlocked(drvdata->base);
> + if (rc)
> + goto done;
>
> etm_set_prog(drvdata);
>
> @@ -417,10 +420,15 @@ static int etm_enable_hw(struct etm_drvdata *drvdata)
> etm_writel(drvdata, 0x0, ETMVMIDCVR);
>
> etm_clr_prog(drvdata);
> +
> +done:
> + if (rc)
> + etm_set_pwrdwn(drvdata);
> CS_LOCK(drvdata->base);
>
> - dev_dbg(drvdata->dev, "cpu: %d enable smp call done\n", drvdata->cpu);
> - return 0;
> + dev_dbg(drvdata->dev, "cpu: %d enable smp call done: %d\n",
> + drvdata->cpu, rc);
> + return rc;
> }
>
> struct etm_enable_arg {
> @@ -561,6 +569,8 @@ static void etm_disable_hw(void *info)
> struct etm_config *config = &drvdata->config;
>
> CS_UNLOCK(drvdata->base);
> +
> +
This is extra, please remove.
> etm_set_prog(drvdata);
>
> /* Read back sequencer and counters for post trace analysis */
> @@ -569,6 +579,8 @@ static void etm_disable_hw(void *info)
> for (i = 0; i < drvdata->nr_cntr; i++)
> config->cntr_val[i] = etm_readl(drvdata, ETMCNTVRn(i));
>
> + coresight_disclaim_device_unlocked(drvdata->base);
> +
> etm_set_pwrdwn(drvdata);
> CS_LOCK(drvdata->base);
>
> diff --git a/drivers/hwtracing/coresight/coresight-etm4x.c b/drivers/hwtracing/coresight/coresight-etm4x.c
> index 4f9e6bb..20d1015 100644
> --- a/drivers/hwtracing/coresight/coresight-etm4x.c
> +++ b/drivers/hwtracing/coresight/coresight-etm4x.c
> @@ -84,13 +84,17 @@ struct etm4_enable_arg {
>
> static int etm4_enable_hw(struct etmv4_drvdata *drvdata)
> {
> - int i;
> + int i, rc;
> struct etmv4_config *config = &drvdata->config;
>
> CS_UNLOCK(drvdata->base);
>
> etm4_os_unlock(drvdata);
>
> + rc = coresight_claim_device_unlocked(drvdata->base);
> + if (rc)
> + goto done;
> +
> /* Disable the trace unit before programming trace registers */
> writel_relaxed(0, drvdata->base + TRCPRGCTLR);
>
> @@ -178,10 +182,12 @@ static int etm4_enable_hw(struct etmv4_drvdata *drvdata)
> dev_err(drvdata->dev,
> "timeout while waiting for Idle Trace Status\n");
>
> +done:
> CS_LOCK(drvdata->base);
>
> - dev_dbg(drvdata->dev, "cpu: %d enable smp call done\n", drvdata->cpu);
> - return 0;
> + dev_dbg(drvdata->dev, "cpu: %d enable smp call done: %d\n",
> + drvdata->cpu, rc);
> + return rc;
> }
>
> static void etm4_enable_hw_smp_call(void *info)
> @@ -326,6 +332,7 @@ static void etm4_disable_hw(void *info)
>
> CS_UNLOCK(drvdata->base);
>
> +
Same here.
> /* power can be removed from the trace unit now */
> control = readl_relaxed(drvdata->base + TRCPDCR);
> control &= ~TRCPDCR_PU;
> @@ -341,6 +348,8 @@ static void etm4_disable_hw(void *info)
> isb();
> writel_relaxed(control, drvdata->base + TRCPRGCTLR);
>
> + coresight_disclaim_device_unlocked(drvdata->base);
> +
> CS_LOCK(drvdata->base);
>
> dev_dbg(drvdata->dev, "cpu: %d disable smp call done\n", drvdata->cpu);
> --
> 2.7.4
>
next prev parent reply other threads:[~2018-08-14 23:44 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-06 13:41 [PATCH 00/13] coresight: Implement device claim protocol Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 01/13] coresight: tmc-etr: Refactor for handling errors Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 02/13] coresight: tmc-etr: Handle errors enabling CATU Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 03/13] coresight: tmc-etb/etf: Prepare to handle errors enabling Suzuki K Poulose
2018-08-15 19:22 ` Mathieu Poirier
2018-08-16 14:47 ` Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 04/13] coresight: etm4x: Add support for handling errors Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 05/13] coresight: etm3: " Suzuki K Poulose
2018-08-15 19:34 ` Mathieu Poirier
2018-08-16 15:45 ` Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 06/13] coresight: etb10: Handle errors enabling the device Suzuki K Poulose
2018-08-14 17:40 ` Mathieu Poirier
2018-08-15 19:38 ` Mathieu Poirier
2018-08-16 15:46 ` Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 07/13] coresight: dynamic-replicator: Handle multiple connections Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 08/13] coresight: Add support for CLAIM tag protocol Suzuki K Poulose
2018-08-14 23:20 ` Mathieu Poirier
2018-08-16 15:47 ` Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 09/13] coresight: etmx: Claim devices before use Suzuki K Poulose
2018-08-14 23:43 ` Mathieu Poirier [this message]
2018-08-06 13:41 ` [PATCH 10/13] coresight: funnel: " Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 11/13] coresight: catu: Claim device " Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 12/13] coresight: dynamic-replicator: Claim device for use Suzuki K Poulose
2018-08-06 13:41 ` [PATCH 13/13] coreisght: tmc: Claim device before use Suzuki K Poulose
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=20180814234302.GA8196@xps15 \
--to=mathieu.poirier@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mike.leach@arm.com \
--cc=robert.walker@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).