Commit 1381a511 authored by Felipe Balbi's avatar Felipe Balbi
Browse files

usb: dwc3: debug: purge usage of strcat 



Now that buffer size is always passed around, we don't need to rely on
strcat anymore.

Signed-off-by: default avatarFelipe Balbi <felipe.balbi@linux.intel.com>
parent 7790b355

drivers/usb/dwc3/debug.h

+9 −5
Original line number Diff line number Diff line
@@ -545,21 +545,25 @@ static inline const char *dwc3_ep_event_string(char *str, size_t size, 
				status & DEPEVT_STATUS_TRANSFER_ACTIVE ?

				" (Active)" : " (Not Active)");



		len = strlen(str);



		/* Control Endpoints */

		if (epnum <= 1) {

			int phase = DEPEVT_STATUS_CONTROL_PHASE(event->status);



			switch (phase) {

			case DEPEVT_STATUS_CONTROL_DATA:

				strcat(str, " [Data Phase]");

				snprintf(str + ret, size - ret,

						" [Data Phase]");

				break;

			case DEPEVT_STATUS_CONTROL_STATUS:

				strcat(str, " [Status Phase]");

				snprintf(str + ret, size - ret,

						" [Status Phase]");

			}

		}

		break;

	case DWC3_DEPEVT_RXTXFIFOEVT:

		strcat(str, "FIFO");

		snprintf(str + ret, size - ret, "FIFO");

		break;

	case DWC3_DEPEVT_STREAMEVT:

		status = event->status;
@@ -571,13 +575,13 @@ static inline const char *dwc3_ep_event_string(char *str, size_t size, 
			break;

		case DEPEVT_STREAMEVT_NOTFOUND:

		default:

			strcat(str, " Stream Not Found");

			snprintf(str + ret, size - ret, " Stream Not Found");

			break;

		}



		break;

	case DWC3_DEPEVT_EPCMDCMPLT:

		strcat(str, "Endpoint Command Complete");

		snprintf(str + ret, size - ret, "Endpoint Command Complete");

		break;

	default:

		snprintf(str, size, "UNKNOWN");