From: Mike Leach <mike.leach@arm.com>
To: Yeoreum Yun <yeoreum.yun@arm.com>,
James Clark <james.clark@linaro.org>, Leo Yan <leo.yan@arm.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-rt-devel@lists.linux.dev
Cc: Suzuki K Poulose <suzuki.poulose@arm.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
Clark Williams <clrkwllms@kernel.org>,
Steven Rostedt <rostedt@goodmis.org>,
nd@arm.com
Subject: Re: [PATCH v12 13/14] coresight: etm3x: rename local config as curr_config referring drvdata->curr_config
Date: Tue, 29 Sep 2026 11:55:27 +0100 [thread overview]
Message-ID: <1550cc8d-d994-44ae-a93d-28f06416db84@arm.com> (raw)
In-Reply-To: <20260924-separate_etm_cfg_v2-v12-13-a2b41d2f7c2b@arm.com>
Reviewed-by: Mike Leach <mike.leach@arm.com>
On 9/24/26 18:33, Yeoreum Yun wrote:
> For readability, rename the local config variable to curr_config in
> places where it refers to drvdata->curr_config.
>
> Signed-off-by: Yeoreum Yun <yeoreum.yun@arm.com>
> ---
> drivers/hwtracing/coresight/coresight-etm3x-core.c | 76 +++++++++++-----------
> 1 file changed, 38 insertions(+), 38 deletions(-)
>
> diff --git a/drivers/hwtracing/coresight/coresight-etm3x-core.c b/drivers/hwtracing/coresight/coresight-etm3x-core.c
> index ca2ba179b6fb..0cad9ecb49b6 100644
> --- a/drivers/hwtracing/coresight/coresight-etm3x-core.c
> +++ b/drivers/hwtracing/coresight/coresight-etm3x-core.c
> @@ -309,7 +309,7 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata,
> struct perf_event *event)
> {
> const struct etm_caps *caps = &drvdata->caps;
> - struct etm_config *config = &drvdata->curr_config;
> + struct etm_config *curr_config = &drvdata->curr_config;
> struct perf_event_attr *attr = &event->attr;
> u8 ts_level;
>
> @@ -317,28 +317,28 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata,
> return -EINVAL;
>
> /* Clear configuration from previous run */
> - memset(config, 0, sizeof(struct etm_config));
> + memset(curr_config, 0, sizeof(struct etm_config));
>
> if (attr->exclude_kernel)
> - config->mode = ETM_MODE_EXCL_KERN;
> + curr_config->mode = ETM_MODE_EXCL_KERN;
>
> if (attr->exclude_user)
> - config->mode = ETM_MODE_EXCL_USER;
> + curr_config->mode = ETM_MODE_EXCL_USER;
>
> /* Always start from the default config */
> - etm_set_default(config);
> + etm_set_default(curr_config);
>
> /*
> * By default the tracers are configured to trace the whole address
> * range. Narrow the field only if requested by user space.
> */
> - if (config->mode)
> - etm_config_trace_mode(config);
> + if (curr_config->mode)
> + etm_config_trace_mode(curr_config);
>
> - config->ctrl = 0;
> + curr_config->ctrl = 0;
>
> if (ATTR_CFG_GET_FLD(attr, cycacc))
> - config->ctrl |= ETMCR_CYC_ACC;
> + curr_config->ctrl |= ETMCR_CYC_ACC;
>
> ts_level = max(ATTR_CFG_GET_FLD(attr, timestamp),
> ATTR_CFG_GET_FLD(attr, deprecated_timestamp));
> @@ -350,7 +350,7 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata,
> }
>
> if (ts_level)
> - config->ctrl |= ETMCR_TIMESTAMP_EN;
> + curr_config->ctrl |= ETMCR_TIMESTAMP_EN;
>
> /*
> * Possible to have cores with PTM (supports ret stack) and ETM (never
> @@ -358,7 +358,7 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata,
> * - trace will still continue normally otherwise.
> */
> if (ATTR_CFG_GET_FLD(attr, retstack) && (caps->retstack))
> - config->ctrl |= ETMCR_RETURN_STACK;
> + curr_config->ctrl |= ETMCR_RETURN_STACK;
>
> return 0;
> }
> @@ -368,7 +368,7 @@ static int etm_enable_hw(struct etm_drvdata *drvdata)
> int i, rc;
> u32 etmcr;
> const struct etm_caps *caps = &drvdata->caps;
> - struct etm_config *config = &drvdata->curr_config;
> + struct etm_config *curr_config = &drvdata->curr_config;
> struct coresight_device *csdev = drvdata->csdev;
>
> CS_UNLOCK(drvdata->csa.base);
> @@ -391,39 +391,39 @@ static int etm_enable_hw(struct etm_drvdata *drvdata)
> etmcr &= ~ETM3X_SUPPORTED_OPTIONS;
> etmcr |= caps->port_size;
> etmcr |= ETMCR_ETM_EN;
> - etm_writel(drvdata, config->ctrl | etmcr, ETMCR);
> - etm_writel(drvdata, config->trigger_event, ETMTRIGGER);
> - etm_writel(drvdata, config->startstop_ctrl, ETMTSSCR);
> - etm_writel(drvdata, config->enable_event, ETMTEEVR);
> - etm_writel(drvdata, config->enable_ctrl1, ETMTECR1);
> - etm_writel(drvdata, config->fifofull_level, ETMFFLR);
> + etm_writel(drvdata, curr_config->ctrl | etmcr, ETMCR);
> + etm_writel(drvdata, curr_config->trigger_event, ETMTRIGGER);
> + etm_writel(drvdata, curr_config->startstop_ctrl, ETMTSSCR);
> + etm_writel(drvdata, curr_config->enable_event, ETMTEEVR);
> + etm_writel(drvdata, curr_config->enable_ctrl1, ETMTECR1);
> + etm_writel(drvdata, curr_config->fifofull_level, ETMFFLR);
> for (i = 0; i < caps->nr_addr_cmp; i++) {
> - etm_writel(drvdata, config->addr_val[i], ETMACVRn(i));
> - etm_writel(drvdata, config->addr_acctype[i], ETMACTRn(i));
> + etm_writel(drvdata, curr_config->addr_val[i], ETMACVRn(i));
> + etm_writel(drvdata, curr_config->addr_acctype[i], ETMACTRn(i));
> }
> for (i = 0; i < caps->nr_cntr; i++) {
> - etm_writel(drvdata, config->cntr_rld_val[i], ETMCNTRLDVRn(i));
> - etm_writel(drvdata, config->cntr_event[i], ETMCNTENRn(i));
> - etm_writel(drvdata, config->cntr_rld_event[i],
> + etm_writel(drvdata, curr_config->cntr_rld_val[i], ETMCNTRLDVRn(i));
> + etm_writel(drvdata, curr_config->cntr_event[i], ETMCNTENRn(i));
> + etm_writel(drvdata, curr_config->cntr_rld_event[i],
> ETMCNTRLDEVRn(i));
> - etm_writel(drvdata, config->cntr_val[i], ETMCNTVRn(i));
> + etm_writel(drvdata, curr_config->cntr_val[i], ETMCNTVRn(i));
> }
> - etm_writel(drvdata, config->seq_12_event, ETMSQ12EVR);
> - etm_writel(drvdata, config->seq_21_event, ETMSQ21EVR);
> - etm_writel(drvdata, config->seq_23_event, ETMSQ23EVR);
> - etm_writel(drvdata, config->seq_31_event, ETMSQ31EVR);
> - etm_writel(drvdata, config->seq_32_event, ETMSQ32EVR);
> - etm_writel(drvdata, config->seq_13_event, ETMSQ13EVR);
> - etm_writel(drvdata, config->seq_curr_state, ETMSQR);
> + etm_writel(drvdata, curr_config->seq_12_event, ETMSQ12EVR);
> + etm_writel(drvdata, curr_config->seq_21_event, ETMSQ21EVR);
> + etm_writel(drvdata, curr_config->seq_23_event, ETMSQ23EVR);
> + etm_writel(drvdata, curr_config->seq_31_event, ETMSQ31EVR);
> + etm_writel(drvdata, curr_config->seq_32_event, ETMSQ32EVR);
> + etm_writel(drvdata, curr_config->seq_13_event, ETMSQ13EVR);
> + etm_writel(drvdata, curr_config->seq_curr_state, ETMSQR);
> for (i = 0; i < caps->nr_ext_out; i++)
> etm_writel(drvdata, ETM_DEFAULT_EVENT_VAL, ETMEXTOUTEVRn(i));
> for (i = 0; i < caps->nr_ctxid_cmp; i++)
> - etm_writel(drvdata, config->ctxid_pid[i], ETMCIDCVRn(i));
> - etm_writel(drvdata, config->ctxid_mask, ETMCIDCMR);
> - etm_writel(drvdata, config->sync_freq, ETMSYNCFR);
> + etm_writel(drvdata, curr_config->ctxid_pid[i], ETMCIDCVRn(i));
> + etm_writel(drvdata, curr_config->ctxid_mask, ETMCIDCMR);
> + etm_writel(drvdata, curr_config->sync_freq, ETMSYNCFR);
> /* No external input selected */
> etm_writel(drvdata, 0x0, ETMEXTINSELR);
> - etm_writel(drvdata, config->timestamp_event, ETMTSEVR);
> + etm_writel(drvdata, curr_config->timestamp_event, ETMTSEVR);
> /* No auxiliary control selected */
> etm_writel(drvdata, 0x0, ETMAUXCR);
> etm_writel(drvdata, drvdata->traceid, ETMTRACEIDR);
> @@ -562,17 +562,17 @@ static void etm_disable_hw(struct etm_drvdata *drvdata)
> {
> int i;
> const struct etm_caps *caps = &drvdata->caps;
> - struct etm_config *config = &drvdata->curr_config;
> + struct etm_config *curr_config = &drvdata->curr_config;
> struct coresight_device *csdev = drvdata->csdev;
>
> CS_UNLOCK(drvdata->csa.base);
> etm_set_prog(drvdata);
>
> /* Read back sequencer and counters for post trace analysis */
> - config->seq_curr_state = (etm_readl(drvdata, ETMSQR) & ETM_SQR_MASK);
> + curr_config->seq_curr_state = (etm_readl(drvdata, ETMSQR) & ETM_SQR_MASK);
>
> for (i = 0; i < caps->nr_cntr; i++)
> - config->cntr_val[i] = etm_readl(drvdata, ETMCNTVRn(i));
> + curr_config->cntr_val[i] = etm_readl(drvdata, ETMCNTVRn(i));
>
> etm_set_pwrdwn(drvdata);
> coresight_disclaim_device_unlocked(csdev);
>
next prev parent reply other threads:[~2026-09-29 10:56 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-24 17:33 [PATCH v12 00/14] fix several inconsistencies with sysfs configuration in etmX Yeoreum Yun
2026-09-24 17:33 ` [PATCH v12 01/14] coresight: etm4x: read-back TRCSEQSTR at disabling and prohibit modifying seq_state while enabling Yeoreum Yun
2026-09-24 17:46 ` sashiko-bot
2026-09-29 10:40 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 02/14] coresight: etm4x: prohibit modifying cntr_val while session is enabled Yeoreum Yun
2026-09-29 10:40 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 03/14] coresight: etm3x: prohibit modifying cntr_val and reset " Yeoreum Yun
2026-09-24 17:58 ` sashiko-bot
2026-09-24 17:33 ` [PATCH v12 04/14] coresight: etm4x: fix inconsistencies with sysfs configuration Yeoreum Yun
2026-09-29 10:48 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 05/14] coresight: etm3x: " Yeoreum Yun
2026-09-29 10:49 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 06/14] coresight: etm3x: remove redundant cpu online check on etm_enable_sysfs() Yeoreum Yun
2026-09-24 17:33 ` [PATCH v12 07/14] coresight: etm4x: introduce struct etm4_caps Yeoreum Yun
2026-09-24 17:33 ` [PATCH v12 08/14] coresight: etm4x: exclude ss_status from drvdata->config Yeoreum Yun
2026-09-24 18:21 ` sashiko-bot
2026-09-29 10:53 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 09/14] coresight: etm4x: remove s_ex_level from config Yeoreum Yun
2026-09-24 17:33 ` [PATCH v12 10/14] coresight: etm4x: rename local config as curr_config referring drvdata->curr_config Yeoreum Yun
2026-09-29 10:54 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 11/14] coresight: etm4x: rename drvdata->config to sysfs_config Yeoreum Yun
2026-09-29 10:55 ` Mike Leach
2026-09-24 17:33 ` [PATCH v12 12/14] coresight: etm3x: introduce struct etm_caps Yeoreum Yun
2026-09-24 17:33 ` [PATCH v12 13/14] coresight: etm3x: rename local config as curr_config referring drvdata->curr_config Yeoreum Yun
2026-09-29 10:55 ` Mike Leach [this message]
2026-09-24 17:33 ` [PATCH v12 14/14] coresight: etm3x: rename drvdata->config to sysfs_config Yeoreum Yun
2026-09-24 18:36 ` sashiko-bot
2026-09-29 10:55 ` Mike Leach
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=1550cc8d-d994-44ae-a93d-28f06416db84@arm.com \
--to=mike.leach@arm.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=bigeasy@linutronix.de \
--cc=clrkwllms@kernel.org \
--cc=coresight@lists.linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=james.clark@linaro.org \
--cc=leo.yan@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rt-devel@lists.linux.dev \
--cc=mathieu.poirier@linaro.org \
--cc=nd@arm.com \
--cc=rostedt@goodmis.org \
--cc=suzuki.poulose@arm.com \
--cc=yeoreum.yun@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
all inboxes | Powered by JetHome®