CoreSight

coresight@lists.linaro.org

4 participants
2500 discussions

Re: [PATCH v3] coresight: Replace scnprintf() with sysfs_emit() in sysfs show functions

by Mike Leach

Hi, If this change is to be made, which is a high degree of churn for no functional difference, then it should be comprehensive. The sysfs _show() functions in coresight_ /cti_sysfs.c / tmc_core.c / trbe.c / funnel.c, which all use sprintf in the show() function should be included On Wed, 9 Jul 2025 at 18:17, Chelsy Ratnawat <chelsyratnawat2001(a)gmail.com> wrote: > > Documentation/filesystems/sysfs.rst mentions that show() should only > use sysfs_emit() or sysfs_emit_at() when formating the value to be > returned to user space. So replace scnprintf() with sysfs_emit(). > > Signed-off-by: Chelsy Ratnawat <chelsyratnawat2001(a)gmail.com> > Reviewed-by: James Clark <james.clark(a)linaro.org> > --- > Changes in v3: > - fixed warnings from checkpatch.pl > > .../hwtracing/coresight/coresight-etm-perf.c | 4 +- > .../coresight/coresight-etm3x-sysfs.c | 2 +- > .../coresight/coresight-etm4x-sysfs.c | 118 +++++++++--------- > drivers/hwtracing/coresight/coresight-stm.c | 8 +- > drivers/hwtracing/coresight/coresight-sysfs.c | 4 +- > 5 files changed, 68 insertions(+), 68 deletions(-) > > diff --git a/drivers/hwtracing/coresight/coresight-etm-perf.c b/drivers/hwtracing/coresight/coresight-etm-perf.c > index f1551c08ecb2..8cfdcbc2f8f6 100644 > --- a/drivers/hwtracing/coresight/coresight-etm-perf.c > +++ b/drivers/hwtracing/coresight/coresight-etm-perf.c > @@ -851,7 +851,7 @@ static ssize_t etm_perf_sink_name_show(struct device *dev, > struct dev_ext_attribute *ea; > > ea = container_of(dattr, struct dev_ext_attribute, attr); > - return scnprintf(buf, PAGE_SIZE, "0x%lx\n", (unsigned long)(ea->var)); > + return sysfs_emit(buf, "0x%lx\n", (unsigned long)(ea->var)); > } > > static struct dev_ext_attribute * > @@ -943,7 +943,7 @@ static ssize_t etm_perf_cscfg_event_show(struct device *dev, > struct dev_ext_attribute *ea; > > ea = container_of(dattr, struct dev_ext_attribute, attr); > - return scnprintf(buf, PAGE_SIZE, "configid=0x%lx\n", (unsigned long)(ea->var)); > + return sysfs_emit(buf, "configid=0x%lx\n", (unsigned long)(ea->var)); > } > > int etm_perf_add_symlink_cscfg(struct device *dev, struct cscfg_config_desc *config_desc) > diff --git a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > index 762109307b86..4868eb05a312 100644 > --- a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > @@ -1182,7 +1182,7 @@ static ssize_t cpu_show(struct device *dev, > struct etm_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->cpu; > - return scnprintf(buf, PAGE_SIZE, "%d\n", val); > + return sysfs_emit(buf, "%d\n", val); > > } > static DEVICE_ATTR_RO(cpu); > diff --git a/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c b/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > index ab251865b893..4ddc0d344b61 100644 > --- a/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > @@ -63,7 +63,7 @@ static ssize_t nr_pe_cmp_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_pe_cmp; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_pe_cmp); > > @@ -75,7 +75,7 @@ static ssize_t nr_addr_cmp_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_addr_cmp; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_addr_cmp); > > @@ -87,7 +87,7 @@ static ssize_t nr_cntr_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_cntr; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_cntr); > > @@ -99,7 +99,7 @@ static ssize_t nr_ext_inp_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_ext_inp; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_ext_inp); > > @@ -111,7 +111,7 @@ static ssize_t numcidc_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->numcidc; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(numcidc); > > @@ -123,7 +123,7 @@ static ssize_t numvmidc_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->numvmidc; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(numvmidc); > > @@ -135,7 +135,7 @@ static ssize_t nrseqstate_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nrseqstate; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nrseqstate); > > @@ -147,7 +147,7 @@ static ssize_t nr_resource_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_resource; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_resource); > > @@ -159,7 +159,7 @@ static ssize_t nr_ss_cmp_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->nr_ss_cmp; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(nr_ss_cmp); > > @@ -287,7 +287,7 @@ static ssize_t mode_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->mode; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t mode_store(struct device *dev, > @@ -453,7 +453,7 @@ static ssize_t pe_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->pe_sel; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t pe_store(struct device *dev, > @@ -488,7 +488,7 @@ static ssize_t event_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->eventctrl0; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t event_store(struct device *dev, > @@ -537,7 +537,7 @@ static ssize_t event_instren_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = FIELD_GET(TRCEVENTCTL1R_INSTEN_MASK, config->eventctrl1); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t event_instren_store(struct device *dev, > @@ -593,7 +593,7 @@ static ssize_t event_ts_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->ts_ctrl; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t event_ts_store(struct device *dev, > @@ -623,7 +623,7 @@ static ssize_t syncfreq_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->syncfreq; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t syncfreq_store(struct device *dev, > @@ -653,7 +653,7 @@ static ssize_t cyc_threshold_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->ccctlr; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t cyc_threshold_store(struct device *dev, > @@ -686,7 +686,7 @@ static ssize_t bb_ctrl_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->bb_ctrl; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t bb_ctrl_store(struct device *dev, > @@ -726,7 +726,7 @@ static ssize_t event_vinst_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = FIELD_GET(TRCVICTLR_EVENT_MASK, config->vinst_ctrl); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t event_vinst_store(struct device *dev, > @@ -758,7 +758,7 @@ static ssize_t s_exlevel_vinst_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = FIELD_GET(TRCVICTLR_EXLEVEL_S_MASK, config->vinst_ctrl); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t s_exlevel_vinst_store(struct device *dev, > @@ -793,7 +793,7 @@ static ssize_t ns_exlevel_vinst_show(struct device *dev, > > /* EXLEVEL_NS, bits[23:20] */ > val = FIELD_GET(TRCVICTLR_EXLEVEL_NS_MASK, config->vinst_ctrl); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t ns_exlevel_vinst_store(struct device *dev, > @@ -827,7 +827,7 @@ static ssize_t addr_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->addr_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_idx_store(struct device *dev, > @@ -866,11 +866,11 @@ static ssize_t addr_instdatatype_show(struct device *dev, > raw_spin_lock(&drvdata->spinlock); > idx = config->addr_idx; > val = FIELD_GET(TRCACATRn_TYPE_MASK, config->addr_acc[idx]); > - len = scnprintf(buf, PAGE_SIZE, "%s\n", > - val == TRCACATRn_TYPE_ADDR ? "instr" : > - (val == TRCACATRn_TYPE_DATA_LOAD_ADDR ? "data_load" : > - (val == TRCACATRn_TYPE_DATA_STORE_ADDR ? "data_store" : > - "data_load_store"))); > + len = sysfs_emit(buf, "%s\n", > + val == TRCACATRn_TYPE_ADDR ? "instr" : > + (val == TRCACATRn_TYPE_DATA_LOAD_ADDR ? "data_load" : > + (val == TRCACATRn_TYPE_DATA_STORE_ADDR ? "data_store" : > + "data_load_store"))); > raw_spin_unlock(&drvdata->spinlock); > return len; > } > @@ -918,7 +918,7 @@ static ssize_t addr_single_show(struct device *dev, > } > val = (unsigned long)config->addr_val[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_single_store(struct device *dev, > @@ -974,7 +974,7 @@ static ssize_t addr_range_show(struct device *dev, > val1 = (unsigned long)config->addr_val[idx]; > val2 = (unsigned long)config->addr_val[idx + 1]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx %#lx\n", val1, val2); > + return sysfs_emit(buf, "%#lx %#lx\n", val1, val2); > } > > static ssize_t addr_range_store(struct device *dev, > @@ -1049,7 +1049,7 @@ static ssize_t addr_start_show(struct device *dev, > > val = (unsigned long)config->addr_val[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_start_store(struct device *dev, > @@ -1104,7 +1104,7 @@ static ssize_t addr_stop_show(struct device *dev, > > val = (unsigned long)config->addr_val[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_stop_store(struct device *dev, > @@ -1152,7 +1152,7 @@ static ssize_t addr_ctxtype_show(struct device *dev, > idx = config->addr_idx; > /* CONTEXTTYPE, bits[3:2] */ > val = FIELD_GET(TRCACATRn_CONTEXTTYPE_MASK, config->addr_acc[idx]); > - len = scnprintf(buf, PAGE_SIZE, "%s\n", val == ETM_CTX_NONE ? "none" : > + len = sysfs_emit(buf, "%s\n", val == ETM_CTX_NONE ? "none" : > (val == ETM_CTX_CTXID ? "ctxid" : > (val == ETM_CTX_VMID ? "vmid" : "all"))); > raw_spin_unlock(&drvdata->spinlock); > @@ -1219,7 +1219,7 @@ static ssize_t addr_context_show(struct device *dev, > /* context ID comparator bits[6:4] */ > val = FIELD_GET(TRCACATRn_CONTEXT_MASK, config->addr_acc[idx]); > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_context_store(struct device *dev, > @@ -1262,7 +1262,7 @@ static ssize_t addr_exlevel_s_ns_show(struct device *dev, > idx = config->addr_idx; > val = FIELD_GET(TRCACATRn_EXLEVEL_MASK, config->addr_acc[idx]); > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t addr_exlevel_s_ns_store(struct device *dev, > @@ -1325,8 +1325,8 @@ static ssize_t addr_cmp_view_show(struct device *dev, > } > raw_spin_unlock(&drvdata->spinlock); > if (addr_type) { > - size = scnprintf(buf, PAGE_SIZE, "addr_cmp[%i] %s %#lx", idx, > - addr_type_names[addr_type], addr_v); > + size = sysfs_emit(buf, "addr_cmp[%i] %s %#lx", idx, > + addr_type_names[addr_type], addr_v); > if (addr_type == ETM_ADDR_TYPE_RANGE) { > size += scnprintf(buf + size, PAGE_SIZE - size, > " %#lx %s", addr_v2, > @@ -1335,7 +1335,7 @@ static ssize_t addr_cmp_view_show(struct device *dev, > size += scnprintf(buf + size, PAGE_SIZE - size, > " ctrl(%#lx)\n", addr_ctrl); > } else { > - size = scnprintf(buf, PAGE_SIZE, "addr_cmp[%i] unused\n", idx); > + size = sysfs_emit(buf, "addr_cmp[%i] unused\n", idx); > } > return size; > } > @@ -1352,7 +1352,7 @@ static ssize_t vinst_pe_cmp_start_stop_show(struct device *dev, > if (!drvdata->nr_pe_cmp) > return -EINVAL; > val = config->vipcssctlr; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static ssize_t vinst_pe_cmp_start_stop_store(struct device *dev, > struct device_attribute *attr, > @@ -1383,7 +1383,7 @@ static ssize_t seq_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->seq_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t seq_idx_store(struct device *dev, > @@ -1419,7 +1419,7 @@ static ssize_t seq_state_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->seq_state; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t seq_state_store(struct device *dev, > @@ -1453,7 +1453,7 @@ static ssize_t seq_event_show(struct device *dev, > idx = config->seq_idx; > val = config->seq_ctrl[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t seq_event_store(struct device *dev, > @@ -1486,7 +1486,7 @@ static ssize_t seq_reset_event_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->seq_rst; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t seq_reset_event_store(struct device *dev, > @@ -1516,7 +1516,7 @@ static ssize_t cntr_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->cntr_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t cntr_idx_store(struct device *dev, > @@ -1556,7 +1556,7 @@ static ssize_t cntrldvr_show(struct device *dev, > idx = config->cntr_idx; > val = config->cntrldvr[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t cntrldvr_store(struct device *dev, > @@ -1594,7 +1594,7 @@ static ssize_t cntr_val_show(struct device *dev, > idx = config->cntr_idx; > val = config->cntr_val[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t cntr_val_store(struct device *dev, > @@ -1632,7 +1632,7 @@ static ssize_t cntr_ctrl_show(struct device *dev, > idx = config->cntr_idx; > val = config->cntr_ctrl[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t cntr_ctrl_store(struct device *dev, > @@ -1664,7 +1664,7 @@ static ssize_t res_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->res_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t res_idx_store(struct device *dev, > @@ -1708,7 +1708,7 @@ static ssize_t res_ctrl_show(struct device *dev, > idx = config->res_idx; > val = config->res_ctrl[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t res_ctrl_store(struct device *dev, > @@ -1746,7 +1746,7 @@ static ssize_t sshot_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->ss_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t sshot_idx_store(struct device *dev, > @@ -1780,7 +1780,7 @@ static ssize_t sshot_ctrl_show(struct device *dev, > raw_spin_lock(&drvdata->spinlock); > val = config->ss_ctrl[config->ss_idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t sshot_ctrl_store(struct device *dev, > @@ -1815,7 +1815,7 @@ static ssize_t sshot_status_show(struct device *dev, > raw_spin_lock(&drvdata->spinlock); > val = config->ss_status[config->ss_idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > static DEVICE_ATTR_RO(sshot_status); > > @@ -1830,7 +1830,7 @@ static ssize_t sshot_pe_ctrl_show(struct device *dev, > raw_spin_lock(&drvdata->spinlock); > val = config->ss_pe_cmp[config->ss_idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t sshot_pe_ctrl_store(struct device *dev, > @@ -1864,7 +1864,7 @@ static ssize_t ctxid_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->ctxid_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t ctxid_idx_store(struct device *dev, > @@ -1911,7 +1911,7 @@ static ssize_t ctxid_pid_show(struct device *dev, > idx = config->ctxid_idx; > val = (unsigned long)config->ctxid_pid[idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t ctxid_pid_store(struct device *dev, > @@ -1972,7 +1972,7 @@ static ssize_t ctxid_masks_show(struct device *dev, > val1 = config->ctxid_mask0; > val2 = config->ctxid_mask1; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx %#lx\n", val1, val2); > + return sysfs_emit(buf, "%#lx %#lx\n", val1, val2); > } > > static ssize_t ctxid_masks_store(struct device *dev, > @@ -2090,7 +2090,7 @@ static ssize_t vmid_idx_show(struct device *dev, > struct etmv4_config *config = &drvdata->config; > > val = config->vmid_idx; > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t vmid_idx_store(struct device *dev, > @@ -2135,7 +2135,7 @@ static ssize_t vmid_val_show(struct device *dev, > raw_spin_lock(&drvdata->spinlock); > val = (unsigned long)config->vmid_val[config->vmid_idx]; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t vmid_val_store(struct device *dev, > @@ -2187,7 +2187,7 @@ static ssize_t vmid_masks_show(struct device *dev, > val1 = config->vmid_mask0; > val2 = config->vmid_mask1; > raw_spin_unlock(&drvdata->spinlock); > - return scnprintf(buf, PAGE_SIZE, "%#lx %#lx\n", val1, val2); > + return sysfs_emit(buf, "%#lx %#lx\n", val1, val2); > } > > static ssize_t vmid_masks_store(struct device *dev, > @@ -2303,7 +2303,7 @@ static ssize_t cpu_show(struct device *dev, > struct etmv4_drvdata *drvdata = dev_get_drvdata(dev->parent); > > val = drvdata->cpu; > - return scnprintf(buf, PAGE_SIZE, "%d\n", val); > + return sysfs_emit(buf, "%d\n", val); > > } > static DEVICE_ATTR_RO(cpu); > @@ -2461,7 +2461,7 @@ static ssize_t coresight_etm4x_reg_show(struct device *dev, > val = etmv4_cross_read(drvdata, offset); > pm_runtime_put_sync(dev->parent); > > - return scnprintf(buf, PAGE_SIZE, "0x%x\n", val); > + return sysfs_emit(buf, "0x%x\n", val); > } > > static bool > diff --git a/drivers/hwtracing/coresight/coresight-stm.c b/drivers/hwtracing/coresight/coresight-stm.c > index e45c6c7204b4..fa09a43889d7 100644 > --- a/drivers/hwtracing/coresight/coresight-stm.c > +++ b/drivers/hwtracing/coresight/coresight-stm.c > @@ -470,7 +470,7 @@ static ssize_t hwevent_enable_show(struct device *dev, > struct stm_drvdata *drvdata = dev_get_drvdata(dev->parent); > unsigned long val = drvdata->stmheer; > > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t hwevent_enable_store(struct device *dev, > @@ -499,7 +499,7 @@ static ssize_t hwevent_select_show(struct device *dev, > struct stm_drvdata *drvdata = dev_get_drvdata(dev->parent); > unsigned long val = drvdata->stmhebsr; > > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t hwevent_select_store(struct device *dev, > @@ -534,7 +534,7 @@ static ssize_t port_select_show(struct device *dev, > spin_unlock(&drvdata->spinlock); > } > > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t port_select_store(struct device *dev, > @@ -581,7 +581,7 @@ static ssize_t port_enable_show(struct device *dev, > spin_unlock(&drvdata->spinlock); > } > > - return scnprintf(buf, PAGE_SIZE, "%#lx\n", val); > + return sysfs_emit(buf, "%#lx\n", val); > } > > static ssize_t port_enable_store(struct device *dev, > diff --git a/drivers/hwtracing/coresight/coresight-sysfs.c b/drivers/hwtracing/coresight/coresight-sysfs.c > index feadaf065b53..547f4970e1e5 100644 > --- a/drivers/hwtracing/coresight/coresight-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-sysfs.c > @@ -315,7 +315,7 @@ static ssize_t enable_sink_show(struct device *dev, > { > struct coresight_device *csdev = to_coresight_device(dev); > > - return scnprintf(buf, PAGE_SIZE, "%u\n", csdev->sysfs_sink_activated); > + return sysfs_emit(buf, "%u\n", csdev->sysfs_sink_activated); > } > > static ssize_t enable_sink_store(struct device *dev, > @@ -343,7 +343,7 @@ static ssize_t enable_source_show(struct device *dev, > struct coresight_device *csdev = to_coresight_device(dev); > > guard(mutex)(&coresight_mutex); > - return scnprintf(buf, PAGE_SIZE, "%u\n", > + return sysfs_emit(buf, "%u\n", > coresight_get_mode(csdev) == CS_MODE_SYSFS); > } > > -- > 2.47.1 > Regards Mike -- Mike Leach Principal Engineer, ARM Ltd. Manchester Design Centre. UK

3 months

[PATCH v11 0/2] coresight: Add Coresight Trace Network On Chip driver

by Yuanfang Zhang

The Trace Network On Chip (TNOC) is an integration hierarchy which is a hardware component that integrates the functionalities of TPDA and funnels. It collects trace from subsystems and transfers it to coresight sink. In addition to the generic TNOC mentioned above, there is also a special type of TNOC called Interconnect TNOC. Unlike the generic TNOC, the Interconnect TNOC doesn't need ATID. Its primary function is to connect the source of subsystems to the Aggregator TNOC. Its driver is different from this patch and will describe it and upstream its driver separately. Signed-off-by: Yuanfang Zhang <quic_yuanfang(a)quicinc.com> --- Changes in v11: - reorder of trace ID release and device unregister. - Link to v10: https://lore.kernel.org/r/20250611-trace-noc-v10-0-a83b5c63da34@quicinc.com Changes in v10: - Rebase to coresight/next branch. - Link to v9: https://lore.kernel.org/r/20250611-trace-noc-v9-0-4322d4cf8f4b@quicinc.com Changes in v9: - Mention the binding is only for Aggregator TNOC. - Link to v8: https://lore.kernel.org/r/20250606-trace-noc-v8-0-833f94712c57@quicinc.com Changes in v8: - Add sysfs node to expose atid. - Link to v7: https://lore.kernel.org/r/20250523-trace-noc-v7-0-d65edbab2997@quicinc.com Changes in v7: - Move the content in header file into coresight-tnoc.c. - Use scoped_guard() to replace spin_lock(). - Invoke coresight_trace_id_put_system_id() for registration failure. - Link to v6: https://lore.kernel.org/r/20250522-trace-noc-v6-0-f5a9bcae90ee@quicinc.com Changes in v6: - Add a newline after return statements. - Use 'x &= foo' to replace 'x = x & foo'. - Use 'x |= foo' to replace 'x = x | foo'. - Link to v5: https://lore.kernel.org/r/20250512-trace-noc-v5-0-f2ef070baee5@quicinc.com Changes in v5: - update cover-letter to describe the Interconnect TNOC. - Link to v4: https://lore.kernel.org/r/20250415-trace-noc-v4-0-979938fedfd8@quicinc.com Changes in v4: - Fix dt_binding warning. - update mask of trace_noc amba_id. - Modify driver comments. - rename TRACE_NOC_SYN_VAL to TRACE_NOC_SYNC_INTERVAL. - Link to v3: https://lore.kernel.org/r/20250411-trace-noc-v3-0-1f19ddf7699b@quicinc.com Changes in v3: - Remove unnecessary sysfs nodes. - update commit messages. - Use 'writel' instead of 'write_relaxed' when writing to the register for the last time. - Add trace_id ops. - Link to v2: https://lore.kernel.org/r/20250226-trace-noc-driver-v2-0-8afc6584afc5@quici… Changes in v2: - Modified the format of DT binging file. - Fix compile warnings. - Link to v1: https://lore.kernel.org/r/46643089-b88d-49dc-be05-7bf0bb21f847@quicinc.com --- Yuanfang Zhang (2): dt-bindings: arm: Add device Trace Network On Chip definition coresight: add coresight Trace Network On Chip driver .../bindings/arm/qcom,coresight-tnoc.yaml | 113 ++++++++++ drivers/hwtracing/coresight/Kconfig | 12 + drivers/hwtracing/coresight/Makefile | 1 + drivers/hwtracing/coresight/coresight-tnoc.c | 242 +++++++++++++++++++++ 4 files changed, 368 insertions(+) --- base-commit: 408c97c4a5e0b634dcd15bf8b8808b382e888164 change-id: 20250403-trace-noc-f8286b30408e Best regards, -- Yuanfang Zhang <quic_yuanfang(a)quicinc.com>

3 months

[PATCH v2 0/5] Refactoring finding CPU phandles in DT

by Alireza Sanaee

This series refactors the way CPU IDs are retrieved from the device tree. Usually, there is a for loop that goes over every single CPU that can be avoided. This also reduces the amount of NULL pointer checks in drivers. I have abstracted away that loop and introduced a new function (of_cpu_node_to_id) for this. This patchset is a subset of [1], where I removed content and patches relevant to hyper-threaded cores for DT. Based on the discussion, the code refactor is still useful, hence this patchset. Changes since v1 [2]: - Rebased on top of the latest mainline. - Addressed Krzysztof Kozlowski's comments -- Hopefully :-) - Addressed Jonathan Cameron's comments. [1] https://lore.kernel.org/all/20250512080715.82-1-alireza.sanaee@huawei.com/ [2] https://lore.kernel.org/all/20250707150414.620-1-alireza.sanaee@huawei.com/ Alireza Sanaee (5): of: add infra for finding CPU id from phandle arch_topology: update CPU map to use the new API coresight: cti: Use of_cpu_phandle_to_id for grabbing CPU id coresight: Use of_cpu_phandle_to_id for grabbing CPU id perf/arm-dsu: refactor cpu id retrieval via new API of_cpu_phandle_to_id drivers/base/arch_topology.c | 16 ++++---- .../coresight/coresight-cti-platform.c | 14 +------ .../hwtracing/coresight/coresight-platform.c | 15 +------ drivers/of/cpu.c | 40 +++++++++++++++++++ drivers/perf/arm_dsu_pmu.c | 8 +--- include/linux/of.h | 9 +++++ 6 files changed, 61 insertions(+), 41 deletions(-) -- 2.43.0

3 months

[PATCH v10 0/2] coresight: Add Coresight Trace Network On Chip driver

by Yuanfang Zhang

The Trace Network On Chip (TNOC) is an integration hierarchy which is a hardware component that integrates the functionalities of TPDA and funnels. It collects trace from subsystems and transfers it to coresight sink. In addition to the generic TNOC mentioned above, there is also a special type of TNOC called Interconnect TNOC. Unlike the generic TNOC, the Interconnect TNOC doesn't need ATID. Its primary function is to connect the source of subsystems to the Aggregator TNOC. Its driver is different from this patch and will describe it and upstream its driver separately. Signed-off-by: Yuanfang Zhang <quic_yuanfang(a)quicinc.com> --- Changes in v10: - Rebase to coresight/next branch. - Link to v9: https://lore.kernel.org/r/20250611-trace-noc-v9-0-4322d4cf8f4b@quicinc.com Changes in v9: - Mention the binding is only for Aggregator TNOC. - Link to v8: https://lore.kernel.org/r/20250606-trace-noc-v8-0-833f94712c57@quicinc.com Changes in v8: - Add sysfs node to expose atid. - Link to v7: https://lore.kernel.org/r/20250523-trace-noc-v7-0-d65edbab2997@quicinc.com Changes in v7: - Move the content in header file into coresight-tnoc.c. - Use scoped_guard() to replace spin_lock(). - Invoke coresight_trace_id_put_system_id() for registration failure. - Link to v6: https://lore.kernel.org/r/20250522-trace-noc-v6-0-f5a9bcae90ee@quicinc.com Changes in v6: - Add a newline after return statements. - Use 'x &= foo' to replace 'x = x & foo'. - Use 'x |= foo' to replace 'x = x | foo'. - Link to v5: https://lore.kernel.org/r/20250512-trace-noc-v5-0-f2ef070baee5@quicinc.com Changes in v5: - update cover-letter to describe the Interconnect TNOC. - Link to v4: https://lore.kernel.org/r/20250415-trace-noc-v4-0-979938fedfd8@quicinc.com Changes in v4: - Fix dt_binding warning. - update mask of trace_noc amba_id. - Modify driver comments. - rename TRACE_NOC_SYN_VAL to TRACE_NOC_SYNC_INTERVAL. - Link to v3: https://lore.kernel.org/r/20250411-trace-noc-v3-0-1f19ddf7699b@quicinc.com Changes in v3: - Remove unnecessary sysfs nodes. - update commit messages. - Use 'writel' instead of 'write_relaxed' when writing to the register for the last time. - Add trace_id ops. - Link to v2: https://lore.kernel.org/r/20250226-trace-noc-driver-v2-0-8afc6584afc5@quici… Changes in v2: - Modified the format of DT binging file. - Fix compile warnings. - Link to v1: https://lore.kernel.org/r/46643089-b88d-49dc-be05-7bf0bb21f847@quicinc.com --- Yuanfang Zhang (2): dt-bindings: arm: Add device Trace Network On Chip definition coresight: add coresight Trace Network On Chip driver .../bindings/arm/qcom,coresight-tnoc.yaml | 113 ++++++++++ drivers/hwtracing/coresight/Kconfig | 12 + drivers/hwtracing/coresight/Makefile | 1 + drivers/hwtracing/coresight/coresight-tnoc.c | 242 +++++++++++++++++++++ 4 files changed, 368 insertions(+) --- base-commit: 408c97c4a5e0b634dcd15bf8b8808b382e888164 change-id: 20250403-trace-noc-f8286b30408e Best regards, -- Yuanfang Zhang <quic_yuanfang(a)quicinc.com>

3 months

[PATCH v6 0/5] coresight: Add remote etm support

by Mao Jinlong

The system on chip (SoC) consists of main APSS(Applications processor subsytem) and additional processors like modem, lpass. There is coresight-etm driver for etm trace of APSS. Coresight remote etm driver is for enabling and disabling the etm trace of remote processors. It uses QMI interface to communicate with remote processors' software and uses coresight framework to configure the connection from remote etm source to TMC sinks. Example to capture the remote etm trace: Enable source: echo 1 > /sys/bus/coresight/devices/tmc_etf0/enable_sink echo 1 > /sys/bus/coresight/devices/remote_etm0/enable_source Capture the trace: cat /dev/tmc_etf0 > /data/remote_etm.bin Disable source: echo 0 > /sys/bus/coresight/devices/remote_etm0/enable_source Changes since V5: 1. Fix the warning and error when compile. 2. Add traceid for remote etm. 3. Change qcom,qmi-id tp qcom,qmi-instance-id. Changes since V4: 1. Add coresight QMI driver 2. Add coresight qmi node and qcom,qmi-id of modem-etm in msm8996 dtsi V5: https://lwn.net/ml/all/20250424115854.2328190-1-quic_jinlmao@quicinc.com/ Changes since V3: 1. Use different compatible for different remote etms in dt. 2. Get qmi instance id from the match table data in driver. Change since V2: 1. Change qcom,inst-id to qcom,qmi-id 2. Fix the error in code for type of remote_etm_remove 3. Depend on QMI helper in Kconfig Changes since V1: 1. Remove unused content 2. Use CORESIGHT_DEV_SUBTYPE_SOURCE_OTHERS as remote etm source type. 3. Use enabled instead of enable in driver data. 4. Validate instance id value where it's read from the DT. Mao Jinlong (5): dt-bindings: arm: Add CoreSight QMI component description coresight: Add coresight QMI driver dt-bindings: arm: Add qcom,qmi-id for remote etm coresight: Add remote etm support arm64: dts: qcom: msm8996: Add coresight qmi node .../bindings/arm/qcom,coresight-qmi.yaml | 65 +++++ .../arm/qcom,coresight-remote-etm.yaml | 9 + arch/arm64/boot/dts/qcom/msm8996.dtsi | 11 + drivers/hwtracing/coresight/Kconfig | 23 ++ drivers/hwtracing/coresight/Makefile | 2 + drivers/hwtracing/coresight/coresight-qmi.c | 198 +++++++++++++ drivers/hwtracing/coresight/coresight-qmi.h | 101 +++++++ .../coresight/coresight-remote-etm.c | 262 ++++++++++++++++++ 8 files changed, 671 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/qcom,coresight-qmi.yaml create mode 100644 drivers/hwtracing/coresight/coresight-qmi.c create mode 100644 drivers/hwtracing/coresight/coresight-qmi.h create mode 100644 drivers/hwtracing/coresight/coresight-remote-etm.c -- 2.25.1

3 months

[PATCH v5 0/5] coresight: Add remote etm support

by Mao Jinlong

The system on chip (SoC) consists of main APSS(Applications processor subsytem) and additional processors like modem, lpass. There is coresight-etm driver for etm trace of APSS. Coresight remote etm driver is for enabling and disabling the etm trace of remote processors. It uses QMI interface to communicate with remote processors' software and uses coresight framework to configure the connection from remote etm source to TMC sinks. Example to capture the remote etm trace: Enable source: echo 1 > /sys/bus/coresight/devices/tmc_etf0/enable_sink echo 1 > /sys/bus/coresight/devices/remote_etm0/enable_source Capture the trace: cat /dev/tmc_etf0 > /data/remote_etm.bin Disable source: echo 0 > /sys/bus/coresight/devices/remote_etm0/enable_source Changes since V4: 1. Add coresight QMI driver 2. Add coresight qmi node and qcom,qmi-id of modem-etm in msm8996 dtsi Changes since V3: 1. Use different compatible for different remote etms in dt. 2. Get qmi instance id from the match table data in driver. Change since V2: 1. Change qcom,inst-id to qcom,qmi-id 2. Fix the error in code for type of remote_etm_remove 3. Depend on QMI helper in Kconfig Changes since V1: 1. Remove unused content 2. Use CORESIGHT_DEV_SUBTYPE_SOURCE_OTHERS as remote etm source type. 3. Use enabled instead of enable in driver data. 4. Validate instance id value where it's read from the DT. Mao Jinlong (5): dt-bindings: arm: Add CoreSight QMI component description coresight: Add coresight QMI driver dt-bindings: arm: Add qcom,qmi-id for remote etm coresight: Add remote etm support arm64: dts: qcom: msm8996: Add coresight qmi node .../bindings/arm/qcom,coresight-qmi.yaml | 65 +++++ .../arm/qcom,coresight-remote-etm.yaml | 10 + arch/arm64/boot/dts/qcom/msm8996.dtsi | 11 + drivers/hwtracing/coresight/Kconfig | 24 ++ drivers/hwtracing/coresight/Makefile | 2 + drivers/hwtracing/coresight/coresight-qmi.c | 209 +++++++++++++++ drivers/hwtracing/coresight/coresight-qmi.h | 107 ++++++++ .../coresight/coresight-remote-etm.c | 252 ++++++++++++++++++ 8 files changed, 680 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/qcom,coresight-qmi.yaml create mode 100644 drivers/hwtracing/coresight/coresight-qmi.c create mode 100644 drivers/hwtracing/coresight/coresight-qmi.h create mode 100644 drivers/hwtracing/coresight/coresight-remote-etm.c -- 2.25.1

3 months

[PATCH 0/2] coresight: Dismiss smatch warnings

by Leo Yan

This series polishes the code to address warnings reported by the smatch static checker. Smatch reports a error "Function too hairy" for etm4x_sysreg_read() and etm4x_sysreg_write(). This is a trade off to avoid large code blocks, the implementation encapsulates logic using nested macros. But this is not friendly to static checker. For now, the code will be kept as is. Signed-off-by: Leo Yan <leo.yan(a)arm.com> --- Leo Yan (2): coresight: stm: Remove redundant NULL checks coresight: perf: Use %px for printing pointers drivers/hwtracing/coresight/coresight-etm-perf.c | 4 ++-- drivers/hwtracing/coresight/coresight-stm.c | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) --- base-commit: 408c97c4a5e0b634dcd15bf8b8808b382e888164 change-id: 20250611-arm_cs_fix_smatch_warning_v1-6c2b5e78e38a Best regards, -- Leo Yan <leo.yan(a)arm.com>

3 months

Re: [PATCH v5 1/5] dt-bindings: arm: Add CoreSight QMI component description

by Jinlong Mao

On 2025/4/25 4:30, Konrad Dybcio wrote: > On 4/24/25 1:58 PM, Mao Jinlong wrote: >> Add new coresight-qmi.yaml file describing the bindings required >> to define qmi node in the device trees. >> >> Signed-off-by: Mao Jinlong <quic_jinlmao(a)quicinc.com> >> --- > > Is the service-id hardcoded/well-known? If so, we can drop > this devicetree node and create a new platform device (& probe the > related driver) based on the presence of qcom,qmi-id that you add > in patch 3 > > Konrad service-id is not hardcoded. Different qmi connections will have different service ids. >

3 months

Re: [PATCH] hwtracing: coresight: Use of_reserved_mem_region_to_resource() for "memory-region"

by Suzuki K Poulose

On Thu, 03 Jul 2025 13:35:33 -0500, Rob Herring (Arm) wrote: > Use the newly added of_reserved_mem_region_to_resource() function to > handle "memory-region" properties. > > Applied, thanks! [1/1] hwtracing: coresight: Use of_reserved_mem_region_to_resource() for "memory-region" https://git.kernel.org/coresight/c/f0148bd7 Best regards, -- Suzuki K Poulose <suzuki.poulose(a)arm.com>

3 months

[PATCH] coresight: Only register perf symlink for sinks with alloc_buffer

by Yuanfang Zhang

Ensure that etm_perf_add_symlink_sink() is only called for devices that implement the alloc_buffer operation. This prevents invalid symlink creation for dummy sinks that do not implement alloc_buffer. Without this check, perf may attempt to use a dummy sink that lacks alloc_buffer operationsu to initialise perf's ring buffer, leading to runtime failures. Fixes: 9d3ba0b6c0569 ("Coresight: Add coresight dummy driver") Signed-off-by: Yuanfang Zhang <quic_yuanfang(a)quicinc.com> --- drivers/hwtracing/coresight/coresight-core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight/coresight-core.c index fa758cc21827552a5c97b6bdd05d22dec4994b22..fddf04c5ee46eb4d559416296f7e85ce6c5689fa 100644 --- a/drivers/hwtracing/coresight/coresight-core.c +++ b/drivers/hwtracing/coresight/coresight-core.c @@ -1374,8 +1374,9 @@ struct coresight_device *coresight_register(struct coresight_desc *desc) goto out_unlock; } - if (csdev->type == CORESIGHT_DEV_TYPE_SINK || - csdev->type == CORESIGHT_DEV_TYPE_LINKSINK) { + if ((csdev->type == CORESIGHT_DEV_TYPE_SINK || + csdev->type == CORESIGHT_DEV_TYPE_LINKSINK) && + sink_ops(csdev)->alloc_buffer) { ret = etm_perf_add_symlink_sink(csdev); if (ret) { --- base-commit: 408c97c4a5e0b634dcd15bf8b8808b382e888164 change-id: 20250630-etm_perf_sink-ee0fcffc6820 Best regards, -- Yuanfang Zhang <quic_yuanfang(a)quicinc.com>

3 months

← Newer
1
...
9
10
11
12
13
14
15
...
250
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

CoreSight