coresight: etm4x: Cleanup TRCIDR2 register accesses
authorJames Clark <james.clark@arm.com>
Fri, 4 Mar 2022 17:18:59 +0000 (17:18 +0000)
committerMathieu Poirier <mathieu.poirier@linaro.org>
Wed, 13 Apr 2022 17:04:19 +0000 (11:04 -0600)
This is a no-op change for style and consistency and has no effect on
the binary output by the compiler. In sysreg.h fields are defined as
the register name followed by the field name and then _MASK. This
allows for grepping for fields by name rather than using magic numbers.

Signed-off-by: James Clark <james.clark@arm.com>
Reviewed-by: Mike Leach <mike.leach@linaro.org>
Link: https://lore.kernel.org/r/20220304171913.2292458-3-james.clark@arm.com
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
drivers/hwtracing/coresight/coresight-etm4x-core.c
drivers/hwtracing/coresight/coresight-etm4x.h

index 9120390a761348749b3e557defad24c2ea16762c..fd44231e9d8a9304dbdf14f81edb08041cd820a0 100644 (file)
@@ -1116,11 +1116,11 @@ static void etm4_init_arch_data(void *info)
        /* maximum size of resources */
        etmidr2 = etm4x_relaxed_read32(csa, TRCIDR2);
        /* CIDSIZE, bits[9:5] Indicates the Context ID size */
-       drvdata->ctxid_size = BMVAL(etmidr2, 5, 9);
+       drvdata->ctxid_size = FIELD_GET(TRCIDR2_CIDSIZE_MASK, etmidr2);
        /* VMIDSIZE, bits[14:10] Indicates the VMID size */
-       drvdata->vmid_size = BMVAL(etmidr2, 10, 14);
+       drvdata->vmid_size = FIELD_GET(TRCIDR2_VMIDSIZE_MASK, etmidr2);
        /* CCSIZE, bits[28:25] size of the cycle counter in bits minus 12 */
-       drvdata->ccsize = BMVAL(etmidr2, 25, 28);
+       drvdata->ccsize = FIELD_GET(TRCIDR2_CCSIZE_MASK, etmidr2);
 
        etmidr3 = etm4x_relaxed_read32(csa, TRCIDR3);
        /* CCITMIN, bits[11:0] minimum threshold value that can be programmed */
index 300741fbc0de39cb115daf68bde175f71a824bea..cfdf966016b7b5682c7c53189c3b4c07ea3d23ab 100644 (file)
 #define TRCIDR0_QSUPP_MASK                     GENMASK(16, 15)
 #define TRCIDR0_TSSIZE_MASK                    GENMASK(28, 24)
 
+#define TRCIDR2_CIDSIZE_MASK                   GENMASK(9, 5)
+#define TRCIDR2_VMIDSIZE_MASK                  GENMASK(14, 10)
+#define TRCIDR2_CCSIZE_MASK                    GENMASK(28, 25)
+
 /*
  * System instructions to access ETM registers.
  * See ETMv4.4 spec ARM IHI0064F section 4.3.6 System instructions