From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 820B1EB64DD for ; Tue, 25 Jul 2023 05:39:12 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 7B43FDF5; Tue, 25 Jul 2023 07:38:19 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 7B43FDF5 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1690263549; bh=oEqOvaYTtYD0cDhlp47C+7XbAkxvJEjRKWIZtt4DxJc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=iMh6JdfZWSiwZ4+keUd91p5pLL9WO1Di1aGAPB2ZTv9mZVAVqd3V7gq/TSuVCyO2F wgZTC9SnvfnfPsp5xTbdZJHmFSVOt9o1dYb1KuMNMfs0r3QhdR/WJoqqTssiDIhKZD JnxbIEBfWlqe7tt3exVc4l19syn4N1hSXwWeo3nA= Received: by alsa1.perex.cz (Postfix, from userid 50401) id 8A2E0F80163; Tue, 25 Jul 2023 07:37:53 +0200 (CEST) Received: from mailman-core.alsa-project.org (mailman-core.alsa-project.org [10.254.200.10]) by alsa1.perex.cz (Postfix) with ESMTP id AF400F8019B; Tue, 25 Jul 2023 07:37:52 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id ACFEBF8019B; Tue, 25 Jul 2023 07:36:06 +0200 (CEST) Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id D9C48F80153 for ; Tue, 25 Jul 2023 07:35:46 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz D9C48F80153 Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key, unprotected) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.a=rsa-sha256 header.s=korg header.b=i9kVXjf1 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id C25FC61526; Tue, 25 Jul 2023 05:35:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id AE699C433C7; Tue, 25 Jul 2023 05:35:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1690263345; bh=oEqOvaYTtYD0cDhlp47C+7XbAkxvJEjRKWIZtt4DxJc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=i9kVXjf1HBDSjHedbx4wMwqTpNqKR8EFUdQJlMmhXfBLthblmuZkU77Z1IhdaR0wK x1OLHGpuEV3kutCyQ2/LY1/oaReoB/VEuw+Gs4FrtcJVkBIZkitCPYTSkInvuDJmCG SETBLrckjO2tXk+VvPn76Wm6pQeqppYZjuaQHbRk= Date: Tue, 25 Jul 2023 07:35:42 +0200 From: Greg KH To: Wesley Cheng Cc: agross@kernel.org, andersson@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, catalin.marinas@arm.com, will@kernel.org, mathias.nyman@intel.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, srinivas.kandagatla@linaro.org, bgoswami@quicinc.com, Thinh.Nguyen@synopsys.com, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-usb@vger.kernel.org, alsa-devel@alsa-project.org, quic_jackp@quicinc.com, pierre-louis.bossart@linux.intel.com, oneukum@suse.com, albertccwang@google.com, o-takashi@sakamocchi.jp, Mathias Nyman Subject: Re: [PATCH v4 01/32] xhci: add support to allocate several interrupters Message-ID: <2023072520-footbath-pessimist-670d@gregkh> References: <20230725023416.11205-1-quic_wcheng@quicinc.com> <20230725023416.11205-2-quic_wcheng@quicinc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230725023416.11205-2-quic_wcheng@quicinc.com> Message-ID-Hash: MJDLK5ZNKGGJZAJRVX5RKCH2XUW7MO6V X-Message-ID-Hash: MJDLK5ZNKGGJZAJRVX5RKCH2XUW7MO6V X-MailFrom: gregkh@linuxfoundation.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-alsa-devel.alsa-project.org-0; header-match-alsa-devel.alsa-project.org-1; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.8 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Mon, Jul 24, 2023 at 07:33:45PM -0700, Wesley Cheng wrote: > From: Mathias Nyman > > Modify the XHCI drivers to accommodate for handling multiple event rings in > case there are multiple interrupters. Add the required APIs so clients are > able to allocate/request for an interrupter ring, and pass this information > back to the client driver. This allows for users to handle the resource > accordingly, such as passing the event ring base address to an audio DSP. > There is no actual support for multiple MSI/MSI-X vectors. > > Factoring out XHCI interrupter APIs and structures done by Wesley Cheng, in > order to allow for USB class drivers to utilze them. > > Signed-off-by: Mathias Nyman > Co-developed-by: Wesley Cheng > Signed-off-by: Wesley Cheng > --- > arch/arm64/boot/dts/qcom/sm8350.dtsi.rej | 14 ++++ > arch/arm64/configs/defconfig.rej | 24 ++++++ > drivers/usb/host/xhci-debugfs.c | 2 +- > drivers/usb/host/xhci-mem.c | 93 ++++++++++++++++++++++-- > drivers/usb/host/xhci-ring.c | 2 +- > drivers/usb/host/xhci.c | 49 ++++++++----- > drivers/usb/host/xhci.h | 77 +------------------- > include/linux/usb/xhci-intr.h | 86 ++++++++++++++++++++++ > 8 files changed, 245 insertions(+), 102 deletions(-) > create mode 100644 arch/arm64/boot/dts/qcom/sm8350.dtsi.rej > create mode 100644 arch/arm64/configs/defconfig.rej > create mode 100644 include/linux/usb/xhci-intr.h > > diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi.rej b/arch/arm64/boot/dts/qcom/sm8350.dtsi.rej > new file mode 100644 > index 000000000000..e96ef1120160 > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi.rej > @@ -0,0 +1,14 @@ > +--- arch/arm64/boot/dts/qcom/sm8350.dtsi > ++++ arch/arm64/boot/dts/qcom/sm8350.dtsi > +@@ -2254,9 +2254,9 @@ > + iommus = <&apps_smmu 0x0 0x0>; > + snps,dis_u2_susphy_quirk; > + snps,dis_enblslpm_quirk; > +- phys = <&usb_1_hsphy>, <&usb_1_qmpphy QMP_USB43DP_USB3_PHY>; > ++ phys = <&usb_1_hsphy>, <&usb_1_qmpphy>; > + phy-names = "usb2-phy", "usb3-phy"; > +- > ++ dr_mode = "host"; > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > diff --git a/arch/arm64/configs/defconfig.rej b/arch/arm64/configs/defconfig.rej I think you need to look at this patch series a bit better and clean it up before sending it out. These files obviously are not going to work, and prevent any of this from being able to be applied or tested by any bots or test systems :( I've stopped here in my review of this series (except for the obvious global symbol name comment.) thanks, greg k-h