Messages in this thread | | | From | Mathieu Poirier <> | Date | Thu, 6 Oct 2016 10:10:46 -0600 | Subject | Re: [PATCH v3] Adding missing features of Coresight PTM components |
| |
On 6 October 2016 at 09:57, Muhammad Abdul WAHAB <muhammadabdul.wahab@centralesupelec.fr> wrote: > Hi, > >> Where is ETMCR_RETURN_STACK_EN defined? Did you send me code that >> doesn't compile? > > I changed the naming in the .h file before submitting it and I forgot to > change it in .c file. I am sorry. It is defined in coresight-etm.h. > Here is the correct patch.
Then send me a proper V4 patch.
From hereon I suggest you compile your code before sending patches to the mailing list - other people won't be as lenient as I am.
> --- > drivers/hwtracing/coresight/coresight-etm.h | 5 +++++ > drivers/hwtracing/coresight/coresight-etm3x-sysfs.c | 10 ++++++++++ > 2 files changed, 15 insertions(+) > > diff --git a/drivers/hwtracing/coresight/coresight-etm.h > b/drivers/hwtracing/coresight/coresight-etm.h > index 4a18ee4..ad063d7 100644 > --- a/drivers/hwtracing/coresight/coresight-etm.h > +++ b/drivers/hwtracing/coresight/coresight-etm.h > @@ -89,11 +89,13 @@ > /* ETMCR - 0x00 */ > #define ETMCR_PWD_DWN BIT(0) > #define ETMCR_STALL_MODE BIT(7) > +#define ETMCR_BRANCH_BROADCAST BIT(8) > #define ETMCR_ETM_PRG BIT(10) > #define ETMCR_ETM_EN BIT(11) > #define ETMCR_CYC_ACC BIT(12) > #define ETMCR_CTXID_SIZE (BIT(14)|BIT(15)) > #define ETMCR_TIMESTAMP_EN BIT(28) > +#define ETMCR_RETURN_STACK BIT(29) > /* ETMCCR - 0x04 */ > #define ETMCCR_FIFOFULL BIT(23) > /* ETMPDCR - 0x310 */ > @@ -110,8 +112,11 @@ > #define ETM_MODE_STALL BIT(2) > #define ETM_MODE_TIMESTAMP BIT(3) > #define ETM_MODE_CTXID BIT(4) > +#define ETM_MODE_BBROAD BIT(5) > +#define ETM_MODE_RET_STACK BIT(6) > #define ETM_MODE_ALL (ETM_MODE_EXCLUDE | ETM_MODE_CYCACC | \ > ETM_MODE_STALL | ETM_MODE_TIMESTAMP | \ > + ETM_MODE_BBROAD | ETM_MODE_RET_STACK | \ > ETM_MODE_CTXID | ETM_MODE_EXCL_KERN | \ > ETM_MODE_EXCL_USER) > > diff --git a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > index 5ea0909..a76009a 100644 > --- a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > @@ -164,6 +164,16 @@ static ssize_t mode_store(struct device *dev, > else > config->ctrl &= ~ETMCR_CTXID_SIZE; > > + if (config->mode & ETM_MODE_BBROAD) > + config->ctrl |= ETMCR_BRANCH_BROADCAST; > + else > + config->ctrl &= ~ETMCR_BRANCH_BROADCAST; > + > + if (config->mode & ETM_MODE_RET_STACK) > + config->ctrl |= ETMCR_RETURN_STACK; > + else > + config->ctrl &= ~ETMCR_RETURN_STACK; > + > if (config->mode & (ETM_MODE_EXCL_KERN | ETM_MODE_EXCL_USER)) > etm_config_trace_mode(config); > > -- > 1.9.1
| |