lkml.org 
[lkml]   [2017]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 08/12] coresight tmc: Handle configuration types properly
    From
    Date
    On 14/06/17 18:59, Mathieu Poirier wrote:
    > On Mon, Jun 12, 2017 at 03:36:47PM +0100, Suzuki K Poulose wrote:
    >> Coresight SoC 600 defines a new configuration for TMC, Embedded Trace
    >> Streamer (ETS), indicated by 0x3 in MODE:CONFIG_TYPE. Make sure
    >> the driver handles the new type properly.
    >
    > From what I see below this patch only improves readability by moving the if/else
    > compound to a case statement - there is no trace of the new ETS block.
    > Readability enhancement are good but the changelog must match the code.

    Mathieu,

    Not really. It fixes an issue with the current code. We check for ETB and ETR,
    if it doesn't match either of those, we assume it is ETF which is fine with SoC-400.
    But with SoC-600, we could have an ETS with id 0x3 and the driver could treat it
    as an ETF and go on with the initialisation. This patch makes sure that we
    do check for the ETF and report that ETS is not a supported configuration.

    Suzuki

    >
    >>
    >> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
    >> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
    >> ---
    >> drivers/hwtracing/coresight/coresight-tmc.c | 14 +++++++++++---
    >> 1 file changed, 11 insertions(+), 3 deletions(-)
    >>
    >> diff --git a/drivers/hwtracing/coresight/coresight-tmc.c b/drivers/hwtracing/coresight/coresight-tmc.c
    >> index fd5a2e0..7152656 100644
    >> --- a/drivers/hwtracing/coresight/coresight-tmc.c
    >> +++ b/drivers/hwtracing/coresight/coresight-tmc.c
    >> @@ -358,11 +358,13 @@ static int tmc_probe(struct amba_device *adev, const struct amba_id *id)
    >> desc.dev = dev;
    >> desc.groups = coresight_tmc_groups;
    >>
    >> - if (drvdata->config_type == TMC_CONFIG_TYPE_ETB) {
    >> + switch (drvdata->config_type) {
    >> + case TMC_CONFIG_TYPE_ETB:
    >> desc.type = CORESIGHT_DEV_TYPE_SINK;
    >> desc.subtype.sink_subtype = CORESIGHT_DEV_SUBTYPE_SINK_BUFFER;
    >> desc.ops = &tmc_etb_cs_ops;
    >> - } else if (drvdata->config_type == TMC_CONFIG_TYPE_ETR) {
    >> + break;
    >> + case TMC_CONFIG_TYPE_ETR:
    >> desc.type = CORESIGHT_DEV_TYPE_SINK;
    >> desc.subtype.sink_subtype = CORESIGHT_DEV_SUBTYPE_SINK_BUFFER;
    >> desc.ops = &tmc_etr_cs_ops;
    >> @@ -373,10 +375,16 @@ static int tmc_probe(struct amba_device *adev, const struct amba_id *id)
    >> ret = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(40));
    >> if (ret)
    >> goto out;
    >> - } else {
    >> + break;
    >> + case TMC_CONFIG_TYPE_ETF:
    >> desc.type = CORESIGHT_DEV_TYPE_LINKSINK;
    >> desc.subtype.link_subtype = CORESIGHT_DEV_SUBTYPE_LINK_FIFO;
    >> desc.ops = &tmc_etf_cs_ops;
    >> + break;
    >> + default:
    >> + pr_err("%s: Unsupported TMC config\n", pdata->name);
    >> + ret = -EINVAL;
    >> + goto out;
    >> }
    >>
    >> drvdata->csdev = coresight_register(&desc);
    >> --
    >> 2.7.4
    >>

    \
     
     \ /
      Last update: 2017-06-15 12:28    [W:3.802 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site