On Saturday 31 October 2009 22:00:09 Tom wrote:
> Mike Frysinger wrote:
> > On Saturday 31 October 2009 13:37:39 Tom Rix wrote:
> >> +#ifdef DEBUG
> >> +static inline void print_device_descriptor(struct usb_device_descriptor
> >> *d) +{
> >> + serial_printf("usb device descriptor \n");
> >
> > do you really need serial_printf() ? what's wrong with debug() ? then
> > you dont even really need "#ifdef DEBUG" around all the functions ...
>
> The explicit serial_printf is done because this patch set changes the
> stdin and stdout for serial to usbtty.
>
> When you use printf to debug printf, you regress into a bad state.so in your specific use case it makes sense *some* of the time (usbtty is enabled and the console has been changed to it), but in the general use case (usb debugging), it does not. why not make it intelligent instead of penalizing everyone to use their serial console: - default to debug() - if usbtty support is enabled, check the current stdout console to see if it's set to a usbtty, and only then fall back to forcing serial_printf() -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
