> From: Felipe Balbi [mailto:[email protected]]
> Sent: Tuesday, August 26, 2014 1:42 PM
> 
> On Fri, Aug 22, 2014 at 04:56:46PM -0500, Felipe Balbi wrote:

...

> yeah, it took longer than expected (been busy lately), but here's an
> example trace with all trace points enabled:
> 
> # tracer: nop
> #
> # entries-in-buffer/entries-written: 1038/1038   #P:1
> #
> #                              _-----=> irqs-off
> #                             / _----=> need-resched
> #                            | / _---=> hardirq/softirq
> #                            || / _--=> preempt-depth
> #                            ||| /     delay
> #           TASK-PID   CPU#  ||||    TIMESTAMP  FUNCTION
> #              | |       |   ||||       |         |
>           <idle>-0     [000] d.h.   155.653881: dwc3_readl: add fa39c40c 
> value 00000004
>           <idle>-0     [000] d.h.   155.653903: dwc3_readl: add fa39c408 
> value 00000100
>           <idle>-0     [000] d.h.   155.653908: dwc3_writel: addr fa39c408 
> value 80000100

Looks like there is an inconsistency between readl/writel (add vs. addr).

Other than that, I really like this.

Reviewed-by: Paul Zimmerman <[email protected]>

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to