On Thu, Aug 16, 2018 at 04:13:03PM +0100, Suzuki K Poulose wrote: > Hi Mathieu, > > The patch looks good to me. One minor nit below.
You have not included any comment - did you change your mind? > > On 14/08/18 23:14, Mathieu Poirier wrote: > >This patch moves the etb_drvdata::mode from a locat_t to a simple u32, > >as it is for the ETF and ETR drivers. This streamlines the code and adds > >commonality with the other drivers when dealing with similar operations. > > > >Signed-off-by: Mathieu Poirier <[email protected]> > >--- > > drivers/hwtracing/coresight/coresight-etb10.c | 62 > > +++++++++++++++------------ > > 1 file changed, 34 insertions(+), 28 deletions(-) > > > >diff --git a/drivers/hwtracing/coresight/coresight-etb10.c > >b/drivers/hwtracing/coresight/coresight-etb10.c > >index 9fd77fdc1244..69287163ce4e 100644 > >--- a/drivers/hwtracing/coresight/coresight-etb10.c > >+++ b/drivers/hwtracing/coresight/coresight-etb10.c > > >@@ -488,7 +494,7 @@ static void etb_dump(struct etb_drvdata *drvdata) > > unsigned long flags; > > spin_lock_irqsave(&drvdata->spinlock, flags); > >- if (local_read(&drvdata->mode) == CS_MODE_SYSFS) { > >+ if (drvdata->mode == CS_MODE_SYSFS) { > > etb_disable_hw(drvdata); > > etb_dump_hw(drvdata); > > etb_enable_hw(drvdata); > > Looks good to me. > > Reviewed-by : Suzuki K Poulose <[email protected]>
