On Tue, Sep 23, 2014 at 08:19:41PM +1000, Alexey Kardashevskiy wrote: > This adds a stub for ohci_td_pkt() function (which traces packets) > when configured without --enable-trace-backend > > Signed-off-by: Alexey Kardashevskiy <a...@ozlabs.ru> > --- > > It should probably be squashed to > [PATCH] ohci: Convert fprint/DPRINTF/print to traces > > Sorry about that... > > --- > hw/usb/hcd-ohci.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c > index 4a4dd02..7ea871d 100644 > --- a/hw/usb/hcd-ohci.c > +++ b/hw/usb/hcd-ohci.c > @@ -920,6 +920,7 @@ static int ohci_service_iso_td(OHCIState *ohci, struct > ohci_ed *ed, > return 1; > } > > +#ifdef trace_event_get_state > static void ohci_td_pkt(const char *msg, const uint8_t *buf, size_t len) > { > bool print16 = !!trace_event_get_state(TRACE_USB_OHCI_TD_PKT_SHORT);
The preferred define for this is CONFIG_TRACE_NOP. Checking for trace_event_get_state could break if trace/control.h changes in the future. Stefan
pgpaJy7ccmPzV.pgp
Description: PGP signature