Received: by 2002:ac0:a591:0:0:0:0:0 with SMTP id m17-v6csp1679826imm; Fri, 6 Jul 2018 04:43:26 -0700 (PDT) X-Google-Smtp-Source: AAOMgpcF1aubJXS/MxYi0CnI1aekm2cmV2G7nc2OCgXsWUgFa5xym2/meJWEH1BpEr7DR1vwGkaN X-Received: by 2002:a62:850d:: with SMTP id u13-v6mr10379182pfd.131.1530877406736; Fri, 06 Jul 2018 04:43:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530877406; cv=none; d=google.com; s=arc-20160816; b=sy+zyJWLTW0V6htE83dh7G3rTxpJPp9hK9ZVY6Po+sGWKF59k7G3VBNC0VIhmnHyjl jCEg0tmwm26pXZXHSaNVX1KSyh3fi05gtOxvwgIk/gvh3qOoFIB4PpwL0I3CVppVTsYS SRMYMCrtOejh7EoViiQbCCg5GNbYfZ/QRzr9YL0agOB9/4t+Zg4asRtceQKQ16bF6WR5 AqaKPT4rePMjAo+in6dvopMa/U7cOqKpINhvu6PHS4xednCbbMUN21MotvQli7GpN9uT Z2DlZ/p82e1HF9jozunlDW7JSwqmxvVVgHrHO2/uwSAm2oglA4H+fAXsZCvc+yJms175 NUhw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=B5vkC3CP3b5x8NMSBNyxvxl+zpU9ZetzuiVgtc1E/to=; b=g03+lv7fmxES0QesdEpGPMXFcxOkjJdZWStu1EdBHTiuKzQbcozYU1QDHNP472QaRs 1Fy+DZLaLAAKBwfPXNtRitZBXedOtrfmHkmLwHK7jtpdGBMlUQ/7Hy9XYP+v9pXGdZoX MBebgSDiLFIelE0dqBlZtqlNYk+R8ut28uI0+w1H3/EE8JbpG2r0M6+2W6otSvZ/DUjW AQ4kXvlKLR+KpNHZRlSr4kwrGPGFgQ7Va/Ef+6fWrzDaqqlYFocOnlbYHa2E5L6vnGt9 tN9ZJ4ehvZN3muVhBK3qw9iz7nF1PTc1pCnKevQbkXy62MSemju4FXVnh59vMz3bWlAf Q0Eg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t127-v6si7926974pfb.303.2018.07.06.04.43.11; Fri, 06 Jul 2018 04:43:26 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932730AbeGFLmc (ORCPT + 99 others); Fri, 6 Jul 2018 07:42:32 -0400 Received: from smtp1.de.adit-jv.com ([62.225.105.245]:41370 "EHLO smtp1.de.adit-jv.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932325AbeGFLma (ORCPT ); Fri, 6 Jul 2018 07:42:30 -0400 Received: from localhost (smtp1.de.adit-jv.com [127.0.0.1]) by smtp1.de.adit-jv.com (Postfix) with ESMTP id E1DB33C09AC; Fri, 6 Jul 2018 13:42:28 +0200 (CEST) Received: from smtp1.de.adit-jv.com ([127.0.0.1]) by localhost (smtp1.de.adit-jv.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xwcU2gppkTFg; Fri, 6 Jul 2018 13:42:21 +0200 (CEST) Received: from HI2EXCH01.adit-jv.com (hi2exch01.adit-jv.com [10.72.92.24]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp1.de.adit-jv.com (Postfix) with ESMTPS id D29B53C0160; Fri, 6 Jul 2018 13:42:21 +0200 (CEST) Received: from vmlxhi-102.adit-jv.com (10.72.93.184) by HI2EXCH01.adit-jv.com (10.72.92.24) with Microsoft SMTP Server (TLS) id 14.3.399.0; Fri, 6 Jul 2018 13:42:21 +0200 Date: Fri, 6 Jul 2018 13:42:18 +0200 From: Eugeniu Rosca To: Ruslan Bilovol CC: Eugeniu Rosca , Felipe Balbi , Felipe Balbi , Greg Kroah-Hartman , Vincent Pelletier , , , Eugeniu Rosca Subject: Re: [PATCH] usb: gadget: f_uac2: fix endianness of 'struct cntrl_*_lay3' Message-ID: <20180706114218.GA15693@vmlxhi-102.adit-jv.com> References: <20180702214647.1574-1-erosca@de.adit-jv.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) X-Originating-IP: [10.72.93.184] Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Ruslan, On Tue, Jul 03, 2018 at 01:36:17AM +0300, Ruslan Bilovol wrote: > On Tue, Jul 3, 2018 at 12:46 AM, Eugeniu Rosca wrote: > > The list [1] of commits doing endianness fixes in USB subsystem is long > > due to below quote from USB spec Revision 2.0 from April 27, 2000: > > > > ------------ > > 8.1 Byte/Bit Ordering > > > > Multiple byte fields in standard descriptors, requests, and responses > > are interpreted as and moved over the bus in little-endian order, i.e. > > LSB to MSB. > > ------------ > > > > This commit belongs to the same family. > > > > [1] Example of endianness fixes in USB subsystem: > > commit 14e1d56cbea6 ("usb: gadget: f_uac2: endianness fixes.") > > commit 42370b821168 ("usb: gadget: f_uac1: endianness fixes.") > > commit 63afd5cc7877 ("USB: chaoskey: fix Alea quirk on big-endian hosts") > > commit 74098c4ac782 ("usb: gadget: acm: fix endianness in notifications") > > commit cdd7928df0d2 ("ACM gadget: fix endianness in notifications") > > commit 323ece54e076 ("cdc-wdm: fix endianness bug in debug statements") > > commit e102609f1072 ("usb: gadget: uvc: Fix endianness mismatches") > > list goes on > > > > Fixes: 132fcb460839 ("usb: gadget: Add Audio Class 2.0 Driver") > > Signed-off-by: Eugeniu Rosca > > Good catch! > > Reviewed-by: Ruslan Bilovol Thanks for the review. FWIW, I've accidentally noticed post-push that a similar fix has been incorporated by you in the UAC3 implementation from https://lkml.org/lkml/2017/11/6/1514 ("[PATCH 1/1] usb: gadget: add USB Audio Device Class 3.0 gadget support"). Best regards, Eugeniu.