From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9963D186E33; Mon, 12 Aug 2024 16:35:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723480523; cv=none; b=ftE8tPQC1ECoYg8x1uB65sUA05BWBv+WvOn/9FFIyWTHNECu9hB6cOLmqeEnZ9GuNIPlbTekDSfAckgMlXKqXDrfsgBNo3wQ3fKjb2DP63kjjkdctPKJmUXDA5gTmMkBgOvzTlGkjMkUu2w9TqMFFLSJHf+U4fON+idnrf7UBv8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723480523; c=relaxed/simple; bh=QXurh/k7PfitLh6A6D2g7uq6l2qrPlZGC1wMeb3E/Hc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QnN+zIlhqleby1tdOdySj91DMIhZgaDxH0JVXOfaAcEN1xTLy1DeZPO9em29UJO3cnC7tfPeBhDsDvQc2lsQv7ZSmWTuKSExMT9Chl6IWaQMH29MI4Uw+ZhtIvK8cu5dX+0QfHsB8TsPVwe5y8bOiDFd3LrpQIWRrVYguO7aFj4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=FcZv2cNC; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="FcZv2cNC" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1FBCCC32782; Mon, 12 Aug 2024 16:35:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1723480523; bh=QXurh/k7PfitLh6A6D2g7uq6l2qrPlZGC1wMeb3E/Hc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=FcZv2cNCuZcDxF7Wb/tVYIVhFGajdsIIH1SyUC5F0tYxN/S3DpwxGDmmdwZAi4Fga E2fL/xvlEtN3amY2IhjgAlxmPJFRZQSZ//w5ItuF1f6mWncMkKM/HJF2oyjKzNTl/J NbU3AJDobLoeHL8mtwfyntgiLveIc+S2gas35V1U= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Takashi Iwai Subject: [PATCH 6.10 190/263] usb: gadget: midi2: Fix the response for FB info with block 0xff Date: Mon, 12 Aug 2024 18:03:11 +0200 Message-ID: <20240812160153.819217656@linuxfoundation.org> X-Mailer: git-send-email 2.46.0 In-Reply-To: <20240812160146.517184156@linuxfoundation.org> References: <20240812160146.517184156@linuxfoundation.org> User-Agent: quilt/0.67 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.10-stable review patch. If anyone has any objections, please let me know. ------------------ From: Takashi Iwai commit 228a953e61d6d608a3facc1c3a27b9fb03c99de7 upstream. When the block number 0xff is given to Function Block Discovery message, the device should return the information of all Function Blocks, but currently the gadget driver treats it as an error. Implement the proper behavior for the block 0xff instead. Fixes: 8b645922b223 ("usb: gadget: Add support for USB MIDI 2.0 function driver") Cc: stable@vger.kernel.org Signed-off-by: Takashi Iwai Link: https://lore.kernel.org/r/20240717095102.10493-1-tiwai@suse.de Signed-off-by: Greg Kroah-Hartman --- drivers/usb/gadget/function/f_midi2.c | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) --- a/drivers/usb/gadget/function/f_midi2.c +++ b/drivers/usb/gadget/function/f_midi2.c @@ -642,12 +642,21 @@ static void process_ump_stream_msg(struc if (format) return; // invalid blk = (*data >> 8) & 0xff; - if (blk >= ep->num_blks) - return; - if (*data & UMP_STREAM_MSG_REQUEST_FB_INFO) - reply_ump_stream_fb_info(ep, blk); - if (*data & UMP_STREAM_MSG_REQUEST_FB_NAME) - reply_ump_stream_fb_name(ep, blk); + if (blk == 0xff) { + /* inquiry for all blocks */ + for (blk = 0; blk < ep->num_blks; blk++) { + if (*data & UMP_STREAM_MSG_REQUEST_FB_INFO) + reply_ump_stream_fb_info(ep, blk); + if (*data & UMP_STREAM_MSG_REQUEST_FB_NAME) + reply_ump_stream_fb_name(ep, blk); + } + } else if (blk < ep->num_blks) { + /* only the specified block */ + if (*data & UMP_STREAM_MSG_REQUEST_FB_INFO) + reply_ump_stream_fb_info(ep, blk); + if (*data & UMP_STREAM_MSG_REQUEST_FB_NAME) + reply_ump_stream_fb_name(ep, blk); + } return; } }