Re: [PATCH 6/7] usb: cdc-wdm: support command echo

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Freitag, 20. Januar 2012, 14:52:50 schrieb Bjørn Mork:
> Marcel Holtmann <marcel@xxxxxxxxxxxx> writes:
> 
> > this is not how application protocols are designed. And we do not even
> > need this for debugging. You think that you do right now, because it
> > makes your life initially easier (and I understand that), but it is
> > actually total waste. In the long term none of this will be needed.
> 
> OK, you are very convincing yet another time. 
> 
> But given that this is something like three lines of code, would it be
> acceptable to have a "debug_echo" module param defaulting to off
> instead?  Somehow I still believe I will want to enable this from time
> to time.

No. It would be trouble removing it and you can just as well apply
it locally. For remote debugging you are better off with other tools.

	Sorry
		Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux