On Thu, Jul 12, 2018 at 06:28:33AM +0000, Pawel Laszczak wrote: > > > This patch add additional functions that converts some fields to string. > > > > > > For example function usbssp_trb_comp_code_string take completion c ode > > > value and return string describing completion code. > > > > Odd line-wrapping :( > Do you mean additional space character between "c" and "ode" It wrapped lines at that point for me, yes. > > And why do you want a string for these things? Who cares about them? > It will be used in next version of driver. I have plan to add gadget-debugfs.c file. > At this moment this functions can be useful during debugging driver if someone want to display > some debug information. > > Should I remove this function ? If no one is using it now, yes, it does not need to be there, right? We don't have dead code around for no reason, otherwise someone will come along and remove it :) thanks, greg k-h -- 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