From nobody Thu Nov 28 21:50:10 2024 Received: from mail-qk1-f181.google.com (mail-qk1-f181.google.com [209.85.222.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0FEEB61FCE; Sat, 28 Sep 2024 15:09:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727536165; cv=none; b=EGgcjHMLx1tG9MdjHOKYG813GFdvEqnknwB1dO4aWId5lQ5hMBQuQZ5KYn/IP22gtA0fa81cM1LAvH4UUazqw0iVDCle3qngJ9rLOES5dQIE2TCPQIweTp9EGmDkDBvZPW5mjMyOh34sCE48pDPFDgVQxdfF54ODRwtlzYreU2U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727536165; c=relaxed/simple; bh=IyJz+iqPMoLhHut/jqmGwsV7ix4D1Ize338TAuomV9c=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=oncPWToX7X+hFqLQrq8/xKxnxTk3R+YOlGxYqM5CIQ+TsQY2ncO7Tv0QBnDazm7k9XhX7cXX/34nwWR+cQJegH5c6lAOpcWURiSM5kBJDc1uh8HVEObBfEqKy1+3cHc6JpQQHezOVFfU+ZAjbmZ47d4zDqffC/VCPWhdD5M+Yes= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=kF6EOpYd; arc=none smtp.client-ip=209.85.222.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kF6EOpYd" Received: by mail-qk1-f181.google.com with SMTP id af79cd13be357-7a9dbdc8baaso26970585a.0; Sat, 28 Sep 2024 08:09:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1727536161; x=1728140961; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=AN8bBg4/BtNlz+HQauYfU+FQJfwrHCBrf2UGuGo3/5w=; b=kF6EOpYd7wx0AdrKCjz5EzhT++kDrnTmatktHVqnMSiPyfShfLbP1wzv3YNQgyq1rv vn8zga076WyxgtQydYw6qtjyJpz+6rlp6JLc/mNjTcdAZ8tmmhKFfKGsvQzn7IaopVgJ 7f/YJ4B/HsuJ08woLFPOLIofPJrpQU6IvTasx4FNVxFzRZIuQEnFBkBqe6r25idJ3tHY j2ICAT+RYYg8gYUijoCIEuksZM7MSrDcQ729Jt1lstFSnTj4tkTfWdBSlOwf7+X8EgqT PklSICLdRGa3imhNxGyP9pT5vcQd0mdeX6ug7xbCh8nEcQnVnPFlZ8W34A+b3/PBx/B5 dJOw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727536161; x=1728140961; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=AN8bBg4/BtNlz+HQauYfU+FQJfwrHCBrf2UGuGo3/5w=; b=Cw7gY3gcmo6+7GE2hF09b68U0YNlLtW7OaVeoeZ0+KEGyrt5KKpD+HHcdLwOwN+hyc HjvX3RQfiXHOs0VGYTCwu+kyR4Yt5RkEkoPH884EfdNbKaaf81p1Ok3j3gKcl9k7RiQq DtBUsfVmTOHzvePvE4pwZctAo4mEOShA8TF61Lu0j5eZqwDSKZE0eG2pJN/2PINK8lwn dHH/PlYgVZKYX2hS4j/tCdBhQ6Jhb9NjSSsOQoRF8MflZYqYsd0xgrsC9ge5cYzcR+kW n+bUb0n9KfLRpSXDbDiQ+MviXCYFKpw0d+y9GMKyQ/njPipW2NEnASjIUNtuPo/sWdd2 LjZA== X-Forwarded-Encrypted: i=1; AJvYcCVdUDAxSSauDYwTgLyLPvALPBYDzA+j+8RtEKY9UpudKHdUC4SkaHv+6NPIC/izSEQeW77X09n6TAC3E8mD@vger.kernel.org, AJvYcCXvCHWWCZ9RUzDHodowdm/geGzNeeWolbcBc1mJcPbbzp+OeJ5UH1W8KXgodYJ4YT4N9mdvifi/XDE=@vger.kernel.org X-Gm-Message-State: AOJu0Yy64wVNa/9XKA3XMuLGRJKM7Sy6KfDkfQCc+WEQRcYXcsXH9+WH cU3kYxT1qfXSHAqIZX6XagJjSkHh9KQI421bzjfsTA6n40ZN3obZ4anh7MVcYzQ= X-Google-Smtp-Source: AGHT+IHaLZrCTIbzpzEWmpbBnoNblHkLEWA5bgRcxrdSy2w810sJF+7rN+mbLSf9Pn6NvPRdvGHCaQ== X-Received: by 2002:a05:622a:609:b0:458:3297:806f with SMTP id d75a77b69052e-45c9f300aa3mr44139931cf.10.1727536161405; Sat, 28 Sep 2024 08:09:21 -0700 (PDT) Received: from localhost.localdomain (syn-104-229-042-148.res.spectrum.com. [104.229.42.148]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-45c9f35392esm18662881cf.82.2024.09.28.08.09.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 28 Sep 2024 08:09:20 -0700 (PDT) From: crwulff@gmail.com To: linux-usb@vger.kernel.org Cc: Pavel Hofman , Greg Kroah-Hartman , James Gruber , Jeff Johnson , John Keeping , Jonathan Corbet , Lee Jones , Perr Zhang , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Chris Wulff Subject: [PATCH RFC 02/14] usb: gadget: f_uac1: Fix fs/hs/ss descriptors to have correct values Date: Sat, 28 Sep 2024 11:08:53 -0400 Message-ID: <20240928150905.2616313-3-crwulff@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240928150905.2616313-1-crwulff@gmail.com> References: <20240928150905.2616313-1-crwulff@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Chris Wulff This fixes two problems with the UAC1 descriptors. bInterval for full-speed is now set to 1. Prior to this fix all speeds were set to 4. Super-speed descriptors are now built dynamically the same way as the other speeds. The prior implementation had a fixed set of descriptors and didn't take the presence of volume function units into account. Both of these changes need the refactoring of setup_descriptor to have a separate setup_header which is called for each speed. This implementation closely follows what was done in f_uac2. Fixes: b8fb6db6cb04 ("usb: f_uac1: adds support for SS and SSP") Signed-off-by: Chris Wulff --- drivers/usb/gadget/function/f_uac1.c | 248 ++++++++++++++++++++++----- 1 file changed, 202 insertions(+), 46 deletions(-) diff --git a/drivers/usb/gadget/function/f_uac1.c b/drivers/usb/gadget/func= tion/f_uac1.c index eeedcfa61fa1..f68d444d1961 100644 --- a/drivers/usb/gadget/function/f_uac1.c +++ b/drivers/usb/gadget/function/f_uac1.c @@ -128,7 +128,25 @@ static struct uac_feature_unit_descriptor *in_feature_= unit_desc; static struct uac_feature_unit_descriptor *out_feature_unit_desc; =20 /* AC IN Interrupt Endpoint */ -static struct usb_endpoint_descriptor ac_int_ep_desc =3D { +static struct usb_endpoint_descriptor fs_ac_int_ep_desc =3D { + .bLength =3D USB_DT_ENDPOINT_SIZE, + .bDescriptorType =3D USB_DT_ENDPOINT, + .bEndpointAddress =3D USB_DIR_IN, + .bmAttributes =3D USB_ENDPOINT_XFER_INT, + .wMaxPacketSize =3D cpu_to_le16(2), + .bInterval =3D 1, +}; + +static struct usb_endpoint_descriptor hs_ac_int_ep_desc =3D { + .bLength =3D USB_DT_ENDPOINT_SIZE, + .bDescriptorType =3D USB_DT_ENDPOINT, + .bEndpointAddress =3D USB_DIR_IN, + .bmAttributes =3D USB_ENDPOINT_XFER_INT, + .wMaxPacketSize =3D cpu_to_le16(2), + .bInterval =3D 4, +}; + +static struct usb_endpoint_descriptor ss_ac_int_ep_desc =3D { .bLength =3D USB_DT_ENDPOINT_SIZE, .bDescriptorType =3D USB_DT_ENDPOINT, .bEndpointAddress =3D USB_DIR_IN, @@ -137,6 +155,14 @@ static struct usb_endpoint_descriptor ac_int_ep_desc = =3D { .bInterval =3D 4, }; =20 +static struct usb_ss_ep_comp_descriptor ss_ac_int_ep_desc_comp =3D { + .bLength =3D sizeof(ss_ac_int_ep_desc_comp), + .bDescriptorType =3D USB_DT_SS_ENDPOINT_COMP, + .bMaxBurst =3D 0, + .bmAttributes =3D 0, + .wBytesPerInterval =3D cpu_to_le16(2), +}; + /* B.4.1 Standard AS Interface Descriptor */ static struct usb_interface_descriptor as_out_interface_alt_0_desc =3D { .bLength =3D USB_DT_INTERFACE_SIZE, @@ -208,7 +234,17 @@ static struct uac_format_type_i_discrete_descriptor as= _out_type_i_desc =3D { }; =20 /* Standard ISO OUT Endpoint Descriptor */ -static struct usb_endpoint_descriptor as_out_ep_desc =3D { +static struct usb_endpoint_descriptor fs_as_out_ep_desc =3D { + .bLength =3D USB_DT_ENDPOINT_AUDIO_SIZE, + .bDescriptorType =3D USB_DT_ENDPOINT, + .bEndpointAddress =3D USB_DIR_OUT, + .bmAttributes =3D USB_ENDPOINT_SYNC_ADAPTIVE + | USB_ENDPOINT_XFER_ISOC, + .wMaxPacketSize =3D cpu_to_le16(UAC1_OUT_EP_MAX_PACKET_SIZE), + .bInterval =3D 1, +}; + +static struct usb_endpoint_descriptor hs_as_out_ep_desc =3D { .bLength =3D USB_DT_ENDPOINT_AUDIO_SIZE, .bDescriptorType =3D USB_DT_ENDPOINT, .bEndpointAddress =3D USB_DIR_OUT, @@ -238,8 +274,18 @@ static struct uac_format_type_i_discrete_descriptor as= _in_type_i_desc =3D { .bSamFreqType =3D 0, /* filled on rate setup */ }; =20 -/* Standard ISO OUT Endpoint Descriptor */ -static struct usb_endpoint_descriptor as_in_ep_desc =3D { +/* Standard ISO IN Endpoint Descriptor */ +static struct usb_endpoint_descriptor fs_as_in_ep_desc =3D { + .bLength =3D USB_DT_ENDPOINT_AUDIO_SIZE, + .bDescriptorType =3D USB_DT_ENDPOINT, + .bEndpointAddress =3D USB_DIR_IN, + .bmAttributes =3D USB_ENDPOINT_SYNC_ASYNC + | USB_ENDPOINT_XFER_ISOC, + .wMaxPacketSize =3D cpu_to_le16(UAC1_OUT_EP_MAX_PACKET_SIZE), + .bInterval =3D 1, +}; + +static struct usb_endpoint_descriptor hs_as_in_ep_desc =3D { .bLength =3D USB_DT_ENDPOINT_AUDIO_SIZE, .bDescriptorType =3D USB_DT_ENDPOINT, .bEndpointAddress =3D USB_DIR_IN, @@ -249,7 +295,7 @@ static struct usb_endpoint_descriptor as_in_ep_desc = =3D { .bInterval =3D 4, }; =20 -/* Class-specific AS ISO OUT Endpoint Descriptor */ +/* Class-specific AS ISO IN Endpoint Descriptor */ static struct uac_iso_endpoint_descriptor as_iso_in_desc =3D { .bLength =3D UAC_ISO_ENDPOINT_DESC_SIZE, .bDescriptorType =3D USB_DT_CS_ENDPOINT, @@ -259,7 +305,41 @@ static struct uac_iso_endpoint_descriptor as_iso_in_de= sc =3D { .wLockDelay =3D 0, }; =20 -static struct usb_descriptor_header *f_audio_desc[] =3D { +static struct usb_descriptor_header *f_audio_fs_desc[] =3D { + (struct usb_descriptor_header *)&ac_interface_desc, + (struct usb_descriptor_header *)&ac_header_desc, + + (struct usb_descriptor_header *)&usb_out_it_desc, + (struct usb_descriptor_header *)&io_out_ot_desc, + (struct usb_descriptor_header *)&out_feature_unit_desc, + + (struct usb_descriptor_header *)&io_in_it_desc, + (struct usb_descriptor_header *)&usb_in_ot_desc, + (struct usb_descriptor_header *)&in_feature_unit_desc, + + (struct usb_descriptor_header *)&fs_ac_int_ep_desc, + + (struct usb_descriptor_header *)&as_out_interface_alt_0_desc, + (struct usb_descriptor_header *)&as_out_interface_alt_1_desc, + (struct usb_descriptor_header *)&as_out_header_desc, + + (struct usb_descriptor_header *)&as_out_type_i_desc, + + (struct usb_descriptor_header *)&fs_as_out_ep_desc, + (struct usb_descriptor_header *)&as_iso_out_desc, + + (struct usb_descriptor_header *)&as_in_interface_alt_0_desc, + (struct usb_descriptor_header *)&as_in_interface_alt_1_desc, + (struct usb_descriptor_header *)&as_in_header_desc, + + (struct usb_descriptor_header *)&as_in_type_i_desc, + + (struct usb_descriptor_header *)&fs_as_in_ep_desc, + (struct usb_descriptor_header *)&as_iso_in_desc, + NULL, +}; + +static struct usb_descriptor_header *f_audio_hs_desc[] =3D { (struct usb_descriptor_header *)&ac_interface_desc, (struct usb_descriptor_header *)&ac_header_desc, =20 @@ -271,7 +351,7 @@ static struct usb_descriptor_header *f_audio_desc[] =3D= { (struct usb_descriptor_header *)&usb_in_ot_desc, (struct usb_descriptor_header *)&in_feature_unit_desc, =20 - (struct usb_descriptor_header *)&ac_int_ep_desc, + (struct usb_descriptor_header *)&hs_ac_int_ep_desc, =20 (struct usb_descriptor_header *)&as_out_interface_alt_0_desc, (struct usb_descriptor_header *)&as_out_interface_alt_1_desc, @@ -279,7 +359,7 @@ static struct usb_descriptor_header *f_audio_desc[] =3D= { =20 (struct usb_descriptor_header *)&as_out_type_i_desc, =20 - (struct usb_descriptor_header *)&as_out_ep_desc, + (struct usb_descriptor_header *)&hs_as_out_ep_desc, (struct usb_descriptor_header *)&as_iso_out_desc, =20 (struct usb_descriptor_header *)&as_in_interface_alt_0_desc, @@ -288,7 +368,7 @@ static struct usb_descriptor_header *f_audio_desc[] =3D= { =20 (struct usb_descriptor_header *)&as_in_type_i_desc, =20 - (struct usb_descriptor_header *)&as_in_ep_desc, + (struct usb_descriptor_header *)&hs_as_in_ep_desc, (struct usb_descriptor_header *)&as_iso_in_desc, NULL, }; @@ -312,7 +392,7 @@ static struct usb_ss_ep_comp_descriptor ss_as_out_ep_de= sc_comp =3D { /* wBytesPerInterval =3D DYNAMIC */ }; =20 -/* Standard ISO OUT Endpoint Descriptor */ +/* Standard ISO IN Endpoint Descriptor */ static struct usb_endpoint_descriptor ss_as_in_ep_desc =3D { .bLength =3D USB_DT_ENDPOINT_AUDIO_SIZE, .bDescriptorType =3D USB_DT_ENDPOINT, @@ -337,8 +417,14 @@ static struct usb_descriptor_header *f_audio_ss_desc[]= =3D { =20 (struct usb_descriptor_header *)&usb_out_it_desc, (struct usb_descriptor_header *)&io_out_ot_desc, + (struct usb_descriptor_header *)&out_feature_unit_desc, + (struct usb_descriptor_header *)&io_in_it_desc, (struct usb_descriptor_header *)&usb_in_ot_desc, + (struct usb_descriptor_header *)&in_feature_unit_desc, + + (struct usb_descriptor_header *)&ss_ac_int_ep_desc, + (struct usb_descriptor_header *)&ss_ac_int_ep_desc_comp, =20 (struct usb_descriptor_header *)&as_out_interface_alt_0_desc, (struct usb_descriptor_header *)&as_out_interface_alt_1_desc, @@ -346,7 +432,6 @@ static struct usb_descriptor_header *f_audio_ss_desc[] = =3D { =20 (struct usb_descriptor_header *)&as_out_type_i_desc, =20 - //(struct usb_descriptor_header *)&as_out_ep_desc, (struct usb_descriptor_header *)&ss_as_out_ep_desc, (struct usb_descriptor_header *)&ss_as_out_ep_desc_comp, (struct usb_descriptor_header *)&as_iso_out_desc, @@ -357,7 +442,6 @@ static struct usb_descriptor_header *f_audio_ss_desc[] = =3D { =20 (struct usb_descriptor_header *)&as_in_type_i_desc, =20 - //(struct usb_descriptor_header *)&as_in_ep_desc, (struct usb_descriptor_header *)&ss_as_in_ep_desc, (struct usb_descriptor_header *)&ss_as_in_ep_desc_comp, (struct usb_descriptor_header *)&as_iso_in_desc, @@ -1082,6 +1166,10 @@ uac1_ac_header_descriptor *build_ac_header_desc(stru= ct f_uac1_opts *opts) /* Use macro to overcome line length limitation */ #define USBDHDR(p) (struct usb_descriptor_header *)(p) =20 +static void setup_headers(struct f_uac1_opts *opts, + struct usb_descriptor_header **headers, + enum usb_device_speed speed); + static void setup_descriptor(struct f_uac1_opts *opts) { /* patch descriptors */ @@ -1137,44 +1225,90 @@ static void setup_descriptor(struct f_uac1_opts *op= ts) ac_header_desc->wTotalLength =3D cpu_to_le16(len); } =20 + setup_headers(opts, f_audio_fs_desc, USB_SPEED_FULL); + setup_headers(opts, f_audio_hs_desc, USB_SPEED_HIGH); + setup_headers(opts, f_audio_ss_desc, USB_SPEED_SUPER); +} + +static void setup_headers(struct f_uac1_opts *opts, + struct usb_descriptor_header **headers, + enum usb_device_speed speed) +{ + struct usb_ss_ep_comp_descriptor *epout_desc_comp =3D NULL; + struct usb_ss_ep_comp_descriptor *epin_desc_comp =3D NULL; + struct usb_ss_ep_comp_descriptor *ep_int_desc_comp =3D NULL; + struct usb_endpoint_descriptor *epout_desc; + struct usb_endpoint_descriptor *epin_desc; + struct usb_endpoint_descriptor *ep_int_desc; + int i; + + switch (speed) { + case USB_SPEED_FULL: + epout_desc =3D &fs_as_out_ep_desc; + epin_desc =3D &fs_as_in_ep_desc; + ep_int_desc =3D &fs_ac_int_ep_desc; + break; + case USB_SPEED_HIGH: + epout_desc =3D &hs_as_out_ep_desc; + epin_desc =3D &hs_as_in_ep_desc; + ep_int_desc =3D &hs_ac_int_ep_desc; + break; + default: + epout_desc =3D &ss_as_out_ep_desc; + epin_desc =3D &ss_as_in_ep_desc; + epout_desc_comp =3D &ss_as_out_ep_desc_comp; + epin_desc_comp =3D &ss_as_in_ep_desc_comp; + ep_int_desc =3D &ss_ac_int_ep_desc; + ep_int_desc_comp =3D &ss_ac_int_ep_desc_comp; + } + i =3D 0; - f_audio_desc[i++] =3D USBDHDR(&ac_interface_desc); - f_audio_desc[i++] =3D USBDHDR(ac_header_desc); + headers[i++] =3D USBDHDR(&ac_interface_desc); + headers[i++] =3D USBDHDR(ac_header_desc); =20 if (EPOUT_EN(opts)) { - f_audio_desc[i++] =3D USBDHDR(&usb_out_it_desc); - f_audio_desc[i++] =3D USBDHDR(&io_out_ot_desc); + headers[i++] =3D USBDHDR(&usb_out_it_desc); + headers[i++] =3D USBDHDR(&io_out_ot_desc); if (FUOUT_EN(opts)) - f_audio_desc[i++] =3D USBDHDR(out_feature_unit_desc); + headers[i++] =3D USBDHDR(out_feature_unit_desc); } =20 if (EPIN_EN(opts)) { - f_audio_desc[i++] =3D USBDHDR(&io_in_it_desc); - f_audio_desc[i++] =3D USBDHDR(&usb_in_ot_desc); + headers[i++] =3D USBDHDR(&io_in_it_desc); + headers[i++] =3D USBDHDR(&usb_in_ot_desc); if (FUIN_EN(opts)) - f_audio_desc[i++] =3D USBDHDR(in_feature_unit_desc); + headers[i++] =3D USBDHDR(in_feature_unit_desc); } =20 - if (FUOUT_EN(opts) || FUIN_EN(opts)) - f_audio_desc[i++] =3D USBDHDR(&ac_int_ep_desc); + if (FUOUT_EN(opts) || FUIN_EN(opts)) { + headers[i++] =3D USBDHDR(ep_int_desc); + if (ep_int_desc_comp) + headers[i++] =3D USBDHDR(ep_int_desc_comp); + } =20 if (EPOUT_EN(opts)) { - f_audio_desc[i++] =3D USBDHDR(&as_out_interface_alt_0_desc); - f_audio_desc[i++] =3D USBDHDR(&as_out_interface_alt_1_desc); - f_audio_desc[i++] =3D USBDHDR(&as_out_header_desc); - f_audio_desc[i++] =3D USBDHDR(&as_out_type_i_desc); - f_audio_desc[i++] =3D USBDHDR(&as_out_ep_desc); - f_audio_desc[i++] =3D USBDHDR(&as_iso_out_desc); + headers[i++] =3D USBDHDR(&as_out_interface_alt_0_desc); + headers[i++] =3D USBDHDR(&as_out_interface_alt_1_desc); + headers[i++] =3D USBDHDR(&as_out_header_desc); + headers[i++] =3D USBDHDR(&as_out_type_i_desc); + headers[i++] =3D USBDHDR(epout_desc); + if (epout_desc_comp) + headers[i++] =3D USBDHDR(epout_desc_comp); + + headers[i++] =3D USBDHDR(&as_iso_out_desc); } if (EPIN_EN(opts)) { - f_audio_desc[i++] =3D USBDHDR(&as_in_interface_alt_0_desc); - f_audio_desc[i++] =3D USBDHDR(&as_in_interface_alt_1_desc); - f_audio_desc[i++] =3D USBDHDR(&as_in_header_desc); - f_audio_desc[i++] =3D USBDHDR(&as_in_type_i_desc); - f_audio_desc[i++] =3D USBDHDR(&as_in_ep_desc); - f_audio_desc[i++] =3D USBDHDR(&as_iso_in_desc); + headers[i++] =3D USBDHDR(&as_in_interface_alt_0_desc); + headers[i++] =3D USBDHDR(&as_in_interface_alt_1_desc); + headers[i++] =3D USBDHDR(&as_in_header_desc); + headers[i++] =3D USBDHDR(&as_in_type_i_desc); + headers[i++] =3D USBDHDR(epin_desc); + if (epin_desc_comp) + headers[i++] =3D USBDHDR(epin_desc_comp); + + headers[i++] =3D USBDHDR(&as_iso_in_desc); } - f_audio_desc[i] =3D NULL; + headers[i] =3D NULL; } =20 static int f_audio_validate_opts(struct g_audio *audio, struct device *dev) @@ -1410,44 +1544,66 @@ static int f_audio_bind(struct usb_configuration *c= , struct usb_function *f) =20 /* allocate AC interrupt endpoint */ if (FUOUT_EN(audio_opts) || FUIN_EN(audio_opts)) { - ep =3D usb_ep_autoconfig(cdev->gadget, &ac_int_ep_desc); + ep =3D usb_ep_autoconfig(cdev->gadget, &fs_ac_int_ep_desc); if (!ep) goto err_free_fu; + + hs_ac_int_ep_desc.bEndpointAddress =3D fs_ac_int_ep_desc.bEndpointAddres= s; + ss_ac_int_ep_desc.bEndpointAddress =3D fs_ac_int_ep_desc.bEndpointAddres= s; + uac1->int_ep =3D ep; - uac1->int_ep->desc =3D &ac_int_ep_desc; + uac1->int_ep->desc =3D &fs_ac_int_ep_desc; =20 ac_interface_desc.bNumEndpoints =3D 1; } =20 /* allocate instance-specific endpoints */ if (EPOUT_EN(audio_opts)) { - ep =3D usb_ep_autoconfig(cdev->gadget, &as_out_ep_desc); + ep =3D usb_ep_autoconfig(cdev->gadget, &fs_as_out_ep_desc); if (!ep) goto err_free_fu; - ss_as_out_ep_desc.bEndpointAddress =3D as_out_ep_desc.bEndpointAddress; + + hs_as_out_ep_desc.bEndpointAddress =3D fs_as_out_ep_desc.bEndpointAddres= s; + ss_as_out_ep_desc.bEndpointAddress =3D fs_as_out_ep_desc.bEndpointAddres= s; + ss_as_out_ep_desc_comp.wBytesPerInterval =3D ss_as_out_ep_desc.wMaxPacke= tSize; + audio->out_ep =3D ep; - audio->out_ep->desc =3D &as_out_ep_desc; + audio->out_ep->desc =3D &fs_as_out_ep_desc; } =20 if (EPIN_EN(audio_opts)) { - ep =3D usb_ep_autoconfig(cdev->gadget, &as_in_ep_desc); + ep =3D usb_ep_autoconfig(cdev->gadget, &fs_as_in_ep_desc); if (!ep) goto err_free_fu; - ss_as_in_ep_desc.bEndpointAddress =3D as_in_ep_desc.bEndpointAddress; + + hs_as_in_ep_desc.bEndpointAddress =3D fs_as_in_ep_desc.bEndpointAddress; + ss_as_in_ep_desc.bEndpointAddress =3D fs_as_in_ep_desc.bEndpointAddress; + ss_as_in_ep_desc_comp.wBytesPerInterval =3D ss_as_in_ep_desc.wMaxPacketS= ize; + audio->in_ep =3D ep; - audio->in_ep->desc =3D &as_in_ep_desc; + audio->in_ep->desc =3D &fs_as_in_ep_desc; } =20 setup_descriptor(audio_opts); =20 /* copy descriptors, and track endpoint copies */ - status =3D usb_assign_descriptors(f, f_audio_desc, f_audio_desc, f_audio_= ss_desc, + status =3D usb_assign_descriptors(f, f_audio_fs_desc, f_audio_hs_desc, f_= audio_ss_desc, f_audio_ss_desc); if (status) goto err_free_fu; =20 - audio->out_ep_maxpsize =3D le16_to_cpu(as_out_ep_desc.wMaxPacketSize); - audio->in_ep_maxpsize =3D le16_to_cpu(as_in_ep_desc.wMaxPacketSize); + audio->in_ep_maxpsize =3D max_t(u16, + le16_to_cpu(fs_as_in_ep_desc.wMaxPacketSize), + le16_to_cpu(hs_as_in_ep_desc.wMaxPacketSize)); + audio->out_ep_maxpsize =3D max_t(u16, + le16_to_cpu(fs_as_out_ep_desc.wMaxPacketSize), + le16_to_cpu(hs_as_out_ep_desc.wMaxPacketSize)); + + audio->in_ep_maxpsize =3D max_t(u16, audio->in_ep_maxpsize, + le16_to_cpu(ss_as_in_ep_desc.wMaxPacketSize)); + audio->out_ep_maxpsize =3D max_t(u16, audio->out_ep_maxpsize, + le16_to_cpu(ss_as_out_ep_desc.wMaxPacketSize)); + audio->params.c_chmask =3D audio_opts->c_chmask; memcpy(audio->params.c_srates, audio_opts->c_srates, sizeof(audio->params.c_srates)); --=20 2.43.0